Как можно изменить?

[kohozabr]

У меня есть код

var answer = prompt("Где находится Дубаи? А - в Англии, Б - в Италии, В - в Эмиратах", "");

if(answer == "В" || answer == "в" || answer == "в Эмиратах" || answer == "в эмиратах" || answer == "В ЭМИРАТАХ" || answer == "В Эмиратах"){
alert("Ура! Вы победили!");
}
else{
alert("Вы проиграли. Попробуйте снова.");
}

Как его можно сократить? Хотя бы, чтобы не было вариантов "в Эмиратах", "в эмиратах", "В Эмиратах" и "В ЭМИРАТАХ".

[ctpz]

Попробуйте так:

var answer = prompt("Где находится Дубаи? А - в Англии, Б - в Италии, В - в Эмиратах", "");

answer = answer.toLowerCase;

if(answer == "в" || answer == "в эмиратах"){
alert("Ура! Вы победили!");
}
else{
alert("Вы проиграли. Попробуйте снова.");
}

Можно еще регуляркой сделать, но я это только начал изучать.

[kohozabr]

Можно еще регуляркой сделать...

А что такое регулярка? Регулярные выражения? Если да, то не плохо было бы увидеть подобное.

Edited August 2, 2011 by kohozabr

[Int]

if (answer.match(/^в\s*(эмиратах)?/i)) alert('Правильно');
else alert('Не правильно');

[Nekromancer]

Проще по моему выводить popup окно с 3мя вариантами ответов в видел radio. Мало ли что впишут в ответ ...

[kohozabr]

А как это сделать? Как сделать popup окно?

[ctpz]

http://g.zeos.in/?q=popup%20%D0%BE%D0%BA%D0%BD%D0%BE%20js

[kohozabr]

ctpz, +1 за остроумность

[ruslan.savenok]

Я так понимаю такой вопрос будет не один и приходить они будут либо с сервера либо сразу в js жить, прекрасно было бы использовать JSON

{
 question: 'вопрос',
 answers: [
  { answer: 'ответ 1', right: true },
  { answer: 'ответ 2', right: false }
 ]
}

Если хотите делать через prompt:

var q = {
 question: 'вопрос',
 answers: [
  { answer: 'ответ 1', right: true },
  { answer: 'ответ 2', right: false }
 ]
};

function generateQuestion(dataObj) {
    var str = dataObj.question + ' ?\n';

    for(var i=0; i<dataObj.answers.length; i++) {
     str += i+1 + '. ' + dataObj.answers[i].answer + '\n';
    }

    var result = prompt(str);

/*
тут надо проверить результат, действительно ли юзер ввел число иначе будет ошибка.

С буквами сделать так же просто, надо лишь создать массив букв.
*/

    if(dataObj.answers[parseInt(result)-1].right) {
     alert('Вы ответили верно');
    } else {
     alert('Облажался!');
    }
}

generateQuestion(q);

На данной основе так же легко сделать popup div c генерацией radio кнопок

Edited August 2, 2011 by ruslan.savenok

[psywalker]

if (answer.match(/^в\s*(эмиратах)?/i)) alert('Правильно');
else alert('Не правильно');

А зачем ты матчишь? Есть же спец. метод под названием test?

	var text = /^в(\s*эмиратах)?/ig;
	if(text.test(answer)) alert('Правильно');
	else alert('Не правильно');

[Int]

И что, такая огромная разница между test и match на современном железе для строки в несколько байт?

[psywalker]

И что, такая огромная разница между test и match на современном железе для строки в несколько байт?

Да, разница есть. match - это более тяжёлый метод, и нужен не для проверок, а для возвращения массива с нужными данными, а вот test как раз лёгкий, проверочный метод и сюда подходит лучше всего. Имхо.

[Int]

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

[psywalker]

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

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

[ruslan.savenok]

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

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

Да ну, я тоже не люблю test