vb.net simple ErrorHandler

04/27/2014 07:33 iSam34#1
An Stellen, wo ein Fehler auftreten könnte, kann man Folgendes einsetzen :

Code:
On Error GoTo ErrHandler
Dazu muss vor
Code:
End Sub
ein
Code:
ErrHandler:
        Exit Sub
eingesetzt werden.

In meinem Fall (Code ersetzt den Text in einer Datei) :

[Only registered and activated users can see links. Click Here To Register...]

Erklärung.
On Error GoTo ErrHandler =
Wenn ein Fehler auftritt, wird der Code an der Stelle ErrHandler: fortgeführt.

Exit Sub =

[Only registered and activated users can see links. Click Here To Register...]

-Dat Random Guy Making Shitty Turorials Because He Is Bored
04/27/2014 07:40 Requi#2
Nein.

Code:
Try
[...]
Catch ex As Exception
MessageBox.Show(ex.ToString())
//oder
Console.WriteLine(ex.ToString())
End Try
Fehler analysieren, wenn nötig Google nutzen und Fehler beheben.

Wer auch immer dieses GoTo nutzt gehört an die Wand geschlagen.
04/27/2014 07:43 iSam34#3
Quote:
Originally Posted by Requi View Post
Nein.

Code:
Try
[...]
Catch ex As Exception
MessageBox.Show(ex.ToString())
//oder
Console.WriteLine(ex.ToString())
End Try
Fehler analysieren, wenn nötig Google nutzen und Fehler beheben.

Wer auch immer diese ":" Labels nutzt gehört an die Wand geschlagen.
Wieso wusste ich das die erste Antwort wieder etwas negatives ist... Ich meine wenn man nen ErrorHandler braucht...

btw: Hast mich auf ne Idee gebracht, danke :)
04/27/2014 07:47 Requi#4
Quote:
Originally Posted by iSam34 View Post
Wieso wusste ich das die erste Antwort wieder etwas negatives ist...
Das ist nicht negativ, sondern eine Verbesserung, da man wenn man anderen Menschen etwas lehren möchte, es nicht auf die falsche Weise machen sollte.
Quote:
Originally Posted by iSam34 View Post
Ich meine wenn man nen ErrorHandler braucht...
Dann macht man es richtig. Die Methode bei jedem klitze-kleinen Fehler zu beenden ist ineffektiv und macht das Programm ziemlich nutzlos.
04/27/2014 07:49 iSam34#5
Quote:
Originally Posted by Requi View Post
Das ist nicht negativ, sondern eine Verbesserung, da man wenn man anderen Menschen etwas lehren möchte, es nicht auf die falsche Weise machen sollte.

Dann macht man es richtig. Das Programm bei jedem klitze-kleinen Fehler zu beenden ist ineffektiv und macht das Programm ziemlich nutzlos.
Das Programm wird nicht beendet. Der aktuelle Sub. Wenn zum Beispiel wie in meinem Fall keine Datei angegeben ist, in die der neue Text gespeichert werden soll, dann speichert er einfach nicht.
04/27/2014 07:52 Requi#6
Quote:
Originally Posted by iSam34 View Post
Das Programm wird nicht beendet. Der aktuelle Sub. Wenn zum Beispiel wie in meinem Fall keine Datei angegeben ist, in die der neue Text gespeichert werden soll, dann speichert er einfach nicht.
Habe meinen Post korrigiert. Selbst wenn nur die Methode einfach beendet wird, ist es ineffektiv.

Anstatt wenn die Datei nicht angebenen ist, einfach gar nichts zu machen, wäre es intelligenter eine neue Datei zu erstellen mit einem wiedererkennbaren Namen und dort den Text zu speichern.

Oder wie bei deinem Code es am Besten wäre, zu überprüfen ob die Textboxen gefüllt sind und eine Meldung (MessageBox.Show()) auftauchen lassen die sagt, dass dort etwas eingetragen werden MUSS.
04/27/2014 07:52 iSam34#7
Zur Veranschaulichung :

