Почему JSLint говорит, что «анонимная функция не всегда возвращает значение» для обработчика события?

Question

Когда я JSLint этот код:

$(document).keydown(function(e){ 
    if ($("#chaptersFunctionality").length !== 0) { 
     if (e.keyCode == '13') 
      return false; 
     if ($("#kelvin").val() === "" && $("#caleb").attr("title") === "Show all steps") { 
      switch(e.which){ 
      case 39: if ((parseInt(currentAnchor)+1) < parseInt(maxi)) 
       expandMe(++currentAnchor); 
       break; 
      case 37: if (currentAnchor > 0) 
       expandMe(currentAnchor-1); 
       break; 
      } 
     } 
    } 
}); 

... Я получаю много предупреждений, большинство из которых я ГРОК, но это один имеет меня в тупик: «анонимная функция не всегда возвращает значение»

Почему это так? Обработчик событий не является анонимной функцией, не так ли? И все равно это не возвращает никакой ценности, не так ли? Итак, что это значит и что будет [moll, pac] ify linter?

Answer 1

JSLint предупреждает вас, когда функция имеет несколько ветвей, которые сталкиваются с return и другими ветвями, которые заканчиваются без столкновения с return. JSLint считает, что функция должна либо всегда явно возвращать значение, либо никогда явно возвращает значение. Функция, которая иногда явно возвращает значение, сама получает предупреждение. Ваш код делает return false в одном случае и в противном случае никогда не использует return.

Если вы хотите успокоить JSLint, вы можете разместить return undefined; в нижней части своей функции, чтобы он всегда явно возвращал что-то.

Теперь, по-моему:

Когда дело доходит до слушателей событий, это 100% приемлемо для возврата значения только иногда. Внутренний движок браузера, который использует ваше событие прослушивателя return, уже запрограммирован на прием возвращаемого значения или отсутствие возвращаемого значения.

Поскольку практика не возвращать значение в прослушивателе событий совершенно нормально, это не повредит читабельности вашего кода. Я думаю, вы можете смело игнорировать JSLint в этом случае.