Problem beim Vergleich zweier Variabeln

[°"Crazy"°]

Grüße!

Ich habe ein Problem beim Vergleich von 2 Variabeln. Ich habe es nun so gelöst, doch bekomm ich eine Errormeldung.

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old >= $ver_new Then
	$upd_nn
ElseIf $ver_old > $ver_new Then
	$upd_nwd
EndIf

$upd_nn und $upd_nwm werden in einem Label angezeigt. Je nachdem, was zuerst zutrifft.

Da mir google nichts hilfreiches zur Zeit ausspuckt, suche ich nun Rat bei Euch. Habe ich einen Fehler in dem Ding gemacht oder kann man dafür etwas besseres verwenden?

MfG

[.AppleTree.]

Ich sehe nicht ganz was dir diese einzelne Variable im nirgendwo bringen soll wenn sie keinen bezug zu etwas anderem hat...

[°"Crazy"°]

#include <ButtonConstants.au3>
#include <ComboConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#Include <string.au3>
#include <INet.au3>

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."
$ver_old = IniRead("version.ini", "version", "version", "Core not found!")
/version.ini"), "[version]" , @CRLF)
$ver_new = _INetGetSource("http://secretgaming.de/trinity/version.ini")
_StringBetween($ver_new, "[version]", @CRLF)



If $ver_old >= $ver_new Then
	$upd_nn
Elseif $ver_old > $ver_new Then
	$upd_nwd
	EndIf
	


#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Privatserver Updater", 620, 215, 433, 143)
$Combo1 = GUICtrlCreateCombo("TrinityCore", 208, 16, 201, 25)
$Label2 = GUICtrlCreateLabel("Ihre aktuelle Version", 128, 72, 116, 25)
$Label3 = GUICtrlCreateLabel("Verfügbare Version", 376, 72, 116, 25)
$Label4 = GUICtrlCreateLabel($ver_old, 144, 104, 84, 25)
$Label5 = GUICtrlCreateLabel($ver_new, 392, 104, 84, 25)
$Button1 = GUICtrlCreateButton("Update", 256, 176, 107, 33, $WS_GROUP)
$Label6 = GUICtrlCreateLabel("$update2", 168, 144, 276, 25)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###


While 1
	$nMsg = GUIGetMsg()
	Switch $nMsg
		Case $GUI_EVENT_CLOSE
			Exit

	EndSwitch

WEnd

Das ist zur Zeit mein Teilscript. Damit hast du den Bezug.

[klavier22]

If $ver_old >= $ver_new Then ;Falls $ver_old größer oder gleich Dann
$upd_nn ;tue bla
ElseIf $ver_old > $ver_new Then ; falls alte Version größer als neue Dann
$upd_nwd ;Bla
EndIf

Ich glaube, das erklärt, warum du deinen Fehler hast. Wenn das 1. zutrifft, trifft auch das 2. zu. besser wäre so:

PHP Code:

If $oldversion < $NewVersion Then
MsgBox(4096,"DU", "KOMISCHER KAUTZ; DEINE TREIBER SIND ALT!")
ElseIf $OldVersion = $NewVersion Then
MsgBox(4096, "DU" ,"OTTONORMALVERBRAUCHER; DEINE TREIBER SIND OK!")
ElseIf $OldVersion > $NewVersion Then
MsgBox(4096, "Sehr lustig,", "Deine Treiber sind neuer als die neueste Version. HAHA")
EndIf 


Außerdem hast du da auch einen Fehler:

PHP Code:

$ver_old = IniRead("version.ini", "version", "version", "Core not found!")
/version.ini"), "[version]" , @CRLF) 


unter den includes...

weiterhin ist deine Neue Version ausleseVariante falsch, sie liest zwischen dem Header [Version] und dem Leerzeichen. Da eine .ini aber so aufgebaut ist:

PHP Code:

[Version]
Ver_New=3.1 


findet er ja automatisch nichts. Du mpsstest theoretisch
_StringBetween($ver_new, "Ver_New=", @CRLF)
machen

[.AppleTree.]

@Klavier das Vergleichen ist schon korrekt fals du das gemeint hast.

#include <ButtonConstants.au3>
#include <ComboConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#Include <string.au3>
#include <INet.au3>

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."
$ver_old = IniRead("version.ini", "version", "version", "Core not found!")
/version.ini"), [COLOR="Red"]"version"[/COLOR] , @CRLF)
$ver_new = _INetGetSource("http://secretgaming.de/trinity/version.ini")
_StringBetween($ver_new, [COLOR="red"]"version"[/COLOR], @CRLF)



If $ver_old >= $ver_new Then
	[COLOR="red"]GUICtrlSetData ($Label6, $upd_nnn)[/COLOR]
