我究竟做错了什么?没有错误

时间:2010-09-17 15:37:59

标签: php jquery ajax

function showComments(wallID){
    $.ajax({
      url: "misc/showComments.php",
             type: "POST",
      data: { mode: 'ajax', wallID: wallID }, 
      success: function(msg){

      var $msg = $('#showWallCommentsFor'+wallID).find('.userWallComment');
// if it already has a comment, fade it out, add the text, then toggle it back in
if ( $msg.text().length ) {
  $msg.fadeOut('fast', function(){
    $msg.text( msg ).slideToggle(300); 
  });
} else {
  // otherwise just hide it, add the text, and then  toggle it in
  $msg.hide().text( msg ).slideToggle(300); 
}
      }
    });
}

msg,我得到的回应:(萤火虫)

    <span class='userWallComment'>
<span style='float: left;'>
<img style='border: 1px solid #ccc; width: 44px; height: 48px; margin-right: 8px;' src='images/profilePhoto/thumbs/noPhoto_thumb.jpg'>
</span></span>
<span style='font-size: 10px; margin-bottom: 2px;'>
<a href='profil.php?id=1'>Navn navn</a> - igår kl. 01:55
</span>
<br>
DETTE ER EN TEST
<br>
<div class="clearfloat"></div>
</span>

它正确地发送和执行ajax调用,它有一些响应,但它没有切换它?

这是div:

<div id="showWallCommentsFor<?php echo $displayWall["id"]; ?>" style="display: none;">
</div>

1 个答案:

答案 0 :(得分:1)

问题

您的if - else声明存在缺陷:

if ( $msg.text().length ) {
  //  ...
} else {
  // $msg has a length of ZERO by definition here!!!
  $msg.hide().text( msg ).slideToggle(300); 
}

第一次触发AJAX调用时#showWallCommentsFor为空,因此它内部没有.userWallComment因此,$msg将不会被定义。

解决方案

您应该使用以下内容将文字直接添加到else中的原始div中

if ( $msg.text().length ) {
  //  ...
} else {
    // otherwise just hide it, add the text, and then  toggle it in
      // You cannot use $msg here, since it has a length of 0.
      // Add text directly to the original div instead.
      // You do not need to hide the DIV first since it is already 
      // invisible.
    $('#showWallCommentsFor'+wallID).text( msg ).slideToggle(300); 
 }

最后,在else中,您无需.hide() #showWall... div,因为由于style="display: none;",div可能是不可见的。