String sent through a socket doesn't match what it should be

Question

Using a PrintWriter in java (server) to send data through a socket:

JAVA

out = new PrintWriter(client.getOutputStream(), true);
out.println("p1");

Then when I get this value in flash (client):

FLASH AS3

line = socket.readUTFBytes(socket.bytesAvailable);

This if statement is not run:

if (line == "p1") {

And when I trace line to, I get p1 in the output (though when I put in a breakpoint and run in debug, it shows line as being equal to "p1, rather than "p1").

Aside from anything else, string comparisons with `==` don't work as you expect them to. See http://stackoverflow.com/questions/513832/how-do-i-compare-strings-in-java — Jon Skeet, Oct 23 '13 at 13:43
[Why doesn’t == work on String?](http://stackoverflow.com/questions/17443201/why-doesnt-work-on-string/17443215#17443215) — Suresh Atta, Oct 23 '13 at 13:45
Maybe I didn't make it very clear, but the if statement is in flash AS3, and == for strings work fine in it as far as I know. — IBOED2, Oct 23 '13 at 13:52
I did mention that line was in flash, sorry if I didn't make it very clear. — IBOED2, Oct 23 '13 at 13:55

Brian Agnew · Accepted Answer · 2013-10-23T13:53:46.297

2

Probably because

out.println("p1");

will append a line separator (line feed and/or carriage return, depending on platform, configuration etc.). I suspect that's why your debugger show the value as "p1 (since the next line would contain the closing quote). I suspect you want:

out.print("p1");

and close or flush the writer (as appropriate).

I note your comment that your string equality is performed in Flash (so the .equals() comments don't apply)

edited Oct 23 '13 at 13:53

answered Oct 23 '13 at 13:41

Brian Agnew

268,207
37
334
440

`out.print("p1");` Doesn't seem to do anything, because flash is waiting for an end of line character I think? – IBOED2 Oct 23 '13 at 13:45
Note my comment re. closing/flushing – Brian Agnew Oct 23 '13 at 13:46

Prabhakaran Ramaswamy · Answer 2 · 2013-10-23T13:45:10.513

0

To compare String objects in java use .equals() method instead of "==" operator

if (line == "p1")

should be

if (line.equals("p1"))

edited Oct 23 '13 at 13:45

answered Oct 23 '13 at 13:43

Prabhakaran Ramaswamy

25,706
10
57
64

score -1 · Answer 3 · answered Oct 23 '13 at 13:51

-1

Check with line.toString()
if (line.toString() == "p1") {

answered Oct 23 '13 at 13:51

Pankaj Tiwari

71
2
8

Why? No reason given. Is this even legal Flash? If it's Java, `line` already is a `String.` Pointless answer. – user207421 Oct 23 '13 at 23:43

String sent through a socket doesn't match what it should be

3 Answers3