关于方法重载的建议

Question

请看以下方法。

    public static ProductsCollection GetDummyData(int? customerId, int? supplierId)
    {
        try
        {
            if (customerId != null && customerId > 0)
            {
                Filter.Add(Customres.CustomerId == customerId);
            }

            if (supplierId != null && supplierId > 0)
            {
                Filter.Add(Suppliers.SupplierId == supplierId);
            }

            ProductsCollection products = new ProductsCollection();

            products.FetchData(Filter);

            return products;
        }
        catch
        {
            throw;
        }
    }

    public static ProductsCollection GetDummyData(int? customerId)
    {
        return ProductsCollection GetDummyData(customerId, (int?)null);
    }

    public static ProductsCollection GetDummyData()
    {
        return ProductsCollection GetDummyData((int?)null);
    }

1- 请建议我如何为 CustomerId 和 SupplierId 进行重载，因为使用 GetDummyData(int?) 只能创建一个重载。我是否应该添加另一个参数来提及第一个参数是 CustomerId 或 SupplierId，例如 GetDummyData(int?, string)。或者我应该使用枚举作为第二个参数并提到第一个参数是 CustoemId 或 SupplierId。

2- 这个条件是正确的还是仅仅检查 > 0 就足够了 -> if (customerId != null && customerId > 0)

3- 像这样使用 Try/catch 是否正确？

4- 传递 (int?)null 是正确的或任何其他更好的方法。

编辑：

我发现了一些其他类似的帖子，因为我对泛型一无所知，这就是我面临这个问题的原因。我对吗？以下是帖子。

Overloaded method calling overloaded method

Answer 1

1. 为什么不创建单独的 GetCustomerData(int) 和 GetSupplierData(int) 方法？ （如果需要，还可以加上 GetData() 和 GetData(int,int)。）

2. 如果您将方法参数更改为 int 而不是 int?，那么您只需检查（自行决定）它们是否大于 0。

3. 在这种情况下不需要try...catch。如果您所做的只是重新抛出异常，那么一开始就不要费心去捕捉它。

4. 参见上面的 1 和 2。

编辑：也许是这样的......

public static ProductsCollection GetData()
{
    return GetDataImpl(-1, -1);
}

public static ProductsCollection GetData(int customerId, int supplierId)
{
    if (customerId <= 0)
        throw new ArgumentOutOfRangeException("customerId");

    if (supplierId <= 0)
        throw new ArgumentOutOfRangeException("supplierId");

    return GetDataImpl(customerId, supplierId);
}

public static ProductsCollection GetCustomerData(int customerId)
{
    if (customerId <= 0)
        throw new ArgumentOutOfRangeException("customerId");

    return GetDataImpl(customerId, -1);
}

public static ProductsCollection GetSupplierData(int supplierId)
{
    if (supplierId <= 0)
        throw new ArgumentOutOfRangeException("supplierId");

    return GetDataImpl(-1, supplierId);
}

private static ProductsCollection GetDataImpl(int customerId, int supplierId)
{
    if (customerId > 0)
        Filter.Add(Customers.CustomerId == customerId);

    if (supplierId > 0)
        Filter.Add(Suppliers.SupplierId == supplierId);

    ProductsCollection products = new ProductsCollection();
    products.FetchData(Filter);

    return products;
}

Answer 2

你没有对 try catch 做任何事情，所以为什么要包含它。无论如何，您的异常都会冒泡到调用方法，因此只需删除无关的 try catch 代码。

我会放弃第二种和第三种方法，因为它们实际上并没有做任何事情。除非你有一些理由保留它们，就像它们是遗留代码一样，我会删除它们。它只会使代码更加复杂，以供将来支持。