“Индюк” - это профессия!
Убеждаешься в этом каждый раз, когда смотришь результаты выполнения тестовых заданий. Вот и в очередной раз
повеселился на славу 
private Hashtable<Integer, Integer> statisticMap = new Hashtable<Integer, Integer>();
public synchronized void incrementCountById(int uid){
boolean ifExist = ifHasRecordInMap(uid);
//..дословно звучит как "если правда, что что-то существует - правда"
if( true == ifExist ) {
int numCount = getCountShowById(uid);
statisticMap.put(uid, ++numCount);
}
}
Предел индючизма - сама функция определения, существует ли значение ключа в мапе.
public synchronized boolean ifHasRecordInMap(int uid){
boolean returnValue = false;
int k = -1;
//statisticMap = new Hashtable<Integer, Integer>();
for( Iterator key=statisticMap.keySet().iterator(); key.hasNext(); ){
k = Integer.parseInt(key.next().toString());
if( k == uid ){
returnValue = true;
break;
}
}
return returnValue;
}
Ну и еще пару перлов на последок
public ArrayList getAllIdOFFiles(){
List<Integer> returnValue = new ArrayList<Integer>();
….
return (ArrayList) returnValue;
}
String saveResult = "false";
try {
doSomeThing();
saveResult = "true";
} catch(Exception e) {
throw new ServletException(e);
}
if( "true".equals(saveResult) ){
doSomeThingElse();
}
}
if (0 < -this.Position.Y) {...}
P.S. Господи, не дай нам быть индюками!
Изменен
