string.format Klasse funktioniert nicht wie gewünscht

popopirat97 · 23. Februar 2016, 21:36

Hi Leute,

ich probiere einen Double (immer) mit 2 Nachkommastellen anzeigen zu lassen und habe dafür folgende Funktion geschrieben:

VB.NET-Quellcode

Public Shared Function format(ByVal Input As Double) As Double
MsgBox(Input)
Return String.Format("0.00", Input)
End Function

Die Msgbox ist wegen der Fehleranalyse da, dachte es harkt bei dem Input, der kommt aber entsprechend richtig rein.

rufe ich die Funktion nun über Msgbox(system_funktionen.format(20.3)) auf, gibt er mir folgende Msgboxen wieder:

1:
20.3 (die Fehleranalyse Box) <- Richtig
2:
0 <- hier sollte 20,30 rauskommen

Grüße
Julian

ErfinderDesRades · 23. Februar 2016, 22:27

du proggst offsichtlich Stict off, und beachtest die Datentypen nicht richtig. Mach zunächstmal Strict On
Mach es ein für alle mal.
Wenn du ein Projekt übernimmst, überprüfe, ob Strict On ist.
Einfach immer Strict On.
Oder wechsel zu c#, da gibts den Strict Off - Unfug garnet.

Nochmal Anleitung, wie man ein für allemal Strict On macht: Visual Studio - Empfohlene Einstellungen

(oder sollte ich dir das noch nie vorgesetzt haben?)

mox · 23. Februar 2016, 23:00

und msgbox ist auch veraltet.

RodFromGermany · 24. Februar 2016, 07:15

popopirat97 schrieb:

VB.NET-Quellcode

Return String.Format("0.00", Input)

machst Du

VB.NET-Quellcode

Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click
MessageBox.Show(format(123.456))
End Sub
Public Shared Function format(ByVal Input As Double) As String
Return String.Format("{0:0.00}", Input)
End Function

ErfinderDesRades · 24. Februar 2016, 08:44

Hmm - das kanner sich jetzt wieder auskopieren und Danke, und Fehler nicht verstanden, und auf zum nächsten Fehler, der Datentypen nicht respektiert.

RodFromGermany · 24. Februar 2016, 09:08

ErfinderDesRades schrieb:

Fehler nicht verstanden

Die Frage ging auf das Ergebnis "0" statt "123.45", das lässt sich mit On nicht beheben.

Schamash · 24. Februar 2016, 09:45

Oder man nimmt einfach:

VB.NET-Quellcode

input.tostring("#0.00")

Niko Ortner · 24. Februar 2016, 13:21

RodFromGermany schrieb:

ErfinderDesRades schrieb:

Fehler nicht verstanden

Die Frage ging auf das Ergebnis "0" statt "123.45", das lässt sich mit On nicht beheben.

Aber man hätte Schritt für Schritt auf die Lösung hinarbeiten können.
Und doch, erklärt es schon zum Teil. String.Format("0.00" ...) ergibt immer "0.00". Und dass das zu Double konvertiert 0 ergibt ist klar. Man muss aber erst mal wissen, dass da ein String zu Double konvertiert wird, was durch Option Strict Off aber verschleiert wird.
Und dann kann man auch gleich weiter gehen: String.Format bzw. Double.ToString mit dem Format-String eignet sich, um eine String-Darstellung zu bekommen. Aber da die Funktion Double zurückgibt, wird vielleicht wirklich ein Double benötigt (auch ein Problem mit Option Strict Off: man weiß es nicht so genau). In dem Fall wäre Math.Round angebracht.

popopirat97 · 24. Februar 2016, 20:55

Hallo,

danke erstmal für die Antworten. Die Funktion muss natürlich einen String wiedergeben, damit die Nachkommastellen angezeigt werden. Zeimlich grober Schnitzer :huh:

Die Funktion sieht also nun wie folgt aus:

VB.NET-Quellcode

Public Shared Function format(ByVal Input As Double) As String
Return String.Format("{0:0.00}", Input)
End Function

ErfinderDesRades · 24. Februar 2016, 22:48

und konnte dir nur unterlaufen, weil? (Strict Off)

ansonsten wie Schamasch schon sagt: Eine Methode mit so wenig Inhalt kann man weglassen, da rufe man besser den Inhalt direkt auf - Schamash hat ja gezeigt, wie.

popopirat97 · 26. Februar 2016, 13:11

und konnte dir nur unterlaufen, weil? (Strict Off)

Nachdem ich Strict On gemacht habe, sah der code wie folgt aus:

VB.NET-Quellcode

