为什么在列表组项目上('点击')之后保留以前点击的ID?

时间:2016-07-08 09:57:15

标签: javascript jquery twitter-bootstrap-3

我有以下部分的聊天室代码:

HTML

<div class="list-group">
  <a class="list-group-item" id="1" href="#">Group A</a>
  <a class="list-group-item" id="2" href="#">Group B</a>
  <a class="list-group-item" id="3" href="#">Group C</a>
  <a class="list-group-item" id="4" href="#">Group D</a>
</div>

<div class="message_box">
  <form action="#" class="form">
    <div class="form-group"></div>
    <div class="input-group">
      <textarea class="form-control message_text emojiable-message"
                name="message_text" id="message_text" rows="3"></textarea>
      <span class="input-group-addon btn btn-primary send_message_btn" name="send_message_btn"
            >
        Send
      </span>
    </div>
  </form>
</div>

的jQuery

$(document).ready(function () {
    $('#groups a.list-group-item').on('click', function () {
    $(this).toggleClass("active");
        $(this).siblings().removeClass("active");

        //chage the textarea and btn id to single user.
        $('.send_message_btn').removeAttr('id');
        $('.send_message_btn').attr('id', 'send_to_topic');

        var selected_group_id = this.id;

        //Handle enter key press and mouse click (group)
        $('#send_to_topic').click(function(e) {

            alert(selected_group_id);

            //sendChatMessage(selected_group_id);
            return false;
        });

        $('#message_text').keydown(function(e) {
            if (e.keyCode == 13) {

                alert(selected_group_id);

                //sendChatMessage(selected_group_id);
                return false;
            }
        });

        $(this).removeClass('active');
        $(this).removeAttr('id');
  });
});

问题是当我选择多个list-group-item然后点击发送按钮时,alert(selected_group_id);会打印我选择的所有项目的ID。我只想选择要在sendChatMessage(selected_group_id);中使用的最后一个选定项目。如果有人能够指出出错的地方会很棒。感谢。

  

编辑。   主要问题是,当我点击发送时,邮件会以重复的形式发送(我在发送之前选择的所有list-group-item ID的数量)。如果我只选择一个组并发送消息不重复(这没关系。)。

2 个答案:

答案 0 :(得分:0)

稍微修改了HTML和JS代码,并在此处工作。请看一下。

$(document).ready(function() {
  $('a.list-group-item').on('click', function() {
    $(this).toggleClass("active");
    $(this).siblings().removeClass("active");
  });

  //Handle enter key press and mouse click (group)
  $('#send_to_topic').click(function(e) {

    var selected_group_id = $(".list-group .active").attr("id");
    alert(selected_group_id);

    return false;
  });

  $('#message_text').keydown(function(e) {
    if (e.keyCode == 13) {
      var selected_group_id = $(".list-group .active").attr("id");
      alert(selected_group_id);

      //sendChatMessage(selected_group_id);
      return false;
    }
  });

});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.0/jquery.min.js"></script>
<div class="list-group">
  <a class="list-group-item" id="1" href="#">Group A</a>
  <a class="list-group-item" id="2" href="#">Group B</a>
  <a class="list-group-item" id="3" href="#">Group C</a>
  <a class="list-group-item" id="4" href="#">Group D</a>
</div>

<div class="message_box">
  <form action="#" class="form">
    <div class="form-group"></div>
    <div class="input-group">
      <textarea class="form-control message_text emojiable-message" name="message_text" id="message_text" rows="3"></textarea>
      <span class="input-group-addon btn btn-primary send_message_btn" name="send_message_btn" id="send_to_topic">
        Send
      </span>
    </div>
  </form>
</div>

答案 1 :(得分:-1)

正如vijayP建议的那样,您的$('#send_to_topic').click()$('#message_text').keydown()应该在$('#groups a.list-group-item').click()事件之外生效。

接下来,由于您的selected_group_id变量是使用var关键字声明的,因此$('#groups a.list-group-item').click()范围之外的函数无法访问它。 所以你应该在所有这些事件之外声明这个变量,如下所示:

<强>的jQuery

$(document).ready(function () {

    // Declare the selected_group_id outside of the events so they can access to it
    var selected_group_id;

    $('#groups a.list-group-item').on('click', function () {
        $(this).toggleClass("active");
        $(this).siblings().removeClass("active");

        //chage the textarea and btn id to single user.
        $('.send_message_btn').removeAttr('id');
        $('.send_message_btn').attr('id', 'send_to_topic');

        selected_group_id = this.id;

        $(this).removeClass('active');
        $(this).removeAttr('id');
    });

    //Handle enter key press and mouse click (group)
    $('#send_to_topic').click(function(e) {

        alert(selected_group_id);

        //sendChatMessage(selected_group_id);
        return false;
    });

    $('#message_text').keydown(function(e) {
        if (e.keyCode == 13) {

            alert(selected_group_id);

            //sendChatMessage(selected_group_id);
            return false;
        }
    });
});