[Only registered and activated users can see links. Click Here To Register...]

Es ist keine Datei angegeben, die gespeichert werden soll. Der Speichervorgang bricht ab, nicht die Ausführung des Programmes.

Edit :

Da das Programm dafür da ist, irgendeine Datei auszulesen (also nicht nur *.txt), ist das sinnlos.
04/27/2014 07:57 Requi#8
Quote:
Originally Posted by iSam34 View Post
Zur Veranschaulichung :

[Only registered and activated users can see links. Click Here To Register...]

Es ist keine Datei angegeben, die gespeichert werden soll. Der Speichervorgang bricht ab, nicht die Ausführung des Programmes.
Dein Code verbesserte Version:
Code:
Private Sub Button2_Click(object sender, EventArgs e) Handles Button2.Click
If TextBox1.Text.Length > 0 And TextBox2.Text.Length < 0 Then
Try
File.WriteAllText(...)
Catch FileNotFoundException
MessageBox.Show("Du musst einen Pfad angeben")
End Try
End If
End Sub
Quote:
Originally Posted by iSam34 View Post
Edit :

Da das Programm dafür da ist, irgendeine Datei auszulesen (also nicht nur *.txt), ist das sinnlos.
Dann fragst du vorher ab, falls der Pfad nicht existiert, ob es auf dem Desktop abgelegt werden soll mit Standardname.Standardendung
04/27/2014 07:59 iSam34#9
Quote:
Originally Posted by Requi View Post
Dein Code verbesserte Version:
Code:
Private Sub Button2_Click(object sender, EventArgs e) Handles Button2.Click
If TextBox1.Text.Length > 0 And TextBox2.Text.Length < 0 Then
Try
File.WriteAllText(...)
Catch FileNotFoundException
MessageBox.Show("Du musst einen Pfad angeben")
End Try
End If
End Sub
Es ist aber ein Tutorial, mit einem Beispiel. Du hast soeben einen anderen Lösungsweg für dieses Beispiel angegeben. Das ist auch nicht Sinn der Sache. Trotzdem danke für deinen Code. :awesome:
04/27/2014 08:04 Requi#10
Quote:
Originally Posted by iSam34 View Post
Es ist aber ein Tutorial, mit einem Beispiel. Du hast soeben einen anderen Lösungsweg für dieses Beispiel angegeben. Das ist auch nicht Sinn der Sache. Trotzdem danke für deinen Code. :awesome:
Es ist aber ein Tutorial mit einem schlechten Beispiel und schlechte Beispiele sind nunmal wie man es schon liest/hört schlecht.

Wie schon oben erwähnt, sollte man jemanden das Coden nicht mit schlechten Angewohnheiten beibringen.

Man möchte am Ende effektiven, funktionierenden und wenn möglich schnellen Code und keinen Haufen Müll als Endprodukt.
Nur weil Microsoft etwas einbaut, heißt es noch lange nicht, dass man es nutzen sollte.
04/27/2014 11:18 'Heaven.#11
Cool, wusste ich noch gar nicht. Du solltest aber den Titel zu VB6 ändern, sonst denkt man noch etwas falsches hier
04/27/2014 17:55 Mostey#12
Ich sehe keinen vernünftigen Grund, ein goto Konstrukt einem simplen try/catch Block vorzuziehen. Du kannst dort sogar effektiv bestimmte Exceptions abfangen und festlegen, wie diese behandelt werden sollen. Für deinen goto Errorhandler müsstest du entweder jedes mal einen neuen Errorhandler definieren oder mit if-Abfragen prüfen, was du meistens nicht kannst.
04/27/2014 18:20 tolio#13
pffff diese aufregung hier...

sag dem TE doch einfach lieber mal einer das dieses konstrukt vb6 ist.
Vb.net ist eine andere technologie somit gibt es dort andere und bessere möglichkeiten