Public Shared Function format(ByVal Input As Double) As Double
Return CDbl(String.Format("{0:0.00}", Input))
End Function

dort war der Denkfehler also immernoch vertreten.

Naja wie gesagt, Problem gelöst, danke alle!

RodFromGermany · 26. Februar 2016, 13:34

Du wechselst häufig Deklarationen und Inhalte.
Beschreib doch bitte mal verbal, was die

popopirat97 schrieb:

Function format

ühaupt machen soll.

ErfinderDesRades · 26. Februar 2016, 14:34

popopirat97 schrieb:

Nachdem ich Strict On gemacht habe, sah der code wie folgt aus:

Kann nicht sein.
Option Strict On verändert nichts am Code - es weist nur den Compiler an, Fehler zu bemeckern.

EaranMaleasi · 26. Februar 2016, 16:53

ErfinderDesRades schrieb:

Fehler zu bemeckern.

, für die VS einem daraufhin möglichkeiten zur Behebung bietet.

ErfinderDesRades · 26. Februar 2016, 19:29

hmm hmm - "Möglichkeiten zur Behebung" ist wohl stark übertrieben. Es sind dumme automatische Standard-Vorschläge - meist Unfug.

Und wenn man denen blindlings folgt, kommt evtl. so son kololores wie in post#12 bei raus (und ich bin dran schuld, weil ich gesagt hab "Strict On ist wichtig"

).
Aber es bleibt wie's ist: Programmieren muss der Mensch, vorzugsweise mit angeschaltetem Gehirn. Und dabei die Datentypen beachten.

popopirat97 · 26. Februar 2016, 23:47

Kann nicht sein.
Option Strict On verändert nichts am Code - es weist nur den Compiler an, Fehler zu bemeckern.

Ich habe mich falsch ausgedrückt. Was ich meinte war, dass ich den Code danach so angepasst hatte, dass das gleiche Ergebnis rauskommen soll. Es sind immernoch die Doubles da wo sie vorher waren und ebenso die Strings. Ich habe nur die nötigen Konvertierungen vorgenommen. Ich meinte also, dass Strict On das Problem nicht behoben hätte. :huh:

Du wechselst häufig Deklarationen und Inhalte.

Beschreib doch bitte mal verbal, was die
popopirat97 schrieb:
Function format
ühaupt machen soll.

Ich schreibe ein Programm zum sagen wir "rumrechnen von Zahlen". Wenn ich also einen Geldbetrag ausgeben will, soll diese Funktion die Formatierung automatisch durchführen. Also eine Formatierungsfunktion, daher auch die Nomenklatur.

Grüße

ErfinderDesRades · 27. Februar 2016, 07:32

also die Funktion, die in post#12 format() heisst, die formatiert nicht, sondern sie rundet einfach nur. Da nimm besser System.Math.Round()

Allerdings braucht man Runden eigentlich fast nie, und ist auch bischen unschön, denn beim Runden geht Information verloren.
Gerundet braucht man die Zahlendarstellung in Strings, und dafür ist die String.Format()-Methode da, bzw. die .ToString()-Methoden der verschiedenen Zahl-Typen.

Interessierts dich, was über Formatierung zu lernen?

RodFromGermany · 27. Februar 2016, 12:15

popopirat97 schrieb:

die Formatierung automatisch durchführen

machst Du bei Geld so ("c" steht für Currency):

VB.NET-Quellcode

Dim geld = 123.456
MessageBox.Show(geld.ToString("c"))

popopirat97 · 28. Februar 2016, 02:23

@RodFromGermany
Die Variante ist mir durchaus bekannt, jedoch rechne ich mit den Zahlen weiter.
Daher macht das nur mehr umstände, als dass es die Sache vereinfacht.
Aber danke für deine Mühe

Grüße
Julian

RodFromGermany · 28. Februar 2016, 09:53

popopirat97 schrieb:

jedoch rechne ich mit den Zahlen weiter.

Mit den gerundeten?
Das üürde ich als Fehler ansehen.

string.format Klasse funktioniert nicht wie gewünscht

string.format Klasse funktioniert nicht wie gewünscht

VB.NET-Quellcode

popopirat97 schrieb:

VB.NET-Quellcode

VB.NET-Quellcode

ErfinderDesRades schrieb:

VB.NET-Quellcode

RodFromGermany schrieb:

ErfinderDesRades schrieb:

VB.NET-Quellcode

VB.NET-Quellcode

popopirat97 schrieb:

popopirat97 schrieb:

ErfinderDesRades schrieb:

popopirat97 schrieb:

VB.NET-Quellcode

popopirat97 schrieb:

Tags

Ähnliche Themen

7 Benutzer haben hier geschrieben