Elseif $ver_old > $ver_new Then
	[COLOR="Red"]GUICtrlSetData ($Label6, $upd_nwd)[/COLOR]
	EndIf
	


#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Privatserver Updater", 620, 215, 433, 143)
$Combo1 = GUICtrlCreateCombo("TrinityCore", 208, 16, 201, 25)
$Label2 = GUICtrlCreateLabel("Ihre aktuelle Version", 128, 72, 116, 25)
$Label3 = GUICtrlCreateLabel("Verfügbare Version", 376, 72, 116, 25)
$Label4 = GUICtrlCreateLabel($ver_old, 144, 104, 84, 25)
$Label5 = GUICtrlCreateLabel($ver_new, 392, 104, 84, 25)
$Button1 = GUICtrlCreateButton("Update", 256, 176, 107, 33, $WS_GROUP)
$Label6 = GUICtrlCreateLabel($update2, 168, 144, 276, 25)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###


While 1
	$nMsg = GUIGetMsg()
	Switch $nMsg
		Case $GUI_EVENT_CLOSE
			Exit

	EndSwitch

WEnd

Bitte nenne alles bissel um in Sinvolle Variablen sonst sitzt du plötzlich stoned vorm PC und hast kp was geht^^

[°"Crazy"°]

@klavier.

Beim 2. Fehler habe ich es hier gelöscht. Schon blöd, wenn man keine Farbe hier hat. Da hast du vollkommen recht.

Ansonsten möchte ich keine MsgBox haben, sondern es als Variable ausgespuckt haben. Besser ist es natürlich wie bei Creeper, nur ohne folgende Fehlermeldung:

