2012-03-21 13 views
-6
<script type="text/javascript"> 
//Confirm Delete 
function confirms(id) { 

    var test=confirm("Confirm Delete?"); 

    if(test== true) 
     window.location="<?php echo $_SERVER['PHP_SELF'].'?'.$_SERVER['QUERY_STRING']; ?>&did="+id; 
    else 
     window.location="<?php echo $_SERVER['PHP_SELF'].'?'.$_SERVER['QUERY_STRING']; ?>&did="+id; 
} 
</script> 
+5

'if(test == true)'は不要です。これは冗長です。 'if(test)'か 'if(test === true)'のどちらかです。あなたが持っているものは良いものではありません。また、 'if'ブロックと' else'ブロックはまったく同じことをしています(コードは同じです)。 –

+0

ifとelseの違いは何ですか?同じことをしています。 – punit

+0

ありがとう...ここで私はそれを管理しました... – Yogs

答えて

6

必ずしも真を返すとは限りません。スクリプトは、常に同じコードを常に実行します。これは以下と同じです:

<script type="text/javascript"> 
//Confirm Delete 
function confirms(id) { 
    window.location="<?php echo $_SERVER['PHP_SELF'].'?'.$_SERVER['QUERY_STRING']; ?>&did="+id; 
} 
</script> 
+0

function confirms(id) { var test = confirm( "Confirm Delete?"); if(test == true) { window.location = "<?php echo ADMINURLPATH;?> managemenu&did =" + id; がtrueを返します。 } } – Yogs

+0

これはなに? – Blender

+0

それは何もない....私はちょうど私のコードを変更し、それはうまく実行されます。いつもあなたのintrest ....ありがとう – Yogs

関連する問題