更新表

vagrant

请问下，对数据库的更新、删除等是自己写action好，还是用平台的相关方法好？？
我目前是直接用平台的相关方法：
if(data.getIndex(id) >= 0){
                        var dataReturn = justep.xbl('returnData');
                        dataReturn.setFilter("filter1", "");
                        dataReturn.setFilter("filter1", "sd_ticketReturn.fQRcode = '"+tempFQRcode+"' ");
                        dataReturn.refreshData();
                        var Returnid = event.data.id;
                        //alert(dataReturn.getIndex(Returnid));
                        if (dataReturn.getIndex(Returnid)<0){
                                var fCode = data.getValue('fCode');
                                var fName = data.getValue('fName');
                                var fPrice = data.getValue('fPrice');
                                var fRebate = data.getValue('fRebate');
                                var fDeadline = data.getValue('fDeadline');
                                var fConductor = data.getValue('fConductor');
                                var fDate = data.getValue('fDate');
                                var fUseDate = data.getValue('fUseDate');
                                var fReturn = data.getValue('fReturn');
                                var fQRcode = data.getValue('fQRcode');
                                var fOrderID = data.getValue('fOrderID');
   
                                //var dataReturn = justep.xbl('returnData');
                                dataReturn.newData();
                                dataReturn.setValue('fCode',fCode);
                                dataReturn.setValue('fName',fName);
                                dataReturn.setValue('fPrice',fPrice);
                                dataReturn.setValue('fRebate',fRebate);
                                dataReturn.setValue('fDeadline',fDeadline);
                                dataReturn.setValue('fConductor',fConductor);
                                dataReturn.setValue('fUseDate',fUseDate);
                                dataReturn.setValue('fReturn',fReturn);
                                dataReturn.setValue('fQRcode',fQRcode);
                                dataReturn.setValue('fOrderID',fOrderID);
                                dataReturn.setValue('fDate',fDate);
                                dataReturn.saveData();
                                dataReturn.setFilter("filter1", "");
                                dataReturn.refreshData();
                        }
                        else{
                                alert('此条码已经扫描过！');
                                dataReturn.setFilter("filter1", "");
                                dataReturn.refreshData();
                        }
                }
不知道这会不会存在什么缺陷？？
请指导下

zhaixin

推荐更多的写java代码，更少的写js代码
针对上面的代码可以改进的有两处
1、查找Returnid是否存在的方法，可以再直接一些，在setFilter里面直接把Returnid用作过滤条件
2、如果可以的话，在取值时，使用data组件的getRowData方法，将这个返回值作为newData方法的参数

vagrant

1、查找Returnid是否存在的方法，可以再直接一些，在setFilter里面直接把Returnid用作过滤条件
请问下这个怎么实现啊？没太看明白

jishuang

  dataReturn.setFilter("filter1", "sd_ticketReturn= '"+Returnid+"' ");
这样过滤出来的就是这个id的记录

vagrant

好，结贴