Code nicht zurück zu erwartende Anzahl von geraden und Chancen

Question

Ich habe eine Funktion, die eine Liste von Zahlen aufnimmt und zurückgibt, wie viele gerade Zahlen und ungerade Zahlen gibt es in der Liste. Ich habe jedoch eine Liste von Zahlen übergeben, aber ich bekomme 0 Ergebnisse.

Hier ist meine Funktion -

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); 

    } 

Alle Ideen, was ich falsch hier? Ich debuggte durch, aber keine der Listen inkrementieren.

Dank

Answer 1

Ich stimme mit Schachaf Görtler's Antwort sowie pswgs Kommentar. Just do:

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++; 
} 

Als p.s.g. erwähnt, gibt es nicht so etwas wie eine gerade und ungerade Zahl, also eliminiere das komplett.

Übrigens ruft numOfEvens ++ den Wert und dann erhöht es, weshalb Ihr Code nicht funktioniert hat.

Answer 2

Ihr in der richtigen Art und Weise i% nicht ungerade sind die Berechnung 3 nicht 5, die auch eine ungerade Zahl ist verfängt, versuchen Sie dies stattdessen

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

Answer 3

ich glaube, Sie haben einen Blick sollte bei Ihrem Test für isOdd

Answer 4

Verwenden Sie die Linq Count Erweiterung.

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

Natürlich müssen Sie nicht beide Ausdrücke bewerten, wie im folgenden Kommentar.