jquery $.each无法终止循环

Question

问题描述：当输入的内容和列表中的sku值相同时复核数量加一，当输入内容和表中的sku匹配不上时弹出‘查无此商品信息’，为什么有时候if、else部分都会弹出，有时候就正常，求指教 用‘全局变量’flag无效，求指教

//enter
$('.confirm_review').click(function () {

$('tbody tr td:nth-child(2)').each(function(index,val){
  if($(this).html()==$('.nav-input').val()){
    var count = $(this).parent().children().eq(-2).html();
    count++;
    $(this).parent().children().eq(-2)[0].innerHTML= count;
  }else{
    layer.msg('订单中无此条码的商品',{icon:5});
  }

});

});

Answer 1

if和else不管执行哪一个，只要出现了异常，很明显是你的判断有问题，这还需要问吗？
首先，你应该在外面用一个变量来接收$('.nav-input').val()的值，而不是在循环中去直接用，因为这会导致多次循环获取dom元素，这样性能不好。

Answer 2

这段代码确实槽多无口。。。
$('.nav-input').val()应该赋值在each外面，因为获取DOM对象很消耗资源

另外，请解释一下
$(this).parent().children().eq(-2).html()
是什么意思？获取当前元素的倒数第二个兄弟元素？siblings了解一下

另外，既然用了jquery，那么给html赋值就尽量别用innerHTML了，而应该用.html(count)

最重要的一点是：
表格应该是根据数据源渲染的，查找数据也应该去数据源里查找，修改数据后再根据数据重新渲染表格