Simplify code

tmp = br.readLine();

tmp = tmp.substring(tmp.indexOf(':')+2);

if(tmp.compareTo("1")==0)

DataN.setSelectedIndex(0);

elseif(tmp.compareTo("2")==0)

DataN.setSelectedIndex(1);

elseif(tmp.compareTo("3")==0)

DataN.setSelectedIndex(2);

elseif(tmp.compareTo("4")==0)

DataN.setSelectedIndex(3);

elseif(tmp.compareTo("5")==0)

DataN.setSelectedIndex(4);

elseif(tmp.compareTo("6")==0)

DataN.setSelectedIndex(5);

elseif(tmp.compareTo("7")==0)

DataN.setSelectedIndex(6);

How can I simplify this code ?

thank u

[1246 byte] By [denisaa] at [2007-11-27 10:56:38]

# 1

tmp = br.readLine();

tmp = tmp.substring(tmp.indexof(':') +2);

int i = Integer.parseInt(tmp);

DatN.setSelectedIndex(i -1);

But I've omitted any checking for non-integers, and various other errors

georgemca at 2007-7-29 12:04:11 >

# 2

Learn about ints and Integer.parseInt

# 3

> tmp = br.readLine();

> tmp = tmp.substring(tmp.indexOf(':')+2);

> ...

>DataN.setSelectedIndex(6);

> How can I simplify this code ?

You could use a char instead of a substring

and use a switch-case instead of nested if.

tschodta at 2007-7-29 12:04:11 >

Java Essentials New Topic