Question

我这段代码有什么问题，为什么点击两次“删除”才能有反应呢？

$("#lookinner #deletelook").click(function(data){
if(!$("#lookinner #mylook").hasClass("selected")){
alert("系统表情不能删除。请点击‘我的表情’再删除");
}else if($("#lookinner #looklist div img[class=selected]").size() == 0){
alert("请选择要删除的表情");
}else {
var i;
successCount = 0;
for(i=0;i<$("#lookinner #looklist div img[class=selected]").size();i++) {
var mylookid = $("#lookinner #looklist div img[class=selected]").eq(i).attr("id");
$.ajax({
type:"POST",
url:"deleteLook.php",
data:"mylookimgid="+mylookid,
success:function(data){
msg = $.trim(data);
}
});
if(msg == "表情删除成功"){
successCount++;
}else{
alert(mylookid + "表情删除失败，删除动作中断。");
break;
}
}
if(successCount == $("#lookinner #looklist div img[class=selected]").size()) {
alert("选中表情全部删除成功！");
}
}
});

Answer 1

是否在加载函数里面执行了。
你的html代码发出来看看

追问

导入导出
删除表情
添加表情

追答

$("#lookinner #looklist div img[class=selected]")

建议用IE 调试看看，一步一步的调试，看看执行到哪里去了。
你这句代码怀疑已经报错了。
looklist 这个对象没有，第一次点击估计是停止到了这里或者什么的报错了，然后等这个对象加载出来了，第二次点击，就成功了。

Answer 2

你这样去调试一下，在每个判定的地方alert 一下数据，看看走的方式是不是按照你预想的流程走的，就能看到什么地方出现错误了

追问

谢谢你的回答！我这段代码是能够达到我预想的目标的：就是从多个表情图片中选择几个，然后一起删除，但是就是必须点击两次（不是双击）才能弹出“选中表情全部删除成功",不知道是怎么回事。

追答

你么太懂我的意思，我说的是，你先在每个地方alert一下，看看走的流程为啥是点击了2次才走全部删除呢个地方，看看是哪个地方出现了问题，比如，你第一次点击的时候都走了，就是不走最下面的 全部删除，上一行代码出现了问题，

追问

我在for循环结束后alert了一下sucessCount这个变量，也是需要点击两次才能显示出来，是不是我的 for循环有逻辑问题？