Проверка NaN или <= 0

Question

Jshint не доволен этим выражением:

var amount = $('.some-input').val() 
// Check for NaN or <= 0 
if (!(+amount > 0)) { 
    ... 
} 

Он утверждает запутанным использование!. Есть ли способ переписать выражение без добавления слишком большого количества кода?

Answer 1

Эквивалент без отрицания:

if (amount <= 0 || isNaN(amount)) { 

Таким образом, вместо того, чтобы говорить «не больше, чем», вы говорите «Меньше или равно». Кроме того, не все могут знать, что NaN действительно не может сравниться с числами (т. Е. Всегда возвращает false), и поэтому явный тест может облегчить чтение и понимание этого выражения.

Видимо, люди находят отрицания более трудными для понимания, и я бы не сказал, что нет правды на это.

Answer 2

Вы могли бы написать

if (amount > 0 === false) { 

строка автоматически преобразуется в число, когда в этом сравнении, и любая строка, которая не может быть правильно разобрано как число преобразуется в NaN, который не является >0. И jshint, кажется, радует, когда видит === false вместо удобного !.

Но реальное решение кажется не Использование jshint, что не имеет отношения к данному вопросу. Этот инструмент не позволяет использовать возможности JavaScript.

Если вы решили отказаться от jshint, я бы до сих пор советую вам упростить немного ваш тест как

if (!(amount > 0)) { 

Качество мудрым, самым важным здесь было бы, на мой взгляд небольшой комментарий вы уже добавили.