C# Sodacrate - "refactor"

I am not a programmer, men spontant tycker jag inte man ska skicka in en sträng som ett argument (skicka kanske en enumerator?). Någonstans på vägen känns det ju som att du kommer behöva en switch/ifelse slinga om du gör det dock. En annan grej är ju göra klassen Beverage abstrakt och då istället skapa klasser som Cola som ärver egenskaperna från den. Då innehåller själva klassdefinitionerna dessa värden, och dessa kan ligga samlade under en och samma fil. Då kan du använda dig av polymorphism typ Beverage cola = new Cola();

Kolla upp Polymorphism, tror det är exakt det du letar efter

Kort och gott: Du skapar olika klasser för dina drycker, t.ex. Cola, Fanta, Sprite osv. Alla dessa typer är baserat på din huvud-klass "Beverage" som har variablerna, Pris, Namn och Typ definierat.

Cola är en typ av Beverage, vilket innebär att den kommer ärva sin typs egenskaper (läs: variabler, funktioner m.m).

Ett simpelt exempel nedan visar vad jag menar

public class Beverage{

	public string Name;
	public float Price;
}

public class Cola : Beverage{ // Cola ärver av Beverage

	public Cola(){
		Name = "Cola";
		Price = 18.99;
	}
}

public class Fanta : Beverage{
	
	public Fanta(){
		Name = "Fanta";
		Price = 16.49;
	}	
}


...
// Skapar en lista med dina drycker
List<Beverage> drycker = new List<Beverage>{
	new Cola(),
	new Fanta()
}

Eftersom du läser programmering 1 (?) så är arv inte något du lärt dig i kursen. Men vill du använda det så blir det säkert ett stort plus i kanten.

Det du annars kan göra är att skapa en konstruktor innehålandes en switch baserad på konstruktorns argument. I switchens cases sätter du attributen för objektet.

Exempel om argumentet är "Cola" så sätter du pris m.m. i detta caset.

Edit: Argumentet kan ju vara ett artikelnummer, tex. -> int art = 92132076; eller liknande.

Att skapa en klass för varje sorts dryck fungerar ju, men det är ingen vidare bra lösning eftersom du då t.ex. inte kan lägga till nya drycker enkelt utan att lägga till mer kod. Det är heller inte någon särskilt bra idé att stoppa logiken i Beverage-klassen, eftersom det gör klassen onödigt komplicerad och oflexibel.

Vad du nog egentligen är ute efter är en Factory-klass, d.v.s. en klass vars enda syfte är att skapa nya Beverage-objekt. Fördelen med en sån klass är att du kan ha flera olika metoder för att skapa nya Beverage-objekt, t.ex. från en lista av hårdkodade parametrar, från en fil, från en databas, etc, utan att behöva ändra på Beverage-klassen i sig.

Det kan alltså vara något så enkelt som t.ex. (jag kan inte C# utantill, så koden kanske inte är helt korrekt):

class BeverageFactory
{
  static public Beverage fromName(string name)
  {
    switch (name) {
      case "Cola": return Beverage("Cola", 5, "Brun sörja"); break;
      ...
      default:
        // Felhantering
        break;
    }
  }
}

// Skapa en Cola:
Beverage cola = BeverageFactory.fromName("Cola");