Почему не работает&& в скрипте?

Question

[Александр шагин]

Не понимаю почему не работает код

Хочу подсчитать сколько раз пользователь выиграл сподряд.

код

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov' && count == 3) {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov' && count == 2) {
alert ("Поздравляем вы ответили верно два раза сподряд!")
count++;
} else if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov' && count == 1) {
alert('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

у меня проблема заключается в том что && не реагирует, т.е если условие которое я задал соблюдается только одно, он всё ровно выдает то что должно выдаваться только при двух правдах.

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

Comments

[Глеб Старков]

нет такого слова "сподряд"

[Александр шагин]

Глеб Старков, сподряд — См. последовательно... Словарь русских синонимов и сходных по смыслу выражений. под. ред. Н. Абрамова, М.: Русские словари, 1999. сподряд оптом, последовательно; подряд, сплошь Словарь русских синонимов …

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

ну а если все таки не понятно то имел ввиду сколько раз пользователь ответил правильно не ошибавшись, если ошибся то счёт обновлялся и считался заново, это тоже самое что комбо делаешь в играх.

Answer 1

[Андрей]

Как уже подсказали, не работает из-за неправильно поставленных скобок, а еще вам лучше немного изменить проверку, без стольких ifов

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') {
	switch(count) {
		case 1: 
			alert('Поздравляем ответ верный.')
			break;
		case 2: 
			alert ("Поздравляем вы ответили верно два раза сподряд!")
			break;
		case 3: 
			alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
			break;
	}
	count++;
} else {
	alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

И вывод поздравлений лучше вывести отдельной функцией

Comments

[Александр шагин]

программа заработала как нужно с кодом

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
  if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') {
    count++;
    switch(count) {
      case 1:
        alert('Поздравляем ответ верный.')
        break;
      case 2:
        alert ("Поздравляем вы ответили верно два раза сподряд!")
        break;
      case 3:
        alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
        break;
    }
  } else {
    alert ('Ответ не верный, рекомендуем больше читать Биографии.')
  }

Но сам вопрос остался для меня не решенным, а именно

