[SOLVED] Integer.parseInt() problem

[eggmanpete]

hi chaps. im having a bit of a problem with Integer.parseInt()

at run time, there is an error with line "int col = new Integer (Integer.parseInt(colStr));"
the field sentence is "x,y" x and y being two numbers.

Any help would be great,

code below:

while(true)
		{
			DatagramPacket receivePacket = new DatagramPacket(
				receiveByte, receiveByte.length);
			serverSocket.receive(receivePacket);
			InetAddress IP = receivePacket.getAddress();
			String sentence = new String(receivePacket.getData());
			int port = receivePacket.getPort();
			// data manipulated here:
			String[] splitInput = (String[]) sentence.split(",");

			// BUG: can't parseInt on splitInput[1]
			String colStr = new String (splitInput[0]);
			String rowStr = new String (splitInput[1]);
			int row = new Integer (Integer.parseInt(rowStr));
			int col = new Integer (Integer.parseInt(colStr));

			board.updateBoard(col, row);
			//String capsSentence = sentence.toUpperCase();

			// TODO something useful with coordinates
			newCoords = (sentence);
			System.out.println("reply: " + newCoords);
			sendByte = newCoords.getBytes();

			DatagramPacket sendPacket = new DatagramPacket(
				sendByte, sendByte.length, IP, port);
			serverSocket.send(sendPacket);
			receivePacket = null;
			sendPacket = null;
			sentence = null;
			newCoords = null;

			// clear any data in byte
			receiveByte = new byte[1024];
			sendByte = new byte[1024];

		}

[Eranga]

Are you sure that string colStr have a number format?

[CJSLMAN]

What is error you're getting?

CJSL

[eggmanpete]

sorry, the problem is with

int row = new Integer (Integer.parseInt(rowStr));

and yes, if I only leave

int col = new Integer (Integer.parseInt(colStr));

then it is fine.

There seems to be a problem with the second part of the "sentence" field.
no matter what the y part of the sentence is, it won't convert to type int.

[MK12]

is your string that you parse "3,1"? If so that's your problem. the comma, it isn't a number.

[Eranga]

First of all check that splitInput[0] and splitInput[1] values. It give the clear idea about your input string split correctly or not.

[eggmanpete]

No, the strings i parse are "3" and then "1".
sentence = "3,1"
splitInput = ["3"], ["1"]

[eggmanpete]

First of all check that splitInput[0] and splitInput[1] values. It give the clear idea about your input string split correctly or not.

done that,
System.out.println(splitInput[0]); = 3
System.out.println(splitInput[1]); = 1

[eggmanpete]

What is error you're getting?

CJSL

error when sentence = "3,1" is:

Exception in thread "main" java.lang.NumberFormatException: For input string: "1"


at java.lang.NumberFormatException.forInputString(Num berFormatException.java:48)
at java.lang.Integer.parseInt(Integer.java:456)
at java.lang.Integer.parseInt(Integer.java:497)
at networkUDP.udpserver.main(udpserver.java:35)
Java Result: 1

[CJSLMAN]

ah.. yes... I remember running into this before with the stupid UDP connection... try the following:

String sentence = new String(receivePacket.getData())[B][COLOR="Blue"].trim();[/COLOR][/B]

CJSL

[eggmanpete]

you sir, are a god.

Thanks.

[Eranga]

I cannot see number format exception on your code.

One more thing, no need to use redundant cast here.

String[] splitInput = (String[]) sentence.split(",");

[Eranga]

And this too.

String colStr = splitInput[0];
			String rowStr = splitInput[1];
			int row = Integer.parseInt(rowStr);
			int col = Integer.parseInt(colStr);

[CJSLMAN]

You're welcome... it really took me a while to figure it out when I ran into it and I haven't seen it mentioned in any books either.
Glad I could help.

CJSL

[eggmanpete]

I cannot see number format exception on your code.

One more thing, no need to use redundant cast here.

String[] splitInput = (String[]) sentence.split(",");

thanks, I left that there when i tried to force it being a string. :o

[Eranga]

So that means this is wrong.

error when sentence = "3,1"

It should be,

error when sentence = "3,1 "

additional space at the end. :rolleyes:

[Eranga]

thanks, I left that there when i tried to force it being a string. :o

split() always return a string array lol. ;)

[eggmanpete]

I'm new to java i guess

[Eranga]

Never mind lol. You can learn a lot form our community, a lot.

At the same time, if you have found the solution here for your question please mark the thread as solved. It's really helpful to all other members who are commenting and referring your question.