一种缩短if语句的方法

我有一个四个数字的数组，我需要将它与另一个数字数组进行比较。我应该告诉这些数字中有多少是正确的，有多少是正确的数字，但不是正确的。一种缩短if语句的方法

F.eks: 
Code: 1501 
Guess: 1305 
Right numbers: 3 
Right placement: 2

正确的位置很容易。但是正确的数字部分有点棘手。这就是我解决它的方法：

//Check if number exist in context 
     Rn = 0; 

     for (int g = 0; g < 4; g++) 
     { 
      resent[g] = 6; 
     } 

     for (int i = 0; i < 4; i++) 
     { 
      for (int j = 0; j < 4; j++) 
      { 
       if 
        (
         guess[i] == Numbers[j] 
        && guess[i] != resent[0] 
        && guess[i] != resent[1] 
        && guess[i] != resent[2] 
        && guess[i] != resent[3] 
        ) 
       { 
        Rn++; 
        resent[i] = guess[i]; 
       } 
      } 

     } 
     ConsolePost("Right Numbers: " + Rn);

有没有一种方法可以做到这一点。或者是否有可能缩短这个？

像：

if(guess[i] == Numbers[j] && != resent[0-3]) 
{ 
    Something; 
}

来源

2014-08-28 RakNoel

这应该是对代码评审：http://codereview.stackexchange.com/ – 2014-08-28 15:20:42

'猜[I] ==编号[j]的&& resent.Take（4）。所有（R => R ！= guess [i]）'如果'resent' ***的长度是*** 4，那么你可以跳过呼叫'.All' – spender 2014-08-28 15:21:34

虽然有很好的想法，但我认为你会把奥斯卡与所有这些方法混淆。确保他知道这些扩展方法和lambda等等。看看他提供的代码。对我来说似乎是一个基本的代码。 – VasileF 2014-08-28 15:24:31

您可以使用Enumerable.Take(4)，并包含您的resent像：

!resent.Take(4).Contains(guess[i])

你的支票可能是：

if(guess[i] == Numbers[j] && !resent.Take(4).Contains(guess[i]))

参见：Enumerable.Take<TSource> Method

从序列的开头处返回指定数量的连续元素。

如果resent大小始终是4，那么你可以跳过Take(4)，而是只使用Contains像：

if(guess[i] == Numbers[j] && !resent.Contains(guess[i]))

来源

2014-08-28 15:18:00 Habib

@Matthew，是的，谢谢:) – Habib 2014-08-28 15:19:11

如果'resent []'总是大小为4，那么当然可以省略'.Take（4）'。 – 2014-08-28 15:22:05

@MatthewWatson，是的，我应该补充说我的回答 – Habib 2014-08-28 15:24:15

你可以用任何（）来检查，如果你想改变你的数组的大小：

!resent.Any(c=>c==guess[i])

来源

2014-08-28 15:23:29

哈哈^^这个人工作谢谢你。但不是我所需要的^^ – RakNoel 2014-08-28 19:26:02

一种缩短if语句的方法

回答

相关问题