Hi zusammen,
ich bin ja nicht Beratungsresistent, aber verfalle hin und wieder beim programmieren in alte Muster.
Ich musste mir heute von eiem an derem Programmierer bei meinem Freizeitprojekt anhören, dass ich Oldshool bin.
natürlich ohne das ich darauf hingeweisen wurde, wie es andere besser machen
Bei meinem aktuellen kleinem Hobby Projekt, was nebenbei läuft habe ich Daten in ein DGV geschrieben.
Diese sollen jetzt in einer Datei gespeichert werden. jedoch nur ausgewählte Spalten.
Bisher habe ich das wie folgt gemacht:
Es funktioniert. Keine Frage. Aber ich lese immer mal was von guten und schlechten geschriebenem Code. Gibt ja hier im Forum einen ganzen Eintrag für solche Themen.
Da ich diese "Funktion" öffters in der Konstelation verwendet, aber das auch schon seit jahren, frage ich mich mittlerweile (und nach dem Gespräch heute) ob ich überhaupt noch auf dem Stand der Dinge bin und der Code vieleicht mittlerweile ein schlechter Code ist.
Ob es vieleicht einfacher und besser geht.
Wie handhabt Ihr das?
Nehmt Ihr Code, der immer funktioniert hat, so wie icheinfach weiter? Oder passt ihr alles immer an, wenn sich die Möglichkeit bietet.
Sorry, ich sehe shcon in der Vorschau, dass der Code etwas vershcoben dargestellt wird. keine Ahnung warum.
ich bin ja nicht Beratungsresistent, aber verfalle hin und wieder beim programmieren in alte Muster.
Ich musste mir heute von eiem an derem Programmierer bei meinem Freizeitprojekt anhören, dass ich Oldshool bin.
natürlich ohne das ich darauf hingeweisen wurde, wie es andere besser machen
Bei meinem aktuellen kleinem Hobby Projekt, was nebenbei läuft habe ich Daten in ein DGV geschrieben.
Diese sollen jetzt in einer Datei gespeichert werden. jedoch nur ausgewählte Spalten.
Bisher habe ich das wie folgt gemacht:
Visual Basic-Quellcode
- 'Datei vorbereiten
- Dim file As System.IO.StreamWriter
- file = My.Computer.FileSystem.OpenTextFileWriter(Dateipfad, False)
- 'Variablen für DGV ertsellen
- dim spalte1 as string= nothing
- dim spalte2 as string=nothing
- dim spalte5 as string = nothing
- dim record as string = nothing
- 'DGV zeilenweise einlesen und daten in record variable schreiben
- For i As Integer = 0 To Me.DataGridView1.Rows.Count - 1
- If Me.DataGridView1.Item(0, i).Value.ToString = Nothing Then
- Continue For
- else
- spalte1 = Me.DataGridView1.Item(0, i).Value.ToString
- spalte2 = Me.DataGridView1.Item(1, i).Value.ToString
- spalte3 = Me.DataGridView1.Item(4, i).Value.ToString()
- 'Record Daten zusammentragen
- If i = 0 Then
- record = spalte1 + ";" + spalte2 + ";" + spalte5
- Continue For
- Else
- record = record + System.Environment.NewLine + spalte1 + ";" + spalte2 + ";" + spalte5
- End If
- Next
- 'Wenn etwas in record hinterlegt wurde, dann speichern
- if not record=nothing then
- file.Write(record)
- file.Close()
- end if
Es funktioniert. Keine Frage. Aber ich lese immer mal was von guten und schlechten geschriebenem Code. Gibt ja hier im Forum einen ganzen Eintrag für solche Themen.
Da ich diese "Funktion" öffters in der Konstelation verwendet, aber das auch schon seit jahren, frage ich mich mittlerweile (und nach dem Gespräch heute) ob ich überhaupt noch auf dem Stand der Dinge bin und der Code vieleicht mittlerweile ein schlechter Code ist.
Ob es vieleicht einfacher und besser geht.
Wie handhabt Ihr das?
Nehmt Ihr Code, der immer funktioniert hat, so wie icheinfach weiter? Oder passt ihr alles immer an, wenn sich die Möglichkeit bietet.
Sorry, ich sehe shcon in der Vorschau, dass der Code etwas vershcoben dargestellt wird. keine Ahnung warum.