创建方法参数时，我应该使用内置数据类型还是接口？ C＃

Question

下面创建的两种方法之间哪个被认为是更好的做法？ MakeCallV1 更简单，代码更少。 MakeCallV2 对扩展开放并遵守 SOLID 原则。

如果认为 V2 更好，那么不创建使用内置数据类型（字符串、int 等）的方法是否是一个好的经验法则？

public class Caller
    {
        public void MakeCallV1(string phoneNumber, string name)
        {
            //Using built-in data type string
        }

        public void MakeCallV2(IContact contact) {
            //Using IContact
            var name = contact.Name;
            var number = contact.PhoneNumber;
            //Make call code
        }
    }


public interface IContact
{
    string PhoneNumber { get; set; }
    string Name { get; set; }
}

Answer 1

像往常一样 - 视情况而定。

如果它只是一个简单的例子，并且在逻辑上该方法需要两个参数，那么没有理由将它们放在一个对象中，更不用说创建一个接口了。

如果已经有一个对象包含这两个参数，你可以考虑把这个函数移到对象本身。

但是，在某些情况下，将参数包装在一起确实有意义，那就是函数签名包含太多参数的情况。 然后，将它们提取到一个类中可能是明智之举。

Answer 2

如果 V2 被认为更好 [不传递原语]，那么不创建使用内置数据类型（字符串、int 等）的方法是否是一个好的经验法则？

如果这是一个好的经验法则，那么 .NET 框架和/或许多其他 API 将不会在其中包含此类方法。这显然不是真的，所以你不能简单地断定将原语传递给方法不是一个好主意。

如何在创建一个接受原语和自定义类型的方法之间做出决定？

问自己这个问题：客户将如何使用我的课程？客户我的意思是任何将使用您的课程的人。要回答这个问题，您可以先检查一下它现在是如何使用的。

1. 如果参数相互关联，那么自定义类型会更好。
2. 如果参数彼此不相关，则基元可能是一个更好的主意。

查看您的代码，我会选择选项 1。为什么？因为您似乎正在管理联系人；因此，您的客户端类将具有实现接口IContact 的类型，并且参数彼此相关。

什么是参数相互关联？

考虑这个方法签名：

public void MakeCallV1(string phoneNumber, string name)

作为客户，我基本上可以做到这一点：

// company is some object of Company class
var phoneNumber = company.PhoneNumber; 
var name = contact.Name;

MakeCallV1(phoneNumber, name);

我传递的参数不属于同一个对象，所以没有关系，它们也不相关。

现在考虑这个方法签名：

public void MakeCallV2(IContact contact) 

在此方法中，客户端将传递实现 IContact 的内容，因此属性将属于同一个对象，因此是相关的。

最后一点要考虑

如果您的方法接受原语，并且您稍后决定您的方法需要额外的输入，您将需要更改您的方法签名（合同、公共接口），这将产生副作用：您的客户将需要更改。