GUICtrlSetData ($Label6, $upd_nnn)
GUICtrlSetData (^ ERROR
>Exit code: 1    Time: 1.025

@Creeper
Die Variablenamen ergebe bei mir Sinn. Man muss sie selbst verstehen können.

Danke bisher von Euch beiden.


Edit:

#include <ButtonConstants.au3>
#include <ComboConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#Include <string.au3>
#include <INet.au3>

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."
$ver_old = IniRead("version.ini", "version", "version", "Core not found!")

$ver_new = _INetGetSource("http://secretgaming.de/trinity/version.ini")
_StringBetween($ver_new, "version=", @CRLF)



If $ver_old >= $ver_new Then
	GUICtrlSetData ($Label6, $upd_nnn)
Elseif $ver_old < $ver_new Then
	GUICtrlSetData ($Label6, $upd_nwd)
	EndIf
	


#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Privatserver Updater", 620, 215, 433, 143)
$Combo1 = GUICtrlCreateCombo("TrinityCore", 208, 16, 201, 25)
$Label2 = GUICtrlCreateLabel("Ihre aktuelle Version", 128, 72, 116, 25)
$Label3 = GUICtrlCreateLabel("Verfügbare Version", 376, 72, 116, 25)
$Label4 = GUICtrlCreateLabel($ver_old, 144, 104, 84, 25)
$Label5 = GUICtrlCreateLabel($ver_new, 392, 104, 84, 25)
$Button1 = GUICtrlCreateButton("Update", 256, 176, 107, 33, $WS_GROUP)
$Label6 = GUICtrlCreateLabel("Update2", 168, 144, 276, 25)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###


While 1
	$nMsg = GUIGetMsg()
	Switch $nMsg
		Case $GUI_EVENT_CLOSE
			Exit

	EndSwitch

WEnd

Nun alles eingesetzt. Doch beim Auslesen der Datei bekomme ich die ganze ini - Datei zurückgeworfen. Zudem der Fehler oben immernoch vorhanden.

[.AppleTree.]

habs nur flüchtig gemacht ohne zu testen deshalb der fehler
Das If zeug muss unter die Koda Section, weil zum oberen Zeitpunkt existiert $Label6 noch net^^ und du hast einen "n" zu viel
@ Klavier: das nennt man vergleichen und ansonsten stimmt If zeug eig abgesehen von der Pos

 Spoiler

Originally Posted by klavier22

du kannst die Variablen auch ruhig $OldVersion und $NewVersion nennen versteht jeder besser.

AUßerdem ist das vergleichen nicht korrekt.
er schaut, ob die alte version neuer ist oder die gleiche ist wie die neue und wenn das zutrifft, sagt er, man habe die neueste.
wenn es nicht zutrifft, guckt er, ob die OldVerion höher ist als die neue - was ja garnicht zutreffen kann. Schau dir die Bedingungen der If Schleife nochmal an, die sind definitiv falsch...ich habs noch mal simpel gemacht, darauf solltest du aufbauen:

PHP Code:

#include <INet.au3>

$OldVersion = IniRead("version.ini", "version", "version", "Core not found!")
$NewVersion = _INetGetSource("http://secretgaming.de/trinity/version.ini")

If $OldVersion = "Core not found!" Then Exit MsgBox(4096, "", "Version.ini nicht gefunden!")

If $OldVersion < $NewVersion Then
    MsgBox(4096, "", "Neuere Version vorhanden!")
Elseif $OldVersion = $NewVersion Then
    MsgBox(4096, "", "Sie haben die neueste Version!")
Elseif $OldVersion > $NewVersion Then
    MsgBox(4096, "", "Wahnsinn! Ihre Version ist neuer als die neueste xD!")
EndIf 


Was ansonsten bei dir noch war:
mit $Bla = "asdf" setzt du die Variable $Bla (welche damit auch deklariert wird) auf den Wert asdf.
Wenn du jetzt iwo einfach nur $Bla hinschreibst, macht das ja wenig sinn. Um den Inhalt eines Labels zu ändern, musst du wie jetzt schon werwähnt GUICtrlSetData nehmen.

Der Fehler, der gekommen ist - versuch mal zu überlegen, das der bedeutet könnte. Es heißt ja, dass bei dem ControlSetData ein Fehler ist, evtl ist die Variable mit dem Handle falsch? Hab jetzt selber nicht geguckt^^

Natürlich musst du dein Programm nicht mit MsgBoxen machen, aber dir dein fertiges Programm schreiben mach ich auch nicht. du musst dann natürlich die MSgBoxen ersetzen durch das, was du willst, das es gemacht wird...
Ach ja - und wegen den farben - du sollst ja nicht hier im Forum deinen Code schrieben. Copiere ihn einfach in Scite, bearbeite ihn docrt und kopier ihn dann wieder zurück - ist sehr hilfreich.

Ich finds besser wen zuerst die Sache um die es sich handelt genannt wird weil es auch bei den Funcs so is (es heist ja nicht "ReadFile" sondern "FileRead")

[°"Crazy"°]

Ist ja egal, wie mans nennt. :P Ich mag halt nicht viel schreiben ^^

If $ver_old = $ver_new Then
	GUICtrlSetData ($Label6, $upd_nn)
Elseif $ver_old < $ver_new Then
	GUICtrlSetData ($Label6, $upd_nwd)
	EndIf

Das habe ich nun umgeändert für die richtige Logikfolge. Doch zeigt er mir nichts im Label an, was er eigentlich anzeigen sollte. Muss ich da noch was ändern?

MfG


//Edit
Fehler gefunden, Vorposter hat mir den entscheidenen Tipp gegeben.

[.AppleTree.]

Wath happens?
Screen von der MsgBox pls?
Muss bald off

 Spoiler

#include <ButtonConstants.au3>
#include <ComboConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#Include <string.au3>
#include <INet.au3>

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."
$ver_old = IniRead("version.ini", "version", "version", "Core not found!")

$ver_new = _INetGetSource("http://secretgaming.de/trinity/version.ini")
_StringBetween($ver_new, "version=", @CRLF)

#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Privatserver Updater", 620, 215, 433, 143)
$Combo1 = GUICtrlCreateCombo("TrinityCore", 208, 16, 201, 25)
$Label2 = GUICtrlCreateLabel("Ihre aktuelle Version", 128, 72, 116, 25)
$Label3 = GUICtrlCreateLabel("Verfügbare Version", 376, 72, 116, 25)
$Label4 = GUICtrlCreateLabel($ver_old, 144, 104, 84, 25)
$Label5 = GUICtrlCreateLabel($ver_new, 392, 104, 84, 25)
$Button1 = GUICtrlCreateButton("Update", 256, 176, 107, 33, $WS_GROUP)
$Label6 = GUICtrlCreateLabel("Update2", 168, 144, 276, 25)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###

If $ver_old >= $ver_new Then
	GUICtrlSetData ($Label6, $upd_nn)
Elseif $ver_old < $ver_new Then
	GUICtrlSetData ($Label6, $upd_nwd)
EndIf

MsgBox (32, "Info", @Error&@LF&$VerOld)

While 1
	$nMsg = GUIGetMsg()
	Switch $nMsg
		Case $GUI_EVENT_CLOSE
			Exit

	EndSwitch

WEnd

[.AppleTree.]

mein Fehler
aber das hat noch probleme^^

 Spoiler

Originally Posted by klavier22

@ Creeper, nein, das If Zeug hat nicht gestimmt, da an der Stelle von dem ElseIf die Bedingung nur zutreffen kann, wenn die Bedingung oben auch zutrifft, und damit würde als ElseIf nicht ausgelöst werden. Das Problem lag dort an dem > was ein < sein muss, hier nochmal, zum anschauen:

If $ver_old >= $ver_new Then
	$upd_nn
ElseIf $ver_old > $ver_new Then
	$upd_nwd
EndIf

die ElseIf Bedingung kann nicht stimmen oder sie wird garnicht aufgerufen. Evtl siehst dus ja jetzt...

Weiterhin hab ich mir mal die Mühe gemacht und dir das ganze aus reiner Nettigkeit zusammengeschrieben:

#include <INet.au3>
#include <String.au3>
#include <ButtonConstants.au3>
#include <ComboConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>

$OldVersion = InputBox("Version", "Bitte gib die Version ein") ;IniRead("version.ini", "version", "version", "Core not found!")
$IniNewVersion = _INetGetSource("http://secretgaming.de/trinity/version.ini")
$NewVersion = _StringBetween($IniNewVersion, "version=", @CRLF)

If [COLOR="Red"]$OldVersion[/COLOR] = "Core not found!" Then [COLOR="Red"] ;$OldVersion ist ne Input Box er hatte es 1. mit ini gemacht und 2. ist eine Input box mit GUICtrlRead zu lesen;)[/COLOR]
Exit MsgBox(4096, "", "Version.ini nicht gefunden!")

