C#代码可读性问题

Question

        int scalar = creature is SpecialCreature ? (creature.IsAwesome ? 700 : 500) : (creature is NotSoNormalCreature ? 
            (creature.IsAwesome ? (creature is IGreatCreature ? 450 : 280) : 240) : 
            (creature.IsAwesome ? (creature is IGreatCreature ? 300 : 200) : 160));

我应该如何编写该代码以使其更具可读性？

我只想构建 ifs，但后来我想到制作某种“ConditionFactory”怎么样？这有意义吗？或者对于这样一个简单的任务来说，它是否太复杂了？

int scalar;

if (creature is SpecialCreature)
{
    scalar = creature.IsAwesome ? 700 : 500;
}
else if (creature is NotSoNormalCreature)
{
    if (creature.IsAwesome)
    {
        scalar = creature is IGreatCreature ? 450 : 280;
    }
    else
    {
        scalar = 240;
    }
}
else
{
    if (creature.IsAwesome)
    {
        scalar = creature is IGreatCreature ? 300 : 200;
    }
    else
    {
        scalar = 160;
    }
}

Answer 1

完全不确定你要做什么，但由于你使用的是基类型继承链，你可能会选择做类似的事情，

interface ICreature
{
    bool IsAwesome { get; set; }
    int GetScalar();
}

abstract class Creature : ICreature
{
    public bool IsAwesome { get; set; }
    public virtual int GetScalar()
    {
        return 160;
    }
}

class SpecialCreature : Creature
{
    public override int GetScalar()
    {
        return this.IsAwesome ? 700 : 500;
    }
}

class NotSoNormalCreature : Creature
{
    public override int GetScalar()
    {
        return this.IsAwesome ? 450 : 280;
    }
}

// more ICreatures...

这将允许你让生物实现自己的逻辑来确定标量，并且你的消费代码可以摆脱照顾的麻烦。

ICreature creature = GetCreatureFromSomewhere();
int scalar = creature.GetScalar();

Answer 2

这并不完全是您所需要的，但当条件可以解析为 Or 或 And 列表时，我使用扩展方法来实现这种链方法。

像

if (true.IfOr(condition1 == a, condition2 == b) 
{ 
  something(); 
}

扩展方法这样的事情就非常简单了：

public static bool IfOr(this bool result, params bool[] tests)
{
  foreach (bool test in tests)
    if (!test)
      return !result;
  return result;
}

另一种可行的方法（尽管可能不是最佳方法）是利用 .net 中的 Predicate 委托并定义执行各个逻辑单元的方法列表。然后，您可以将嵌套的三级运算符替换为 lambda。
不过，抱歉，我手头没有这方面的代码示例。

最后，有时没有什么比一个好的旧 switch 语句更好的了。我相信 .Net 倾向于将它们编译为跳转表，因此只要您首先按最可整除的来安排测试，那么您实际上可以获得相当高性能和可读的代码。而且它是可维护的，而不是用技巧隐藏逻辑或实现。

Answer 3

我认为真正的问题是您正在对“配置数据”进行硬编码。如果你想说的是，将这些“设置”撕掉并将它们放入 XML 配置文件中，那么整个混乱状态不会消失吗？

这也可能看起来有点矫枉过正，直到你调整你的各种配置，使游戏更具可玩性......一个单独的配置文件可以让你轻松地玩（和恢复）。

---

编辑：

顺便说一句，我将按如下所示格式化嵌套三元语句......以使其更具可读性。

int scalar =
  creature is SpecialCreature
  ? creature.IsAwesome ? 700 : 500
  : creature is NotSoNormalCreature
    ? creature.IsAwesome
      ? creature is IGreatCreature ? 450 : 280
      : 240
    : creature.IsAwesome
      ? creature is IGreatCreature ? 300 : 200
      : 160
;

干杯。基思.

Answer 4

这就是我重新编写代码并使其可读的方法，

// Original code spread apart
int scalar = creature is SpecialCreature ? (
    creature.IsAwesome ? 700 : 500
) : (
    creature is NotSoNormalCreature ? (
        creature.IsAwesome ? (
            creature is IGreatCreature ? 450 : 280
        ) : 240
    ) : (
        creature.IsAwesome ? (
            creature is IGreatCreature ? 300 : 200
        ) : 160
    )
);

// Readable code with hybrid if() and ? :
if (creature is SpecialCreature)
{
    scalar = creature.IsAwesome ? 700 : 500;
}
else if (creature is NotSoNormalCreature)
{
    if (creature.IsAwesome)
    {
        scalar = creature is IGreatCreature ? 450 : 280;
    }
    else
    {
        scalar = 240;
    }
}
else
{
    if (creature.IsAwesome)
    {
        scalar = creature is IGreatCreature ? 300 : 200;
    }
    else
    {
        scalar = 160;
    }
}

我确实想建议如果可能的话，在每个类中移动此计算，并覆盖不同的分支。

Answer 5

好老的怎么样：

if (creature is SpecialCreature)
{
    scalar=getSpecialCreatureScalar(creature);
}
else if (creature is NotSoNormalCreature)
{
    scalar=getNotSoNormalCreatureScalar(creature);
}
else
{
    scalar=getScalar(creature);
}

..然后

int GetSpecialCreatureScalar(SpecialCreature creature)
{
    return creature.IsAwesome ? 700 : 500;
}   

int GetNotSoNormalCreatureScalar(NotSoNormalCreature creature)
{
    if (creature.IsAwesome)
    {
        return creature is IGreatCreature ? 450 : 280;
    }
    else
    {
        return 240;
    }
}   

int GetScalar(Creature creature)
{
    if (creature.IsAwesome)
    {
        return creature is IGreatCreature ? 300 : 200;
    }
    else
    {
        return 160;
    }
}  

..赋予 if 一个含义。创造了一个不同的IMO。