2013-03-05 5 views
-3

Как поместить значение таблицы во всплывающее окно? Вместо того, чтобы выбранное значение таблицы, она выводит «неопределенный»Значение всплывающего окна

<script> 
    function conf() { 
    var x, res, y; 
    res = document.getElementById('t').onclick 
    if(document.formxml.num1.value >= 2) { 
     var r = confirm("Reserve for " + document.formxml.num1.value + " persons in "+document.formxml.res. + "?"); 
    } else if(document.formxml.num1.value == 1) { 
     var r = confirm("Reserve for " + document.formxml.num1.value + " person in " + document.formxml.res. + "?"); 
    } else if(document.formxml.num1.value < 0) { 
     var r = confirm("You cannot input a negative number!"); 
    } 

    if(r == true) { 
     x = ""; 
     location.href = 'res3.html' 
    } else if(r == false) { 
     x = ""; 
    } 
    } 
</script> 

</head> 
<body> 

    <style type="text/css"> 
    .txt {width:50px;} 
    </style> 

    <form action="" method="get" name="formxml"> 
    For how many persons? 
    <input type="text" name="num1" value="" class="txt" id="num1" maxlength="2"> 
    <input type="button" value="Table 1" id="t" name="res" onclick="conf();"> 
    <input type="button" value="Table 3" id="t" name="res" onclick="conf();"> 
    </form> 

Я не знаю, что дальше после var r=confirm("Reserve for "+document.formxml.num1.value+" person in "+document.formxml.res.+"?");

+3

По крайней мере, потратьте время на отступ от кода. – PeeHaa

ответ

2

Вы держите вашу r повторного объявления переменных в пределах каждого блока if, эффективно вызывая возвращаемое значение из confirm() чтобы никогда не доходить до линии, где вы ее читали.

Try объявить r раз и навсегда вместе с другими переменными:

var x, res, y, r; 

и падение var ключевое слово на линиях, где присваивают значения r.

Смежные вопросы