If Abfrage

Lausbube

Hallo Leute,

kaum hat das neue Jahr begonnen und ich stehe schon das erste Mal an

Der Reg Eintrag unter \HKEY_LOCAL_MACHINE\SOFTWARE\SIP Company beinhaltet einen von folgenden Werten: K2221, K2222, K22223, K22224, K22225, K22226, K22227, K22228, K22229, K22230
Das "K" kann vor den Zahlen stehen muss aber nicht immer, daher schneide ich es weg.

[autoit]

$company = RegRead("" & $computer & "\HKEY_LOCAL_MACHINE\SOFTWARE\SIP", "Company")
If StringLower(StringLeft($company, 1)) = "k" Then
$company = StringRight($company, 4)
EndIf

[/autoit][autoit][/autoit][autoit]

If $company = "2221" Or "2222" Or "2223" Or "2224" Or "2225" Then
$empfaenger = "A"
GUICtrlSetState($nic_radio1, $GUI_CHECKED)
ElseIf $company = "2226" Or "2227" Or "2228" Or "2229" Then
$empfaenger = "B"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
ElseIf $company = "2230" Then
$empfaenger = "C"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
EndIf

[/autoit]

Der Regwert wird ausgelesen und die Filterfunktion/Schneidefunktion funktioniert auch wie erwartet, jedoch liefert mir das zweite If immer $empfaenger = "A" zurück.
Ganz egal ob ich die Zahlen in Anführungszeichen setze oder nicht, immer wird $empfaenger = "A" zurück geliefert.

Vielen Dank für die Hilfe!

**alpines**

Die Abfrage is falsch da Zahlen immer true sind.
Statt If $sVar = 1 or 3
muss da If $sVar = 1 or $sVar = 3 hin.
Alternativ tuts ein Switch auch.

NightmarePhoenix

Ich kenne mich jetzt nicht so gut mit RegRead aus, insofern kann ich nicht sicher sagen ob das so funktioniert...
Aber wenn man eine GUICtrlCreateInput hat muss man den Wert immer erst auslesen, bevor man ihn weiterverarbeiten kann.
Dazu kann man bei GUICtrlCreateInput einfach GUICtrlRead verwenden. Evtl musst du das auch machen.

Als Beispiel:

Du hast $company das du zum Auslesen der Registry nimmst.
Jetzt kann If aber noch nichts mit $company Anfangen.
Also musst du mit GUICtrlRead den Wert auslesen den du in $company hast.
Jetzt kannst du mit dem Befehl,
$comp = GUICtrlRead ($company)
den wert auslesen, so das es beim If-Befehl verwendet werden kann.

Wie gesagt, bei GUICtrlCreateInput funktioniert es so, wie es bei RegRead aussieht weiß ich nicht. Ansonsten könntest du mal versuchen die Werte erstmal in ein *.txt-File zu speichern um zu sehen ob die Werte richtig ausgelesen werden und dann weiterschauen.

Lausbube

Zitat von alpines

Die Abfrage is falsch da Zahlen immer true sind.
Statt If $sVar = 1 or 3
muss da If $sVar = 1 or $sVar = 3 hin.
Alternativ tuts ein Switch auch.

*Kopf gegen Tisch knall*
Besten Dank, das war es auch schon!

**alpines**

Kannst dir auch überlegen Switch oder Select zu nutzen, der Übersichtlichkeit halber

[autoit]

$iCompany = 2230

[/autoit][autoit][/autoit][autoit]

Switch $iCompany
Case 2221, 2222, 2223, 2224, 2225
MsgBox(64, "Empfänger", "A")
Case 2226, 2227, 2228, 2229
MsgBox(64, "Empfänger", "B")
Case 2230
MsgBox(64, "Empfänger", "C")
EndSwitch

[/autoit][autoit][/autoit][autoit]

Select
Case $iCompany < 2226
MsgBox(64, "Empfänger", "A")
Case $iCompany > 2225 and $iCompany < 2230
MsgBox(64, "Empfänger", "B")
Case $iCompany = 2230
MsgBox(64, "Empfänger", "C")
EndSelect

[/autoit]

meistertogo

Ebenso funktionieren müsste:

[autoit]

$iCompany = 2230

[/autoit][autoit][/autoit][autoit]

Switch $iCompany
Case 2221 To 2225
MsgBox(64, "Empfänger", "A")
Case 2226 To 2229
MsgBox(64, "Empfänger", "B")
Case 2230
MsgBox(64, "Empfänger", "C")
EndSwitch

[/autoit]

Außerdem könnte man "A", "B" und "C" auch durch eine Variable ersetzen, sodass nur ein allgemeiner(er) Aufruf der MsgBox-Funktion nötig wäre.
Ist aber vermutlich alles Geschmackssache.

Rodny_le_lude

hier hast du mal zwei Varianten:

Spoiler anzeigen

[autoit]

$company = RegRead("" & $computer & "\HKEY_LOCAL_MACHINE\SOFTWARE\SIP", "Company")
If StringLower(StringLeft($company, 1)) = "k" Then
$company = StringRight($company, 4)
EndIf

[/autoit] [autoit][/autoit] [autoit]

;Variante 1####################################
Switch $company
Case 2221 To 2225
$empfaenger = "A"
GUICtrlSetState($nic_radio1, $GUI_CHECKED)
Case 2226 To 2229
$empfaenger = "B"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
Case 2230
$empfaenger = "C"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
Case Else
MsgBox(64,"Error","Error")
EndSwitch

[/autoit] [autoit][/autoit] [autoit]

;Variante 2####################################
If $company >= "2221" And $company <="2225" Then
$empfaenger = "A"
GUICtrlSetState($nic_radio1, $GUI_CHECKED)
ElseIf $company >= "2226" And $company<="2229" Then
$empfaenger = "B"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
ElseIf $company = "2230" Then
$empfaenger = "C"
GUICtrlSetState($nic_radio3, $GUI_CHECKED)
EndIf

[/autoit]