Skip to content

B
business-java
Commit 5d9baf5c authored by manohar's avatar manohar
Browse files
Options

Prepared statement for detail taken outside loop as only one row was inserted in detail 


git-svn-id: http://15.206.35.175/svn/proteus/business-java/trunk@94092 ce508802-f39f-4f6c-b175-0d175dae99d5
parent 5eeaf28c
Show whitespace changes
Inline Side-by-side
Showing with 60 additions and 47 deletions
webitm-mfg/ibase/webitm/ejb/mfg/WOIssueReplacementPrc.java
View file @ 5d9baf5c
...@@ -144,7 +144,7 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -144,7 +144,7 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
//java.sql.Timestamp chgDate = new java.sql.Timestamp(System.currentTimeMillis()); //java.sql.Timestamp chgDate = new java.sql.Timestamp(System.currentTimeMillis());
String sql = "",sql2 = ""; String sql = "",sql2 = "";
PreparedStatement pstmt = null,pstmtdet =null,pstmt1 = null ,pstmt2 = null ; PreparedStatement pstmt = null,pstmtdet =null,pstmt1 = null ,pstmt2 = null, pstmtDetail = null ;
ResultSet rs = null,rsdet=null,rs1 = null ,rs2 = null ; ResultSet rs = null,rsdet=null,rs1 = null ,rs2 = null ;
String chgUser = "",chgTerm = ""; String chgUser = "",chgTerm = "";
NodeList parentNodeList = null,parentNodeList2 = null; NodeList parentNodeList = null,parentNodeList2 = null;
...@@ -411,6 +411,13 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -411,6 +411,13 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
pstmt2.close(); pstmt2.close();
rs2 = null; rs2 = null;
pstmt2 = null; pstmt2 = null;
// 06/01/14 manoharan
sql ="insert into workorder_issdet (tran_id,line_no,exp_lev,item_code,loc_code,lot_no," +
" lot_sl,quantity,unit,rate,gross_rate,calc_potency, potency_adj," +
" match_potency,min_potency_perc,adj_potency,work_order)" +
" values ( ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
pstmtDetail = conn.prepareStatement(sql);
// end 06/01/14 manoharan
if ( !( diffReplaceQty.isEmpty() )) if ( !( diffReplaceQty.isEmpty() ))
{ {
...@@ -962,30 +969,33 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -962,30 +969,33 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
// insert into workorder_issdet // insert into workorder_issdet
lnNo++; lnNo++;
batchLoadCnt++; batchLoadCnt++;
// 06/01/14 manoharan commented and taken outside loop
/*
sql ="insert into workorder_issdet (tran_id,line_no,exp_lev,item_code,loc_code,lot_no," + sql ="insert into workorder_issdet (tran_id,line_no,exp_lev,item_code,loc_code,lot_no," +
" lot_sl,quantity,unit,rate,gross_rate,calc_potency, potency_adj," + " lot_sl,quantity,unit,rate,gross_rate,calc_potency, potency_adj," +
" match_potency,min_potency_perc,adj_potency,work_order)" + " match_potency,min_potency_perc,adj_potency,work_order)" +
" values ( ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)"; " values ( ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
pstmt = conn.prepareStatement(sql); pstmt = conn.prepareStatement(sql);
pstmt.setString(1,tranId); */
pstmt.setInt(2,lnNo); pstmtDetail.setString(1,tranId);
pstmt.setString(3,expLev); pstmtDetail.setInt(2,lnNo);
pstmt.setString(4,itemCode); pstmtDetail.setString(3,expLev);
pstmt.setString(5,locCode); pstmtDetail.setString(4,itemCode);
pstmt.setString(6," "); pstmtDetail.setString(5,locCode);
pstmt.setString(7," "); pstmtDetail.setString(6," ");
pstmt.setDouble(8,reqQty); pstmtDetail.setString(7," ");
pstmt.setString(9,unit); pstmtDetail.setDouble(8,reqQty);
pstmt.setDouble(10,rate); pstmtDetail.setString(9,unit);
pstmt.setDouble(11,grossRate); pstmtDetail.setDouble(10,rate);
pstmt.setString(12,calcPotency); pstmtDetail.setDouble(11,grossRate);
pstmt.setDouble(13,extraQty); pstmtDetail.setString(12,calcPotency);
pstmt.setString(14,matchPotency); pstmtDetail.setDouble(13,extraQty);
pstmt.setDouble(15,minPotencyPerc); pstmtDetail.setString(14,matchPotency);
pstmt.setString(16,adjPotency); pstmtDetail.setDouble(15,minPotencyPerc);
pstmt.setString(17,workOrder); pstmtDetail.setString(16,adjPotency);
pstmt.addBatch(); pstmtDetail.setString(17,workOrder);
pstmt.clearParameters(); pstmtDetail.addBatch();
pstmtDetail.clearParameters();
// for stock allocation // for stock allocation
System.out.println("@@@@@ inside stkopt lineNo :["+lineNo+"] itemCode :["+itemCode+"] loc_code :["+locCode+"]:: reqQty :["+reqQty+"]"); System.out.println("@@@@@ inside stkopt lineNo :["+lineNo+"] itemCode :["+itemCode+"] loc_code :["+locCode+"]:: reqQty :["+reqQty+"]");
...@@ -1019,30 +1029,33 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -1019,30 +1029,33 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
// insert into workorder_issdet // insert into workorder_issdet
lnNo++; lnNo++;
batchLoadCnt++; batchLoadCnt++;
// 06/01/14 manoharan commented and taken outside loop
/*
sql ="insert into workorder_issdet (tran_id,line_no,exp_lev,item_code,loc_code,lot_no," + sql ="insert into workorder_issdet (tran_id,line_no,exp_lev,item_code,loc_code,lot_no," +
" lot_sl,quantity,unit,rate,gross_rate,calc_potency, potency_adj," + " lot_sl,quantity,unit,rate,gross_rate,calc_potency, potency_adj," +
" match_potency,min_potency_perc,adj_potency,work_order)" + " match_potency,min_potency_perc,adj_potency,work_order)" +
" values ( ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)"; " values ( ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
pstmt = conn.prepareStatement(sql); pstmt = conn.prepareStatement(sql);
pstmt.setString(1,tranId); */
pstmt.setInt(2,lnNo); pstmtDetail.setString(1,tranId);
pstmt.setString(3,expLev); pstmtDetail.setInt(2,lnNo);
pstmt.setString(4,itemCode); pstmtDetail.setString(3,expLev);
pstmt.setString(5,stock_loc_code); pstmtDetail.setString(4,itemCode);
pstmt.setString(6,stock_lot_no); pstmtDetail.setString(5,stock_loc_code);
pstmt.setString(7,stock_lot_sl); pstmtDetail.setString(6,stock_lot_no);
pstmt.setDouble(8,reqQty); pstmtDetail.setString(7,stock_lot_sl);
pstmt.setString(9,unit); pstmtDetail.setDouble(8,reqQty);
pstmt.setDouble(10,rate); pstmtDetail.setString(9,unit);
pstmt.setDouble(11,grossRate); pstmtDetail.setDouble(10,rate);
pstmt.setString(12,calcPotency); pstmtDetail.setDouble(11,grossRate);
pstmt.setDouble(13,extraQty); pstmtDetail.setString(12,calcPotency);
pstmt.setString(14,matchPotency); pstmtDetail.setDouble(13,extraQty);
pstmt.setDouble(15,minPotencyPerc); pstmtDetail.setString(14,matchPotency);
pstmt.setString(16,adjPotency); pstmtDetail.setDouble(15,minPotencyPerc);
pstmt.setString(17,workOrder); pstmtDetail.setString(16,adjPotency);
pstmt.addBatch(); pstmtDetail.setString(17,workOrder);
pstmt.clearParameters(); pstmtDetail.addBatch();
pstmtDetail.clearParameters();
// for stock allocation // for stock allocation
...@@ -1082,14 +1095,9 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -1082,14 +1095,9 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
if( batchLoadCnt > 0) if( batchLoadCnt > 0)
{ {
pstmt.executeBatch(); pstmtDetail.executeBatch();
pstmt.clearBatch(); pstmtDetail.clearBatch();
System.out.println("Record inserted into detail:: count["+batchLoadCnt+"]"); System.out.println("Record inserted into detail:: count["+batchLoadCnt+"]");
if( pstmt != null )
{
pstmt.close();
pstmt = null;
}
} }
else if( batchLoadCnt == 0) else if( batchLoadCnt == 0)
...@@ -1097,6 +1105,11 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt ...@@ -1097,6 +1105,11 @@ public class WOIssueReplacementPrc extends ProcessEJB implements GenerateReceipt
System.out.println("@@@@@@ no detail record inserted ["+batchLoadCnt+"]"); System.out.println("@@@@@@ no detail record inserted ["+batchLoadCnt+"]");
retStr = itmDBAccessLocal.getErrorString("","VTPROUNSU9",""); retStr = itmDBAccessLocal.getErrorString("","VTPROUNSU9","");
} }
if( pstmtDetail != null )
{
pstmtDetail.close();
pstmtDetail = null;
}
} }
else else
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment