FindBugs - обработка нулевой чек и кодирования при чтении входного потока

Question

Я добавил FindBugs плагин для моего проекта, и я вдруг начал получать следующую ошибку: Dereference of the result of readLine() without nullcheck

У меня есть следующий код, который считывает запрос HTTP построчно:

InputStream input = clientSocket.getInputStream(); 

String line; 
while (!(line = in.readLine()).equals("")) { 
... 
} 

Я пытался переписывать это в какой-то другой для с nullcheck:

String line = ""; 
while (line != null) { 
    line = in.readLine(); 

    if (line.equals("")) return; 
} 

Но это застревает навсегда (так что не переписан гр orrectly). Я извиняюсь за такой основной вопрос, но я не могу показаться, чтобы получить это право ...

Другое дело, что помечено как ошибка является Found reliance on default encoding in ..InputStream...

Как я могу указать кодировку в InputStreamReader?

Answer 1

Фиксированный цикл выглядит так:

InputStream input = clientSocket.getInputStream(); 

String line; 
while (null != (line = in.readLine())) { 
    if("".equals(line)) break; 
    ... 
} 

Почему? Прежде всего, удаленная сторона (клиент) закрывает соединение, readLine() вернет null. То, что нарушает внешняя проверка.

readLine() не будет вообще возвращаться, если клиент просто прекратит отправку данных. Таким образом, пока клиент держит соединение открытым, ваш «фиксированный» цикл зависает.

При сравнении строковых литералов, я всегда ставлю их первым:

"".equals(line)) 

никогда не выходит из строя, даже если line является null. Это также часто более читаемо, так как вы часто хотите знать, с чем вы сравниваете; переменная, которую вы хотите проверить, менее информативна.

Answer 2

Видимо readLine может вернуться null, так что вы должны проверить его после line = in.readLine();

Вашего обновленного кода все еще может бросить NullPointerException, если readLine вернулся null.

Answer 3

Я сомневаюсь, что ваши изменения будут работать, так как проверка производится на предыдущем значение строки, таким образом, если ваша предыдущая строка была действительной (но вы, где читаете последнюю строку), любые последующие вызовы могут потенциально a NullPointerException.

ходу вокруг этого, как правило, следующий шаблон применяется:

InputStream input = clientSocket.getInputStream(); 

String line = ""; 
while ((line = in.readLine()) != null) { 
    ... 
}