Geht der folgende Code auch effizienter?

Hallo zusammen,
ich möchte Daten aus einer Datei einlesen. Diese möchte ich gerne in eine Liste<Person> eintragen lassen. Hinbekommen habe ich es, jedoch stehe ich noch am Anfang was meine Erfahrung in C# angeht und ich denke, dass man das folgende Beispiel effizienter lösen kann:

Die Datei hat folgenden Aufbau:

Code: 1: 2: 3: 4: Vorname; Nachname; Daniel; Müller ... Hans; Schneider

C#-Code:

class Person { public string Vorname{ get; set; } public string Nachname{ get; set; } }

C#-Code:

string path = @"C:\Daten.csv";             var csvData = File.ReadAllLines(path)                    .Skip(1)                    .Select(x => x.Split(';'))                    .Select(x => new Person()                    {                        Vorname = x[0],                        Nachname = x[1]                    });             var Person= new List<Person>();             foreach (var item in csvData)             {                 Person.Add(new Person () {                                                                 Vorname = item.Vorname,                                                                 Nachname = item.Nachname                                                             });             }

Ich hatte probiert an die LinQ Abfrage wie folgt zu erstellen:

C#-Code:

List<Person> csvData  = File.ReadAllLines(path)                    .Skip(1)                    .Select(x => x.Split(';'))                    .Select(x => new Person()                    {                        Vorname = x[0],                        Nachname = x[1]                    }).ToList();

Was leider nicht erfolgreich war (auch andere diverse Formen haben nicht funktioniert).

Was mache ich falsch? Wie bekomme ich die Daten sofort in die List vom Typ Person?

Noch eine andere Fragen:
- Mir ist beim Ausführen aufgefallen, dass ich mal eine Fehlermeldung bekommen habe, dass auf die Datei nicht zugegriffen werden kann da diese von einem anderen Prozess verwendet wird (ich hatte die Datei geöffnet). Daher würde es vermutlich Sinn machen zu prüfen ob die Datei "zur Verfügung" steht. Was würde man noch prüfen, bevor man mit dem File.ReadAllLines auf die Datei zugreift?

Danke für eure Hilfe!

Zitat von mchrd:

Was würde man noch prüfen, bevor man mit dem File.ReadAllLines auf die Datei zugreift?

Hallo mchrd,

eine vorangestellt Prüfung macht keinen Sinn, da selbst wenn deine Prüfung positiv ist, kann die Datei zwischen deiner Prüfung und deinem Zugriff gelöscht oder gelockt werden. In diesem Fall bekommst du wieder eine Exception ... bist also genau so weit wie vorher. Daher einfach versuchen die Datei zu öffnen, so wie du es bereits jetzt machst.

Kleiner Verbesserungsvorschlag:
Anstatt File.ReadAllLines könntest du File.ReadLines verwenden, so dass nicht erst die komplette Datei geladen werden muss.


Grüße
spooky

Hallo,

ich danke euch herzlich für eure Antworten! Diese haben mir wirklich geholfen.

@p!lle
Danke für den Link zu dem guten Artikel. Dass der Typ csvData Person ist wusste ich, da ich aus dem anonymen Typ new ein new Person

C#-Code:

