1

По заданию:Напишите функцию checkSpam(str), которая возвращает true, если строка str содержит „viagra“ или „XXX“, а иначе false.Функция должна быть нечувствительна к регистру.

Вот такой вопрос: а если не ставить 2 восклицательных знака(двойное НЕ)... обе проверки выдают 0, получаем false - то, что и надо по условию, одна или обе проверки дают значение больше или меньше нуля - это true - опять же удовлетворяет условию ...

Вопрос зачем нужно это двойное отрицание(true-false-true;false-true-false)?

Оно тут не лишнее ??

<!doctype html>
<html lang="en">

<head>
  <meta charset="utf-8">
  <title>Just a Generic Page</title>
  <script>
    function checkSpam(str) {
      var lowstr = str.toLowerCase();

      return !!(~lowstr.indexOf("viagra") || ~lowstr.indexOf("XXX"));
    }
    checkSpam('buy ViAgRA now');
    checkSpam('free xxxxx');
    checkSpam("innocent rabbit");
  </script>
</head>

<body>


</body>

</html>
3
  • 2
    в данном случае результаты функции checkSpam вообще не используются, значит можно удалить функцию полностью
    – Grundy
    5 ноя 2017 в 7:02
  • Классный троллинг)) ну а всё таки.. нужно ли тут двойное НЕ?) 5 ноя 2017 в 7:06
  • все зависит от того как будет использоваться данная функция. !! гарантирует, что выходное значение будет либо true, либо false. То есть все будет работать в случае проверок на равенство checkSpam('buy ViAgRA now') == true. Если убрать, то выражение checkSpam('buy ViAgRA now') == true вернет false
    – Grundy
    5 ноя 2017 в 7:12

1 ответ 1

3

Логический оператор ! всегда возвращает тип boolean, поэтому двойное применение этого оператора можно использовать для конвертирования объектов в этот тип.

console.log(!true);
console.log(true);
console.log(!0);
console.log(!!0);

Операция || примененная к числам возвращает либо левое число если оно не 0, либо правое. Двойное отрицание в данном случае необходимо чтобы гарантировать контракт - функция должна всегда возвращать тип boolean. Наличие этой операции не влияет на результат только если его позднее привести к типу boolean.

function checkSpam(str) {
  var lowstr = str.toLowerCase();

  return (~lowstr.indexOf("viagra") || ~lowstr.indexOf("XXX"));
}
console.log(checkSpam('buy ViAgRA now'));
console.log(checkSpam('free xxxxx'));
console.log(checkSpam("innocent rabbit"));

Чтобы избавиться от необходимости использовать двойное отрицание, необходимо изменить код таким образом, чтобы он всегда оперировал со значениями типа boolean.

function checkSpam(str) {
  var lowstr = str.toLowerCase();

  return (lowstr.indexOf("viagra") != -1 || lowstr.indexOf("XXX") != -1);
}
console.log(checkSpam('buy ViAgRA now'));
console.log(checkSpam('free xxxxx'));
console.log(checkSpam("innocent rabbit"));

4
  • есть два оператора ! и !!. = ты не прав. Оператор один !
    – Grundy
    5 ноя 2017 в 7:26
  • @Grundy, логично. переписал это место
    – Uranus
    5 ноя 2017 в 7:30
  • 1
    в последним примере ~ не нужен можно сразу сравнивать с -1
    – Grundy
    5 ноя 2017 в 7:38
  • @Grundy, согласен. так будет правильнее. Функцию видимо написал человек, которые не любит простых решений :-)
    – Uranus
    5 ноя 2017 в 8:57

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.