C#-Klasse kapselt Spielkarte mit einem Bild

[Th69]

Sehe ich auch so wie @DocShoe: die Klasse sollte so bleiben wie sie ist und nur in der GUI-Anwendung solltest du eine Klasse bzw. Methode haben, welche dann das betreffende Bild lädt:

Image GetImage(Karte karte)
{
  string cardName = karte.Farbe + "_" + karte.Wert + ".jpg"; // oder so ähnlich
  
  return new Bitmap(cardName); // alternativ aus Ressource laden
}

Evtl. kannst du ja noch ein Caching einbauen (mittels eines Dictionary<string, Image>).

Edit: Sehe gerade, daß @DocShoe auch fast zeitgleich mit mir geantwortet hat, jedoch ist Hashtable veraltet und sollte
durch das generische Dictionary ersetzt werden (wie ich ja auch geschrieben habe).

[uhomm]

@DocShoe @Th69

Herzlichen Dank an Euch beide!
Die Nutzung der generischen Dictionary-Klasse hatte ich mir auch schon überlegt, war mir aber nicht sicher, ob es vielleicht eine elegantere Variante gibt.
Den Vorschlag werde ich mal weiter verfolgen...mit Kartenwert und -farbe als Schlüssel für den Wert, der dann die Bilddatei ist, die ich als Ressource eingebunden habe.

Das bedeutet aber, dass ich bei der Initialisierung des Dictionary 52 Codezeilen brauche...oder die eingebundenen Bilder nach dem Key benenne. Dann gehts wohl auch in einer Schleife...

LG Uwe

[Th69]

Ja, du solltest dafür dann eine Schleife benutzen (wäre ja eigenartig, wenn du die Bilder anders als die Karten benennen würdest).
Zusätzlich kannst du auch eine String-List (bzw. -Array) für jeweils Farbe und Wert anlegen, welches dann die Bildnamensteile bezeichnet (falls deine Enum-Namen nicht identisch dazu sind):

List<string> ColorNames = new List<string>() { "Karo", "Herz", "Pik", "Kreuz" };
List<string> ValueNames = new List<string>() { "7", "8," "9", "10", "B", "D", "K", "A" };

Und aus diesen beiden Listen dann den Bildnamen zusammensetzen.

[uhomm]

@Th69

Meine enum für die Farbe ist genauso wie bei Dir, der enum für den Wert habe ich mit den ausgeschriebenen Zahlen/Bezeichnungen gemacht; also "Sieben" oder "Dame".
Gerade habe ich alle 52 Bilder umbenannt in z.B. Pik_Bube.png oder Karo_Sieben.png.
Damit werde ich nun das Dictionary problemlos befüllen können
Und kann dann mit einem Karten-Objekt mir das Bild aus dem Dictionary besorgen.
Nochmals vielen Dank an Euch beide!

LG Uwe

[inflames2k]

Hallo uhomm,

eine derartige Umsetzung der alten "Cards.dll" aus WIndows nach C# habe ich im Jahr 2010 schon einmal erstellt.

Den Code habe ich leider nicht mehr zur Hand, aber die Bibliothek kannst du unter https://www.mycsharp.de/wbb2/thread.php?threadid=82049&hilight=cards+dll finden.

Ich habe auch nichts dagegen, wenn du sie durch einen Decompiler jagst. - Eventuell findest du darin ja noch Anregungen.

[Ein ehemaliger Benutzer]

@uhomm sagte in C#-Klasse kapselt Spielkarte mit einem Bild:

Die Klasse sollte möglichst so sein, dass man sie ohne Änderungen sowohl in einem Konsolen- als auch in einem GUI Projekt verwenden kann. In der Konsole kann man dann das Bild nicht nutzen.

vorschlag zur nächsten version: in der konsole das bild als ascii-art anzeigen.

[Ein ehemaliger Benutzer]

@inflames2k sagte in C#-Klasse kapselt Spielkarte mit einem Bild:

eine derartige Umsetzung der alten "Cards.dll" aus WIndows nach C# habe ich im Jahr 2010 schon einmal erstellt.

ich habe zwar keine ahnung von c#, aber ich wette, die cards.dll lässt sich auch von c# aus nutzen.

[inflames2k]

Clever, und das zeigst du mir ab WIndows 7.

[Ein ehemaliger Benutzer]

@inflames2k sagte in C#-Klasse kapselt Spielkarte mit einem Bild:

Clever, und das zeigst du mir ab WIndows 7.

die dll aus dem netz saugen: https://fix4dll.com/cards_dll

[inflames2k]

Egal, kommen wir zum Eingangsthema zurück.

@uhomm
Da du ja selbst schon schreibst, dass die Klassen unabhängig von der GUI sein sollen würde ich davon, wie andere vor mich auch schon, abraten die Grafiken mit in die Klasse aufzunehmen.

Das wäre dann übergeordnete Logik, wie die Darstellung zu erfolgen hat und hat dann in der Klasse nichts zu suchen.