eleganter Code für wenn in swift

Question

Ich habe einen einfachen Code, der funktioniert und den ich in altmodischer Weise programmieren und ich bin mir sicher, es gibt eine elegantere Art, dies in swift zu tun. Hier ist der Code:

var cardsInCompartment1:Int = 0 
    var cardsInCompartment2:Int = 0 
    for card in cards{ 
     if card.compartment == 1{ 
      cardsInCompartment1 += 1 
      print(cardsInCompartment1) 
     } 
     if card.compartment == 2{ 
      cardsInCompartment2 += 1 
      print(cardsInCompartment2) 
     } 
    } 

Ich habe im Grunde Karten in verschiedenen Fächern und jetzt will ich zählen, wie viele Karten in jedem Fach ist.

Answer 1

Wie wäre es mit filter die gewünschten Karten auszuwählen? Dann können Sie einfach count sie:

let cardsInCompartment1 = cards.filter { $0.compartment == 1 }.count 
let cardsInCompartment2 = cards.filter { $0.compartment == 2 }.count 

---

Wenn Sie eine Reihe von Fächern haben, können Sie die Zählungen in einem Wörterbuch speichern könnte:

var compartmentCounts = [Int:Int]() 

cards.forEach { 
    compartmentCounts[$0.compartment] = (compartmentCounts[$0.compartment] ?? 0) + 1 
} 

In diesem Fall würde der Schlüssel sein, der Fach #, und der Wert wäre die Kartenanzahl. Etwas wie [1: 32, 2: 42], wenn 32 und 42 Karten in jedem Fach vorhanden sind.

Answer 2

Versuchen Sie folgendes:

var cardsInCompartment1:Int = 0 
var cardsInCompartment2:Int = 0 
for card in cards { 
    (card.compartment == 1) ? (cardsInCompartment1 += 1) : (cardsInCompartment2 += 1) 
} 

Answer 3

Was ist mit einer switch-Anweisung? Etwas wie das?

var card:Int = 1 
var CardsInCompartment:Int = 0 
switch (card) { 
case 1: 
    CardsInCompartment += 1 
    print("CardsInCompartment \(CardsInCompartment)") 
case 2: 
    CardsInCompartment += 2 
    print("CardsInCompartment \(CardsInCompartment)") 
default: 
} 

Answer 4

Oder verwenden Sie einen Array Sie zählt zu halten:

var counts = [ 0, 0, 0 ] // create an array of integers, where the Ints in the array represent the count of cards in each compartment 
cards.forEach { counts[ $0.compartment ] += 1 } // for each card, increment the count in array corresponding to the compartment of the card. (if card.compartment == 1, increment counts[1], and so on 

print("cards in compartment 1 \(counts[1])") 
print("cards in compartment 2 \(counts[2])") 

(Dies setzt voraus, Ihre einzigen Fächer sind ganze Zahlen 1 und 2)

Answer 5

Ich glaube, Sie die cardsInCompartment als Arrays gespeichert werden sollen:

var cardsInCompartment = [0, 0] // you can add more to this array 

Dann können Sie einfach Schleife durch die cards und die Werte zu den Array-Elementen hinzufügen:

Answer 6

Ich mag Aaron Brager Idee, die Werte in Wörterbuch zählt. Ich benutze reduzieren, um änderbare Wörterbuch außerhalb der "Schleife" (mehr funktionale) zu beseitigen

let d = cards.reduce([:]) { (d, card) -> [Int:Int] in 
    var d = d 
    let s = d[card.compartment] ?? 0 
    d[card.compartment] = s + 1 
    return d 
}