Az objektumhivatkozás nincs beállítva semmilyen objektumpéldányra. Mi a probléma?

Ez a kód nem csinál ilyen hibát, amit írsz.

Egyébként kicsit átgondolhatnád ezt az osztályszerkezetet, mert borzasztó zagyvaság az egész.

Amit most csinál a programod, az az, hogy kettesével olvassa a sorokat, ami nyilván nem jó:

Musorlista.Add(new Musorlista(sr.ReadLine(), sr.ReadLine()));

Az első apraméterhez beolvasol egy sort, a másodikhoz beolvasol még egyet. Emiatt ha páratlan számú sor van a fájlban, akkor a fájl végére érve az első paraméternek még átad egy sort, a másodikra viszont a fájl végére ér, és nullt ad át.

A feladat megoldását ott kéne kezdeni, hogy egy ÉRTELMES adatszerkezetet találsz ki hozzá. Ez, hogy vegyesen vannak space-ekkel, meg kettősponttal elválasztva az adatblokkok, főleg hogy bizonyos adatok tartalmazhatnak space-t, vállalhatatlan, és közel kezelhetetlen. Legyen egy egységes elválasztókarakter, ami az összes adatblokkot elválasztja. Pl pontosvessző:

1;5;3;Deep Purple;Bad Attitude

3;4;33;Omega;A szamuzott

És így máris egy sokkal könnyebben kezelhető adatállományt kapsz (amit egyébként CSV fájlnak hívnak)

A StreamReaderrel kapcsolatos problémákat már leírták, nem ismételném meg, de ezentúl is elég nagy katyvasz van az adatszerkezeteid és osztályaid kapcsán.

Először is van egy ilyen adatszerkezeted:

Műsorlista:class

  id:int

  hossz:TimeSpan

  music:string[2]

Tehát azt mondod, hogy egy műsorlistának van egy azonosítója, hossza és "mjuzikja", ami két szöveg.

Hát nem tudom, én ilyen műsorlistát még nem láttam.

Felénk egy műsorlista dalokból áll.

A másik, mi az hogy music:string[2]? Egy zene az két stringből áll? Nevezd nevén az adattogokat, névtelen tömbindexekkel bohóckodni horrorisztikusan nehézkes lesz.

Sokkal értelmesebb lenne valami olyasmi szerkezet, hogy:

Műsorlista:class

  Dalok:List<Dal>

  TeljesHossz:TimeSpan

Dal:class

  Előadó:string

  Cím:string

  Hossz:TimeSpan

A másik nagy probléma, hogy a Reader osztályod teljes logikája a konstruktorban van. A konstruktor dolga, hogy létrehozza az adatszerkezetet és beállítsa a kezdeti értékeket. Üzleti logikát nem érdemes beleírni, mert nem számít rá senki.

Megintcsak sokkal szebb lenne úgy megoldani, hogy:

Reader:class

  Reader()

  Read(path:string):Műsorlista

1.)

public int Id { get => id; set => id = value; }

Ezekre nem lenne egyszerűbb auto-property?

[link]

Valahogy igy: public int Id{ get; set; }

2.)

Ha már StreamReader-t használsz, olvasd el a dokumentációját, using-al érdemes használni.

[link]

using (StreamReader sr = new StreamReader("TestFile.txt"))

{

string line;

while ((line = sr.ReadLine()) != null)

{

Console.WriteLine(line);

}

}

3.) Ez igy ronda: string[] music=new string[2];

Kételemű tömb helyett inkább használj két string adattagot.

4.) Itt az adatok-ot háromszor is splitteled, ez csak felesleges erőforráshasználat a programtól.

Id = int.Parse(adatok.Split(' ')[0]);

int hour= int.Parse(adatok.Split(' ')[1]);

int minute= int.Parse(adatok.Split(' ')[2]);

Az adatok.Split(' ')-et tedd ki egy változóba.

A song-ot is hasonlóképpen kellene:

string artist = song.Split(':')[0];

string track = song.Split(':')[1];

5.)

Miért van a TimeSpan hossznak default értéke, ha úgyis mindig felülírod?

TimeSpan hossz=new TimeSpan(0,0,0);

Igazából az egész kód elférne 15-20 sorban, és szebb is lenne sokkal!