如何创建使用条形码的搜索

Question

我正在为使用条形码搜索产品的销售点创建一个搜索系统。当用户扫描条形码时，带有相应条形码的产品会被添加到购物车中。我使用Ajax将条形码传递给控制器​​。问题是，当产品添加到购物车时，生成的查询运行两次，使订单数量增加一倍。我不知道为什么会这样。

查看/搜索栏

<div class="frmSearch">
    <input type="text" id="search" name="search" class="form-control" placeholder="Type Product Name..."
           onmouseover="this.focus();"/>
</div>

<script type="text/javascript">
    $('#search').on('keyup', function () {
        $value = $(this).val();
        $.ajax({
            type: 'get',
            url: '{{URL::to('search')}}',
            data: {'search': $value},
            success: function (data) {
                window.location.href = "/addsale/" + data;
            }
        });
    });
</script>

调节器

public function search(Request $request)
{
    if ($request->ajax()) {
        $raws = DB::table('items_masters')->where('bcode', '=', $request->search)->first();

        if ($raws) {
            $output = $product->id;
        }

        return Response($output);
    }
}

将产品添加到购物车的方法

public function add($id)
{
    $userid = Auth::user()->id;
    $shop = Auth::user()->shop_id;
    $product_id = $id;

    $tdo = item_quantity::getQuery('item_quantities')->where([
        ['shop_id', '=', $shop],
        ['item_id', '=', $product_id]
    ])->get();
    foreach ($tdo as $key => $tad) {
        $product_quantity = $tad->quantity;
    }

    if (empty($product_quantity)) {
        session()->flash('message', 'Item Not In Your Shop');
        return redirect('/sales_area');
    } elseif ($product_quantity <= 0) {
        session()->flash('message', 'Item Out Of Stock');
        return redirect('/sales_area');
    } else {
        $todo = tblcart::getQuery('tblcarts')->where([
            ['product_id', '=', $id], ['shop_id', '=', $shop],
            ['member_id', '=', $uid]
        ])->get();
        foreach ($todo as $key => $tada) {
            $cart_id = $tada->id;
        }
        if (empty($cart_id)) {
            $tem = new tblcart;
            $tem->product_id = $product_id;
            $tem->quantity = 1;
            $tem->shop_id = $shop;
            $tem->member_id = $userid;
            $tem->save();
            return redirect('/sales_area');
        } else {
            $tem = tblcart::find($cid);
            $tem->quantity = $tem->quantity + 1;
            $tem->save();
            return redirect('/sales_area');
        }
    }
}

目前，当用户将产品添加到购物车时，订单数量增加了一倍，因此不是一个，而是两个。如果用户再次添加相同的产品，他将获得四个而不是两个。我该如何解决这个问题？

Answer 1

看起来可能是一个计时问题，具体取决于是否直接从搜索中调用add方法。使用keyup，即使在根据搜索中键入的前一个字符发送了原始ajax请求之后，您仍然可能会触发搜索功能至少一次，并且可能不止一次基于仍然键入他们想要的产品的用户酒吧。

建议是在keyup函数或搜索代码中添加时间延迟。在控制器中说出（伪代码）之类的东西

if($timeOfLastTransaction < 30 seconds && $productId == $previousProductId)
 // cancel this duplicate
else
  set $timeOfLastTransaction 
  set $previousProductId

我仍然不确定是什么叫做add()方法，这对于寻找特定加倍错误的bug来说是个好的开始。

但是，在add()方法中查看该代码，有一些事情可能会略微清理 - 并可能解决您的问题。建议直接在方法的参数中命名$id变量$product_id而不是在顶部重命名 - 你在多个地方使用两个不同的变量用于同一个东西，这有点令人困惑。

一个更大的问题是，在从数据库中提取集合后，您可能会在多个位置重新分配相同的变量：

$tdo = item_quantity::getQuery('item_quantities')->where( [['shop_id', '=', $shop],
['item_id', '=', $product_id]])->get();
foreach ($tdo as $key => $tad) {
    $product_quantity=$tad->quantity;
}

这一行和它下面的下一个DB都循环遍历一个集合并在这种情况下重新分配/覆盖$product_quantity，并在下一个中重新分配$cart_id。为什么不使用->first()并跳过循环？我认为这也可以减少任何不可预见的错误