FindBugs Dead Store Предупреждение - Должен ли я реструктурировать код или игнорировать предупреждение

Question

Я получаю предупреждение DeadStore с Findbugs на int i ниже. Я предпочитаю не писать однострочные изображения из-за удобочитаемости. Есть ли лучший способ написать это, чтобы не было DeadStore до i, но как можно читать?

if (aqForm.getId() != null) { 
     try { 
      int i = Integer.parseInt(aqForm.getId()); 
      aqForm.setId(aqForm.getId().trim()); 
     } catch (NumberFormatException nfe) { 
      result.rejectValue("id", "error.id", "Please enter an integer."); 
      foundError = true; 
     } 
    } 

Answer 1

Вам не нужно назначить i. Вы можете просто позвонить parseInt() и игнорировать результат:

if (aqForm.getId() != null) { 
     try { 
      Integer.parseInt(aqForm.getId()); // validate by trying to parse 
      aqForm.setId(aqForm.getId().trim()); 
     } catch (NumberFormatException nfe) { 
      result.rejectValue("id", "error.id", "Please enter an integer."); 
      foundError = true; 
     } 
    } 

То есть, я бы создать вспомогательную функцию:

public static boolean isValidInteger(String str) { 
     ... 
    } 

и переписать свой фрагмент кода, как так:

String id = aqForm.getId(); 
    if (id != null) { 
     if (isValidInteger(id)) { 
     aqForm.setId(id.trim()); 
     } else { 
     result.rejectValue("id", "error.id", "Please enter an integer."); 
     foundError = true; 
     } 
    } 

Answer 2

Просто вызовите метод и игнорировать результат, в идеале с комментарием, чтобы объяснить, почему:

// Just validate 
Integer.parseInt(aqForm.getId()); 

Это не понятно, почему вы обрезку версию вы не проверенную, а не версия у вас есть, заметьте. Я бы предпочел:

String id = aqForm.getId(); 
if (id != null) { 
    try { 
     id = id.trim(); 
     // Validate the ID 
     Integer.parseInt(id); 
     // Store the "known good" value, post-trimming 
     aqForm.setId(id); 
    } catch (NumberFormatException nfe) { 
     result.rejectValue("id", "error.id", "Please enter an integer."); 
     foundError = true; 
    } 
}