干(不要重复自己)和如果分配

我想我忘记了一些明显的东西但是我似乎无法找到一种方法来分配一个值,如果它确认一个条件尽可能保持DRY …一些代码来解释我的意思…

a = (b > 1) ? b : c; 

甚至a = (a > 1) ? a : b; a = (a > 1) ? a : b;

所以当然这里没什么大不了的,但如果要用方法调用替换,(可能是那里的收益率返回)或其他什么,那么我就要调用它两次……

我只看到将它存放在一个变量中,然后就像上面的代码一样……

有什么好主意吗?

编辑以便更好地理解:假设我在xml文件中搜索一个值,使用空检查(?。?[])等等

 string store_no = myXmlNode.SelectSingleNode("aChildNode")?.SelectSingleNode("myNode")?.Attributes?["store_no]?.Value; 

所以在这里我将它存储在一个变量中,以便我可以稍后测试它的值。 如果我想检查一个特定的store_no!我将不得不做类似的事情

 store_no = (store_no=="STORE23")? store_no : "unknown"; 

…是的,不确定这个例子是否足够明确,但这个想法就在这里; 我可能不想将数据存储在变量(例如巨大的数据块)中是否有办法获得相同的结果?

我想我忘记了一些明显的东西,但如果它确认一个条件尽可能保持DRY,我似乎找不到分配值的方法

让我们首先解除您对常见误解的看法。

这完全是对DRY含义的误解。 如果您有一个Customer对象,并且您有一个Address对象,而CustomerBillingCityBillingPostalCodeHomeCity等字段,那么您的代码就不是DRY,因为在两个地方冗余地表示了相同的信息。 您应该重新设计代码,以便Customer拥有一组Address对象。

现在,避免在整个节目中剪切和粘贴重复代码确实是一个好主意,但DRY是关于中到大规模的代码设计。 DRY绝对不意味着你的代码不应该在同一个表达式中两次使用相同的变量!

现在我们已经解决了这个问题,让我们来看看你对语言的批评。

我们经常处于一种“表达背景”的情况 – 即一种长期的,可能流畅的表达方式,我们希望避免做多余的工作。 例如,我们可能有:

 x = M() > 0 ? M() : 0; 

也许两次调用M()是昂贵的,或者它可能不是幂等的。 随你。 无所谓。 重点是,我们不想两次打电话。

令人恼火的是,我们必须退出表达式上下文并进入语句上下文:

 var m = M(); x = m > 0 ? m : 0; 

这当然是合法的,但有点令人烦恼。 此外,在某些情况下,它可能很棘手:

 N(P() ?? (M() > 0 ? M() : 0)); 

现在我们做什么? 假设我们只想在P()为空时调用M()那么没有明显的方法可以保留语义而不用长手写出来。

 var t = default(T); var p = P(); if (p == null) { var m = M(); t = m > 0 ? m : 0; } else { t = p.Value; } N(t); 

YUCK。 天哪,这太可怕了。

其他语言通过引入let表达式解决了这个问题。 我们真正想要的是能够在表达式的中间引入一个新的局部。 一个常见的语法是let ID = EXPRESSION in EXPRESSIONID变成一个只读变量,它具有特定含义,但仅在in中的范围内:

 N(P() ?? (let m = M() in m > 0 ? m : 0)); 

请注意,C# 确实支持let表达式,但支持查询理解 。 如果它在语言中更普遍地支持它会很棒。

多年来,有许多建议将let表达式或其更一般的forms序列表达式添加到C#中。 有关示例,请参阅github roslyn问题跟踪器。 也许这会进入C#8; 如果你想要它,去参加论坛。

那么你能做些什么呢?

事实certificate在C#中 let-expression。 let x = y in z只是一种很好的写法(((Func)(x=>z))(y)) 。 所以你可以写:

 N(P() ?? (((Func)(m => m > 0 ? m : 0))(M()))); 

但这看起来几乎是可怕的。 这是一个难以理解的混乱。

问题是lambda。 这会更好:

 Func f = m => m > 0 ? m : 0; ... N(P() ?? f(M())); 

但这有点不透明。 我们如何才能进一步改进?

我们可以使它成为一个局部函数,但更好的是,我们可以使它成为一个扩展方法并进行流畅的编程:

 public static int NotNegative(this int x) => x > 0 ? x : 0; ... N( P() ?? M().NotNegative()); 

完成。 这只会评估M()一次,但超级奖励 ,它更容易阅读,因为现在程序文本代表正在执行的操作,而不是程序文本是一系列难以阅读的标点符号。

一些流畅的扩展方法可以使您的代码更容易阅读。 养成使用它们的习惯。