#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Privatserver Updater", 620, 215, 433, 143)
$Combo1 = GUICtrlCreateCombo("TrinityCore", 208, 16, 201, 25)
$Label2 = GUICtrlCreateLabel("Ihre aktuelle Version", 128, 72, 116, 25)
$Label3 = GUICtrlCreateLabel("Verfügbare Version", 376, 72, 116, 25)
$LOldVersion= GUICtrlCreateLabel("Alte Version", 144, 104, 84, 25)
$LNewVersion = GUICtrlCreateLabel("Neue Version", 392, 104, 84, 25)
$BUpdate = GUICtrlCreateButton("Update", 256, 176, 107, 33, $WS_GROUP)
$LUpdateNeeded = GUICtrlCreateLabel("", 168, 144, 276, 25)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###

GuiCtrlSetData($LOldVersion, $OldVersion)
GuiCtrlSetData($LNewVersion, $NewVersion)

If $OldVersion < $NewVersion Then
	GuiCtrlSetData($LUpdateNeeded, "Es ist eine neuere Version vorhanden!")
Elseif $OldVersion = $NewVersion Then
	GuiCtrlSetData($LUpdateNeeded, "Sie haben die neueste Version!")
Elseif $OldVersion > $NewVersion Then
	GuiCtrlSetData($LUpdateNeeded, "Ihre Version ist neuer als die neueste!")
EndIf


While 1
	$nMsg = GUIGetMsg()
	Switch $nMsg
		Case $GUI_EVENT_CLOSE
			Exit
		Case $BUpdate
			_update()

	EndSwitch

WEnd

Func _update()
	MsgBox(4096, "Fehler", "Diese Funktion steht noch in der Entwicklungsphase!")
EndFunc

[°"Crazy"°]

Danke Dir vielmals. Zweite seite bis eben nicht entdeckt und war selbst noch am Umschreiben einiger Dinge.

Vielen lieben Dank.

[.AppleTree.]

du hast den Fehler nicht gesehen weil ich gefailt hab, habs geeditet
€:Bin off ihr schafft das schon

[Emo!]

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old = $ver_new Then
$upd_nn
EndIf
If $ver_old = $ver_new Then
$upd_nwd
EndIf

nicht

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old >= $ver_new Then
$upd_nn
ElseIf $ver_old > $ver_new Then
$upd_nwd
EndIf

[omer36]

Originally Posted by Emo!

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old = $ver_new Then
$upd_nn
EndIf
If $ver_old = $ver_new Then
$upd_nwd
EndIf

nicht

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old >= $ver_new Then
$upd_nn
ElseIf $ver_old > $ver_new Then
$upd_nwd
EndIf

nichts gegen dich, aber bitte antworte nie auf fragen anderer, wenns um programmieren/scripten geht.

[°"Crazy"°]

Originally Posted by Emo!

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old = $ver_new Then
$upd_nn
EndIf
If $ver_old = $ver_new Then
$upd_nwd
EndIf

nicht

$upd_nn = "Ihre Core ist auf dem aktuellsten Stand."
$upd_nwd = "Bitte updaten Sie Ihre Core."

If $ver_old >= $ver_new Then
$upd_nn
ElseIf $ver_old > $ver_new Then
$upd_nwd
EndIf

Bitte überdenke deinen Fehler. Die Vorposter haben aber meine Frage schon beantwortet.