.Select(x => new Person()

gemacht hatte. Nur wollte ich es gerne in einer generischen Liste haben (List<Person>) haben. Der Artikel hat mir aber dennoch geholfen, da ich jetzt einige Dinge dazu gelernt habe.

Ich habe das mit dem zweiten Code durchgespielt und es hat wirklich funktioniert. Ich hatte einen anderen Fehler (dort hatte ich eine Datei verwendet, welche eine Spalte weniger hatte. Danke für den Hinweis!



@Spook
Stimmt das ist ein wirklich guter Einwand (wenn zwischen der positiven Prüfung und dem Aufruf die Datei doch noch von einem anderen Prozess blockiert wird).

Das File.ReadLines zu verwenden ist ein super Hinweis! Danke! Ich habe noch ein wenig nachgelesen und es macht absolut Sinn. Danke!


@Abt
Nein das stimmt Linq ist nicht unbedingt immer performant. Bei SQL Abfragen habe ich das bereits mehrfach gelesen, dass komplexe Abfragen nicht unbedingt derart optimiert sind, dass sie mit einer guten SQL Abfrage mithalten können.

Kannst du mir noch einen Tipp geben, mit welcher Klasse ich die CSV Datei sonst einlesen kann? Da ich beim googlen i.d.R. immer nur Linq-Abfragen finde.
Danke auch für deine Hilfe!



Ich wünsche euch allen einen schönen Abend

Guten Morgen Abt,
danke für deine Antwort.

Zitat von Abt:

LINQ ist nicht SQL. Zitat von mchrd: Bei SQL Abfragen .... Das sind zwei paar Stiefel, die erst mal nichts miteinander zutun haben. Da solltest Du evtl. nochmal genauer lesen ;-)

Ok, dann werde ich das tun. Wird aber sicher noch einige Zeit in Anspruch nehmen bis ich den Stand habe.

Zitat von Abt:

Zitat von mchrd: Kannst du mir noch einen Tipp geben, mit welcher Klasse ich die CSV Datei sonst einlesen kann? Da ich beim googlen i.d.R. immer nur Linq-Abfragen finde. Linq liest Dir nicht Dein CSV ein. Auch bei Deinem Code nicht. Du kannst problemlos auf Linq in Deinem Code verzichten und trotzdem identisch Dein CSV einlesen. Und dass Du "nur Linq" in Zusammenhang mit CSV findest - bezweifle ich sehr stark.

Wenn ich es richtig verstanden habe, gehören die Funktionen Split(), Select(), Skip() zu LINQ. Dann ist es wirklich so, dass ich auf StackOverflow überall mindestes eine der Funktionen gefunden habe. Selbst wenn ich nach "C# read csv file without linq" eingebe, komme ich auf derarige Seiten mit linq Lösungen. Ich finde zwar auch so Ausdrücke wie:

C#-Code:

var query = from line in File.ReadLines(filename)     let csvLines = line.Split(';')     from csvLine in csvLines     where !String.IsNullOrWhiteSpace(csvLine)     let data = csvLine.Split(',')     select new     {         ID = data[0],         FirstNumber = data[1],         SecondNumber = data[2],         ThirdNumber = data[3]     };

Aber selbst hier ist ja das "Split" von LINQ oder verstehe ich das völlig falsch?

Zitat von Abt:

Vielleicht solltest Du auch noch mal lesen, was Linq ist ;-) Gerade Einsteiger tun sich oft schwer mit Linq; weil das Wissen fehlt, was Linq ist - und was nicht. -> Linq ist nichts anderes als ein funktionaler Ansatz in .NET, um gewisse Schreibweisen zu vereinfachen. Alles in Linq besteht aber im Endeffekt aus Schleifen und Bedingungen.

Ja wahrscheinlich habe ich da noch viel bedarf und muss noch einiges lernen um es wirklich zu verstehen.

[quote=Abt]Ein Einstieg dazu:  Besserer C#-Code durch funktionale Programmierung
Aber wie gesagt; übersichtlicher oder schneller ist Linq nicht unbedingt; letzteres aber auch in dem Delta nicht relevant.[/CSHARP]
Danke für den super Link. Von der Basta Konferenz gibt es ja einige interessante Videos.

Danke für deine/eure Hilfe!

Zitat von mchrd:

Aber selbst hier ist ja das "Split" von LINQ oder verstehe ich das völlig falsch?

Wenn sich Dir diese Frage stellt und Dir damit unsicher bist, dann schau in die Doku:  String.Split Method
Da siehst Du schon an Namespace, dass diese eben NICHT zu Linq gehört, wie auch T-Virus schon sagte.

LaTino, der einzige Fall, der mir dazu aktuell einfällt den er meinen könnte wäre, wenn jedes Linq-Segment durch ein ToList aktiv materialisiert wird.

Zitat von T-Virus:

Wenn man diese bei großen Listen, Arrays o.ä. zusammen hängt, muss auch zusätzlich iterriert werden.

Nein.

Entschuldige bitte, wenn ich das so offen sage, aber du hast ganz offenbar weder Iteratoren noch das Prinzip von Linq ansatzweise verstanden.

LaTino

@gfoidl
Hast recht.
Hab mir den Code nochmal angeschaut und ein 0815 Test gefahren.
Hier sehe ich auch, dass erst bei ToList das fertige Ergebnis gebaut wird.
Entsprechend lag ich da wirklich ganz falsch.

Ich bin bisher davon ausgegangen, dass dort Teilmengen durch die Where/Select und andere Methoden geliefert werden, die dann eben von der Folgeoperation wieder verarbeitet werden.
Aber mit meinem Test und etwas Debuggen, habe ich auch gesehen, dass ich falsch lag.
Die Daten werden dann erst beim ToList erst durch die Iteratoren gefiltert und geliefert.

T-virus

Stimmt, hatte ich mit ReadAllLines vertauscht, die meinte ich auch eigentlich.
Ist für den Einstieg verständlicher, wenn man ein String[] bekommt als das IEnumerable<string> :)

Es wird zwar einige Zeit dauern, aber wenn du dran bleibst kommst du auch da hin wo wir heute sind.
Ist alles nur eine Frage der Zeit bis damn einen Stand hat, wo man die meisten Dinge im Alltag abdeckt.
Aber gerade in der Programmierung lernt man nie aus.
Selbst nach 10 Jahren muss man auch immer wieder neues Lernen, weil sich die Praxis hier teilweise sehr schnell verändert.

T-Virus