コードが偶数とオッズの期待数を返さない

Question

私は、数値のリストを取り、リストにいくつの偶数と奇数があるかを返す関数を持っています。しかし、私は数字のリストを渡したが、私は0の結果を得ている。ここで

は私の関数である -

public static string HowManyEvenAndOdds(List<int> numbers) 
    { 
    int numOfOdds = 0; 
    int numOfEvens = 0; 
    int numOfBoth = 0; 

    foreach (int i in numbers) { 
     bool isEven = i % 2 == 0; 
     bool isOdd = i % 3 == 0; 

     numOfBoth = isEven && isOdd ? numOfBoth++ : numOfBoth; 
     numOfEvens = isEven ? numOfEvens++ : numOfEvens; 
     numOfOdds = isOdd ? numOfOdds++ : numOfOdds; 
    } 

    return string.Format("This list has {0} odd numbers,\n{1} even numbers,\nand {2} numbers that are even and odd.", numOfOdds, numOfEvens, numOfBoth); 

    } 

任意のアイデア私が間違ってここにやっているものに？私はそれを通してデバッグしましたが、リストのどれも増分していません。あなたは正しい方法で I％3を奇数計算されていません

おかげ

Answer 1

私はSchswaf Gortlerの答えとpswgのコメントに同意します。ちょうど：

foreach (var number in numbers) 
{ 
    // A number is even if, and only if, it's evenly divisible by 2 
    if (number % 2 == 0) 
     numEvens++; 
    // A number is odd if, and only if, it's NOT evenly divisible by 2 
    // Alternatively, a number is odd if it isn't even and vice versa 
    else 
     numOdds++; 
} 

as p.s.w.g.前述のように、偶数と奇数の両方の数字はないので、完全に削除してください。

numOfEvens ++は値を取得してからをインクリメントするため、コードが機能しませんでした。

Answer 2

も奇数である5をキャッチしていない、私はあなたが見ているべきだと思う

bool isEven = i % 2 == 0; 
bool isOdd =!isEven; 

Answer 3

代わりにこれを試してくださいあなたのテストでisOdd

Answer 4

Linq Count拡張子を使用してください。

int numOfOdds = numbers.Count(x => x % 2 != 0); 
int numOfEvens = numbers.Count(x => x % 2 == 0); 

もちろん、両方の式を次のように評価する必要はありません。