почему при таком коде

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 3) {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 2) {
alert ("Поздравляем вы ответили верно два раза сподряд!")
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 1) {
alert('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

Программа выполняла только одну из условий, имею ввиду оператор && не соблюдался..

[Андрей]

Александр шагин, возможно count у вас не был равен ни одному из этих значений.
Чтобы выяснить наверняка, поставьте в начале (перед первым if)

console.log(a, count, (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov'))

И в консоли посмотрите какие значения у вас будут

[Александр шагин]

Протестировал ещё раз и понял что работает не совсем так как я хотел

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

т.к если к примеру на 1 -вый вопрос ответил верно.
если на второй не верно
на третий снова вено - то alert должен был выдать - Поздравляем ответ верный. Так как была нарушена последовательность правильных ответов.

а в данном коде он это не учитывает и просто учитывает - количество верных ответов. Без последовательности.

[Андрей]

Александр шагин, просто поменяйте последний блок с

else {
    alert ('Ответ не верный, рекомендуем больше читать Биографии.')
  }

на

else {
    alert ('Ответ не верный, рекомендуем больше читать Биографии.')
    count = 0
  }

[Александр шагин]

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

исправил новой переменной для подсчета - серии правильных ответов.

[Андрей]

Александр шагин, еще лучше хранить вопросы и ответы в виде объектов, тогда добавление и изменение вопросов и ответов будет очень простым, а код выводящий и проверяющий ответ будет компактным.

[Даниил]

2 минуса, может возникнуть исключение в

prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
//  может вернуть null, у null нету методов, NullPointerException
// возьми в try-catch

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

Answer 2

[Stalker_RED]

Для того чтобы оператор ИЛИ вернул true достаточно, чтобы выполнилось любое из условий. Когда одно из условий выполнилось, то что записано левее - не проверяется.

Но вы можете в скобки обернуть

if ( (условие1 ИЛИ условие2 ИЛИ условие3) И условие4) { . . . }

Comments

[Александр шагин]

Сделал скобки как посоветовали, теперь он пишет ответ не верный хотя ввожу правильный.

и не считает количество отвеченных сподряд правильных ответов.

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

a = prompt('В каком году распался СССР?').toLowerCase();
if ( (a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 2) {
alert ('Поздравляем вы ответили верно два раза сподряд!')
count++;
} else if ( (a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 1) {
alert ("Поздравляем ответ верный.")
count++;
} else {
alert('Ответ не верный, рекомендуем больше читать историю.')
}
вот что у меня получилось все ведь правильно записана

[Александр шагин]

Мне нужно что бы именно два условия соблюдалась и только тогда
alert true выдавать, если слева и справа выражения правдивы.

а он почему условие && не видит(

[Danila]

Александр шагин, ну так а count у тебя изначально чему равен? Я подозреваю что нулю, соответственно ни одно из условий никогда не сработает

Answer 3

[Даниил]

В скобки возьми все до && у тебя проверяется не со всем выражением а с предпоследним a == 'pavel durov' && count == 3

Comments

[Александр шагин]

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 3 {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 2 {
alert ("Поздравляем вы ответили верно два раза сподряд!")
count++;
} else if (a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 1 {
alert('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

сделал так, взял в скобки все до && теперь скрипт не запускается ругается на &&

[Глеб Старков]

Александр шагин, ну вы даёте.
Скобки должны быть вложенные.
((...)...), а не (...)...)

[Александр шагин]

Глеб Старков, я так уже делал

вот что получается

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 3) {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 2) {
alert ("Поздравляем вы ответили верно два раза сподряд!")
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 1) {
alert('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

при таком коде программа не видит правильные ответы, иногда пишет верный иногда не верный

весь код

var count = 0; // Переменная правильных ответов
var a; // Переменная для загадок
a = prompt('Угадайте кто живёт на дне океана?').toLowerCase();
if (a == "спанч боб" || a == "spanchbob" || a == 'спанчбоб' || a == 'губка боб') {
alert ('Поздравляю ответ верный')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше смотреть мультики')
}

if (count == 1) {
alert ('Игра окончена');
alert ('Вы слишком умный, у нас для вас есть ещё три загадки');
} else {
alert ('Не расстраивайтесь попробуйте угадать ещё три загадки')
}

a = prompt('В каком году распался СССР?').toLowerCase();
if ((a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 2) {
alert ('Поздравляем вы ответили верно два раза сподряд!')
count++;
} else if ((a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 1) {
alert ("Поздравляем ответ верный.")
count++;
} else {
alert('Ответ не верный, рекомендуем больше читать историю.')
}

a = prompt('Как зовут создателя Соц. сети Вконтакте?').toLowerCase();
if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 3) {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 2) {
alert ("Поздравляем вы ответили верно два раза сподряд!")
count++;
} else if ((a == "павел дуров" || a == "павел" || a == 'pavel' || a == 'pavel durov') && count == 1) {
alert('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать Биографии.')
}

a = prompt('Самая дорогая валюта в мире?').toLowerCase();
if ((a == "биткоин" || a == "bitcoin" || a == 'btc' || a == 'бтс') && count == 4) {
alert ('Вы гениальный человек, 4-тый раз сподряд вы отвечаете верно!')
count++;
} else if ((a == "биткоин" || a == "bitcoin" || a == 'btc' || a == 'бтс') && count == 3) {
alert ('Потрясающе уже три раза сподряд вы отвечаете верно!')
count++;
} else if ((a == "биткоин" || a == "bitcoin" || a == 'btc' || a == 'бтс') && count == 3) {
alert ('Поздравляем вы ответили верно два раза сподряд!')
count++;
} else if ((a == "биткоин" || a == "bitcoin" || a == 'btc' || a == 'бтс') && count == 1) {
alert ('Поздравляем ответ верный.')
count++;
} else {
alert ('Ответ не верный, рекомендуем больше читать о финансовом рынке.')
}

[Глеб Старков]

Александр шагин, вы не знаете, что здесь есть форматирование исходного кода?

[Александр шагин]

Глеб Старков, может, вы сюда пришли, что проблему решить или отвечать вопросам на вопрос?

Если не знаете решения проблемы, то зачем писать?

[Даниил]

Александр шагин, у вас слишком не красивый код, в плане объектно ориентированности,

У вас count ни когда не буде равна 2 и потом следовательно 3.
Плюс если вы нажмете отмена в promt() то вернете не строку, а не строка не имеет метода toLowerCase() // будет исключение вызвано, прога умрет
У вас не совсем корректно оповещение о в подряд угаданных, если угадаете 1, но не угадаете 2, а 3 вопрос решите, то он скажет что вы 2 раза в подряд угодали, но вы не в подряд угодали

/// count ни при каких условиях до этого не будет равно 2, либо 1 либо 2, 
/// вы до этого лишь один раз ее увеличивали,
if ((a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 2) {
alert ('Поздравляем вы ответили верно два раза сподряд!')
count++;
} else if ((a == '1991' || a == '1991 год' || a == 'тысяча девятьсот девяносто первом') && count == 1) {
alert ("Поздравляем ответ верный.")
count++;
} else {
alert('Ответ не верный, рекомендуем больше читать историю.')
}
// count 1 или 2 или 0, и ни 3

[Даниил]

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

function guessedResponse(){
    var count=0;  // число всех угаданных вопросов
    var questions=0; // число вподряд угаданых вопросов
    function convert( number){  // возращает строку по числу
        var map=["один","два","три", "четыре", "пять","шесть", "семь","восемь","девять", "десять"];
        if(typeof number =="number" && number>0 && number<10)
            return map[number-1];
        return null;
    }
    function check(question, arrResponseSuccess){ // проверяет правен ли ответ, если да то увеличивает сыетчик угаданных
        if(arrResponseSuccess instanceof Array){
            for(let i=0; i<arrResponseSuccess.length;i++){
                if(question===arrResponseSuccess[i]) {
                    count++;
                    return true;
                }
            }
            questions=0;
            return false;
        }
        else  return false; // моожно дополнить для 1 отета а не массива ответов
    }
    // эту функцию вернет эта функция, в которую мы будем добавлять вопрос и ответы и сообщение при неугаданном ответе
    return function appendQuestiuon(questionV, arrResponseSuccess, incorectUnswerAlert){

        questions++;
        try {
            var question = prompt(questionV).toLowerCase();
        }catch(error){
            // если была нажата отмена, то undefined не имеет метода toLowerCase()/ удет вызванно мсключение, перехватываем
            console.error("Вы не ввели ответ на вопрос");
            return;
        }
        if(check(question,arrResponseSuccess)){
            if(questions===count)
                alert ('Потрясающе уже '+ convert(questions)+ ' '+(questions===1?'раз':"раза")+' в подряд вы отвечаете верно!');
            else if(count===1)
                alert("Поздравляем ответ верный.");

            else {
                alert ('Поздравляем вы ответили верно '+convert(count) +" " +(count===1?'раз':"раза"));
            }
            return true;
        } else {
            alert(incorectUnswerAlert);
            return false;
        }
    }
}
// получаем функция, в которую будем передавать вопрос, массив ответов, и сообщение при неудачи ответа
var questions =guessedResponse();
// теперь questions эта наша функция appendQuestion(), мы можем ее использовать, вся логика скрыта в ней, нам лишь параметры передовать
var firstResult = questions('Угадайте кто живёт на дне океана?',[  "спанч боб" ,  "spanchbob" , 'спанчбоб',   'губка боб' ],
    'Ответ не верный, рекомендуем больше смотреть мультики');
/**************************************/
if (firstResult ) {
    alert ('Игра окончена');
    alert ('Вы слишком умный, у нас для вас есть ещё три загадки');
} else {
    alert ('Не расстраивайтесь попробуйте угадать ещё три загадки')
}
/*****************************************/
questions('В каком году распался СССР?',[  "1993","1993 году", 'тысяча девятьсот девяносто первом'],
        'Ответ не верный, рекомендуем больше читать историю.');
questions('Как зовут создателя Соц. сети Вконтакте',["павел дуров" , "павел" , 'pavel' , 'pavel durov'],
    'Ответ не верный, рекомендуем больше читать Биографии.' );