diff --git a/src/main/java/com/xujie/common/utils/QueryPage.java b/src/main/java/com/xujie/common/utils/QueryPage.java
index bf80d74..62a4baf 100644
--- a/src/main/java/com/xujie/common/utils/QueryPage.java
+++ b/src/main/java/com/xujie/common/utils/QueryPage.java
@@ -13,7 +13,7 @@ public class QueryPage {
 
 
     public int getLimit() {
-        return limit;
+        return limit <= 0 ? 10 : limit;
     }
 
     public void setLimit(int limit) {
@@ -21,7 +21,7 @@ public class QueryPage {
     }
 
     public int getPage() {
-        return page;
+        return page <= 0 ? 1 : page;
     }
 
     public void setPage(int page) {
diff --git a/src/main/java/com/xujie/modules/baseInformation/mapper/SrmSupplierAddressMapper.java b/src/main/java/com/xujie/modules/baseInformation/mapper/SrmSupplierAddressMapper.java
index f677456..155dad1 100644
--- a/src/main/java/com/xujie/modules/baseInformation/mapper/SrmSupplierAddressMapper.java
+++ b/src/main/java/com/xujie/modules/baseInformation/mapper/SrmSupplierAddressMapper.java
@@ -10,7 +10,7 @@ import org.apache.ibatis.annotations.Param;
 @Mapper
 public interface SrmSupplierAddressMapper extends BaseMapper<SrmSupplierAddress> {
     
-    IPage<SrmSupplierAddress> selectSupplierAddressWithPaging(Page<SrmSupplierAddress> page, @Param("query") SrmSupplierAddress query);
+    IPage<SrmSupplierAddress> selectSupplierAddressWithPaging(IPage<SrmSupplierAddress> page, @Param("query") SrmSupplierAddress query);
     
     int insertSupplierAddress(SrmSupplierAddress supplierAddress);
     
diff --git a/src/main/java/com/xujie/modules/baseInformation/service/impl/SrmSupplierAddressServiceImpl.java b/src/main/java/com/xujie/modules/baseInformation/service/impl/SrmSupplierAddressServiceImpl.java
index 6cdc28b..621fa41 100644
--- a/src/main/java/com/xujie/modules/baseInformation/service/impl/SrmSupplierAddressServiceImpl.java
+++ b/src/main/java/com/xujie/modules/baseInformation/service/impl/SrmSupplierAddressServiceImpl.java
@@ -7,7 +7,6 @@ import com.xujie.common.utils.R;
 import com.xujie.modules.baseInformation.entity.SrmSupplierAddress;
 import com.xujie.modules.baseInformation.mapper.SrmSupplierAddressMapper;
 import com.xujie.modules.baseInformation.service.SrmSupplierAddressService;
-import jakarta.mail.Address;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
@@ -16,7 +15,7 @@ public class SrmSupplierAddressServiceImpl implements SrmSupplierAddressService
     
     @Autowired
     private SrmSupplierAddressMapper srmSupplierAddressMapper;
-    
+
     @Override
     public PageUtils getSupplierAddressList(SrmSupplierAddress query) {
         Page<SrmSupplierAddress> page = new Page<>(query.getPage(), query.getLimit());
@@ -26,6 +25,13 @@ public class SrmSupplierAddressServiceImpl implements SrmSupplierAddressService
 
     @Override
     public R addSupplierAddress(SrmSupplierAddress supplierAddress) {
+        if (supplierAddress.getSupplierId() == null || supplierAddress.getSupplierId().trim().isEmpty()) {
+            return R.error("供应商ID不能为空");
+        }
+        if (supplierAddress.getAddressId() == null || supplierAddress.getAddressId().trim().isEmpty()) {
+            return R.error("地址ID不能为空");
+        }
+        
         // 检查供应商地址是否已存在
         SrmSupplierAddress existing = srmSupplierAddressMapper.selectSupplierAddressByAddress(supplierAddress.getSupplierId(), supplierAddress.getAddressId(), supplierAddress.getSite());
         if (existing != null) {
@@ -42,6 +48,13 @@ public class SrmSupplierAddressServiceImpl implements SrmSupplierAddressService
 
     @Override
     public R updateSupplierAddress(SrmSupplierAddress supplierAddress) {
+        if (supplierAddress.getSupplierId() == null || supplierAddress.getSupplierId().trim().isEmpty()) {
+            return R.error("供应商ID不能为空");
+        }
+        if (supplierAddress.getAddressId() == null || supplierAddress.getAddressId().trim().isEmpty()) {
+            return R.error("地址ID不能为空");
+        }
+        
         // 检查供应商地址是否存在
         SrmSupplierAddress existing = srmSupplierAddressMapper.selectSupplierAddressByAddress(supplierAddress.getSupplierId(), supplierAddress.getAddressId(), supplierAddress.getSite());
         if (existing == null) {
diff --git a/src/main/resources/mapper/baseInformation/SrmSupplierAddressMapper.xml b/src/main/resources/mapper/baseInformation/SrmSupplierAddressMapper.xml
index ef45e24..58e0506 100644
--- a/src/main/resources/mapper/baseInformation/SrmSupplierAddressMapper.xml
+++ b/src/main/resources/mapper/baseInformation/SrmSupplierAddressMapper.xml
@@ -39,7 +39,7 @@
                 AND site = #{query.site}
             </if>
             <if test="query.supplierId != null and query.supplierId != ''">
-                AND supplier_id = #{query.supplierId}
+                AND supplierId = #{query.supplierId}
             </if>
             <if test="query.addressType != null and query.addressType != ''">
                 AND addressType = #{query.addressType}