最佳实践，1 种方法和 2 种方法具有相同的功能

Question

我有以下场景。 有些人使用方法 1，有些人使用方法 2。 两种方法具有相同的功能，根据新闻表中的新闻 ID 锁定或解锁新闻。 你有什么建议，哪一个更好，为什么？

注意：为了简单起见，我使用 void 返回类型而不是 bool，因此请忽略这一点

方法 1：

public void LockNews(long newId)
{
 ......
}

public void UnlockNews(long newId)
{
 ...
}

方法 2：

public void LockUnlockNews(long newId,bool Islock)
{
 ......
}

Answer 1

方法 1，对我来说，因为恕我直言，方法应该代表单个操作。为了编写可测试的代码或任何形式的自动化测试，它将保持清晰的分离并且更有意义。

方法 2 倾向于“执行所有操作”，恕我直言，应该避免这种操作。

Answer 2

我更喜欢方法 1。它可以清楚地说明发生了什么。如果您使用方法 2 并调用

LockUnlockNews(42, true);

并不能立即明确这是锁定还是解锁。
只是火上浇油：如果你将 bool 更改为 enum 或 const，那么我的论点就无效了。

LockUnlockNews(42, LOCK);

就像

LockNews(42);

Answer 3

第一种方法。

您的方法是一个命令，并且应该尽可能明确。我什至会问你为什么除了新闻本身之外还有人知道如何锁定/解锁？对我来说新闻应该对此负责：

var news = GetNewsSomehow(newsId);
news.Lock();
news.Unlock();

更有意义，不是吗？ :) 你可以清楚地看到你的对象是什么以及它们有什么行为。这就是 OOP 中所谓的封装。

Answer 4

我通常将这两种方法结合起来，如下所示：

公共方法提供清晰的界面并且更易于测试：

public void LockNews(long newId)
{
    LockUnlockNews(newId, true);
}

public void UnlockNews(long newId)
{
    LockUnlockNews(newId, false);
}

私有方法执行实际业务并提高模块化从一个位置管理事物：

private void LockUnlockNews(long newId,bool Islock)
{
    ......
}