Problem beim Vergleich zweier Variabeln

10/21/2011 21:48 �"Crazy"�#1

Gr��e!

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

Code:

$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

10/21/2011 21:58 .AppleTree.#2

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

10/21/2011 21:59 �"Crazy"�#3

Code:

#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. ;)

10/21/2011 22:09 klavier22#4

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

10/21/2011 22:10 .AppleTree.#5

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

Code:

#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^^

10/21/2011 22:24 �"Crazy"�#6

@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:

Code:

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:

Code:

#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. ;)

10/21/2011 22:39 .AppleTree.#7

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

Quote:

Originally Posted by klavier22

du kannst die Variablen auch ruhig $OldVersion und $NewVersion nennen :D 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")

10/21/2011 22:41 �"Crazy"�#8

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

Code:

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. :)

10/21/2011 22:50 .AppleTree.#9

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

Spoiler

10/21/2011 22:56 .AppleTree.#10

mein Fehler;)
aber das hat noch probleme^^

Spoiler

Quote:

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:

Code:

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:

Code:

#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

10/21/2011 23:03 �"Crazy"�#11

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

Vielen lieben Dank. :)

10/21/2011 23:04 .AppleTree.#12

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

10/22/2011 14:47 Emo!#13

$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

10/22/2011 15:10 omer36#14

Quote:

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.

10/22/2011 15:21 �"Crazy"�#15

Quote:

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. :)