Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+71
public class ... {
public Double toDoubleValue(String s){
Double value;
if (s!=""){
value = Double.valueOf(s);
return value;
}
else {
value = 0.00;
return value;
}
....
toDoubleValue(row.getColumnValue(col.getId()).toString());
}
1. На null не проверяется
2. Можно было тоже самое написать короче, не объявляя дополнительный метод, с помощью ?:
Запостил: 
-EZ-,
02 Мая 2012
Follow us!
....
следует читать
}
....
2) Легче исправлять баги - к примеру проверку на null, если она действительно понадобится, нужно будет добавлять только в одно место а не в каждый тернарный оператор.
P.S. естественно если метод вызывался 1 раз - то тернарка лучше
Всё предельно ясно и без лишних переменных
P.S. А row.getColumnValue() что возвращает?
меня одного такая проверка смущает?
s.intern() != "" точно будет работать всегда, но очень быстро придет ООМ, при частых выборках разных данных