diff --git a/code/zcdev/zcgj-zcdev-zcdev-pr/src/main/java/zcgj/zcdev/zcdev/pr/plugin/form/InContractFinaceConfirmeInvoicePlugin.java b/code/zcdev/zcgj-zcdev-zcdev-pr/src/main/java/zcgj/zcdev/zcdev/pr/plugin/form/InContractFinaceConfirmeInvoicePlugin.java
index e20d1cb..a708de6 100644
--- a/code/zcdev/zcgj-zcdev-zcdev-pr/src/main/java/zcgj/zcdev/zcdev/pr/plugin/form/InContractFinaceConfirmeInvoicePlugin.java
+++ b/code/zcdev/zcgj-zcdev-zcdev-pr/src/main/java/zcgj/zcdev/zcdev/pr/plugin/form/InContractFinaceConfirmeInvoicePlugin.java
@@ -93,12 +93,14 @@ public class InContractFinaceConfirmeInvoicePlugin extends AbstractBillPlugIn im
             filter1.and(new QFilter("contract", "=", contractId));
             filter1.and(new QFilter("unapplyamount", ">", BigDecimal.ZERO));
             DynamicObjectCollection contInvEntryCol = this.getModel().getEntryEntity("zcgj_entryentity");
-            if (contInvEntryCol.size() > 0) {
-                List<Long> selectedInvIds = new ArrayList(8);
+            if (!contInvEntryCol.isEmpty()) {
+                List<Long> selectedInvIds = new ArrayList<Long>(8);
 
                 for(DynamicObject contInvEntryObj : contInvEntryCol) {
                     DynamicObject invoice = contInvEntryObj.getDynamicObject("zcgj_invoice");
-                    selectedInvIds.add(invoice.getLong("id"));
+                    if(invoice!=null){
+                        selectedInvIds.add(invoice.getLong("id"));
+                    }
                 }
 
                 filter1.and(new QFilter("id", "not in", selectedInvIds));