W
Ich glaube, der Hinweis von @Th69 ist etwas kurz geraten. Deine Funktionen EUR und EURT sollten nicht Member der Klasse sein, sondern frei bleiben. Viel mehr würde ich mir Gedanken machen, was eigentlich genau die Aufgabe dieser Funktionen ist. Füge JEDER Funktion einen kurzen Kommentar hinzu, was diese Funktion tut, was ihre Input-Parameter sind und was sie zurückgibt. Wenn das schwer ist, weil die Beschreibung kompliziert wäre, dann ist die Funktion nicht gut. Warum hängen diese beiden Funktionen von Form1 ab? Das ist schlecht, wie Th69 schon schrieb. Hier wäre es aber keine sinnvolle Lösung, die Funktion einfach als Member zu verwenden. Vielmehr sollte sie wohl noch einen dritten Parameter, nämlich den Float, den du aus dem Form1->Edit1 holst, bekommen.
Und wo ich gleich dabei bin: deine Funktion EURT - was tut die? Was bedeuten all die x1 und x2-Werte? Wer soll das verstehen/lesen? Viel klarer wären Funktionen celsius_to_fahrenheit und fahrenheit_to_celsius, vielleicht auch noch je 2 Fkt zur Konvertierung in Kelvin. Dann sieht man am Namen direkt, was die Funktion tut. Diese braucht dann nur einen float als Parameter zu bekommen. Und selbstverständlich darf sie nicht von Form1 (oder irgendeiner anderen Form) abhängen, denn die Umrechnung hat überhaupt nichts mit deinen Forms zu tun. Im OnChange/OnClick oder anderen Events der Form kannst du dann die Konvertierungsfunktion aufrufen.
