一堆条件if-else所需的设计思想

Question

我有一个简单的逻辑要实现。但不确定是否有更好的方法来设计它，除了简单的 if-else 或 switch 语句。

有 4 个权限（将它们视为布尔变量），可以是真或假。根据各种条件（这些权限的排列），我需要返回需要在 UI 上显示的下拉字段的字符串值列表。

所以现在就像这样-

if(!permission1 && !permission2){return list_of_strings_1;}
else if (permission1 && permission2 && !permission3){return list_of_strings_2;}

等等。其中一些只是if语句。所以多个条件可能是真的，我们必须收集所有字符串列表并显示它们。

那些如果其他的持续了相当长的一段时间（大约100行）。每个都将返回不同的字符串列表。其中大部分在未来不太可能改变。所以可能设计得太深了，可能有点过头了。

但只是想知道专家将如何重构这段代码（或者他们是否会重构它）。也许坚持开关/如果-否则可以？

Answer 1

我不明白四个标志是如何给你100行代码的。这可以通过16个条目的映射来完成（或者更少，如果某些组合无效并且可以映射到默认值）。如果字符串表示确实是字符串列表，每个可能的权限对应一个字符串，则解决方案更加紧凑。

键是表示权限组合的对象，值是该组合的字符串表示。您可以为密钥创建一个自定义类型，但在本例中，我只使用四位整数，其中每一位表示是否授予权限：

private static final int P1 = 1 << 0, P2 = 1 << 1, P3 = 1 << 2, P4 = 1 << 3;

private static final Map<Integer, String> permissionsToString = Map.ofEntries(
  Map.entry( 0, "No permissions granted."),
  Map.entry( 1, "Permissions 2-3 revoked."),
  Map.entry( 2, "Permission 2 granted."),
  ...
  Map.entry(14, "Permission 1 revoked"),
  Map.entry(15, "Superuser"));

public static String toString(boolean p1, boolean p2, boolean p3, boolean p4) {
  int key =   (p1 ? 0 : 1) << 0
            | (p2 ? 0 : 1) << 1
            | (p3 ? 0 : 1) << 2
            | (p4 ? 0 : 1) << 3;
  return permissionsToString.get(key);
}

如果您不理解位，您可以使用ENumSet或定义您自己的值对象来表示更高级别的键。想法是一样的：将所有可能的组合（2⁴=16）映射到它们对应的标签。