Need help for refactoring this code

[Chetan Parekh]

CheckStyle is giving Cyclomatic Complexity value 11 for following method, while the maximum allowed is 10.

Is there any scope for Refactoring?

//Here data1, data2... data10 are class level variables.
 
public String getValue(int check) throws JRScriptletException {
 
String result = null;
switch (check) {
case 1: {
result = this.checkData(data1);
break;
}
case 2: {
result = this.checkData(data2);
break;
}
case 3: {
result = this.checkData(data3);
break;
}
case 4: {
result = this.checkData(data4);
break;
}
case 5: {
result = this.checkData(data5);
break;
}
case 6: {
result = this.checkData(data6);
break;
}
case 7: {
result = this.checkData(data7);
break;
}
case 8: {
result = this.checkData(data8);
break;
}
case 9: {
result = this.checkData(data9);
break;
}
case 10: {
result = this.checkData(data10);
}
default: {
result = null;
}
}
 
return result;
}

[Varun Chopra]

Why do you need default in this case?

[Kyle Brown]

Where does the number come from? Do you have to use a number or could you use a name? If so -- you might be able to use reflection instead.

[Ilja Preuss]

Well, even with the number, you could use reflection:

... getClass().getDeclaredField("data" + check);
...

It would be even simpler if you could make data an array:

result = checkData(data[check]);