AHK v2 --- Bugreport

[Holle]

Entweder ist das ein Bug, oder ich verstehe die Variablen nicht mehr:

Code: Select all

vier := "4"
zehn := "10"
vierzehn := vier + zehn
if (vier >= zehn)
	MsgBox vier >= zehn`nvierzehn = %vierzehn%

Das Addieren klappt (vierzehn = 14), aber "vier > 10" ???
Das wird damit zusammenhängen daß in "" stehende Zahlen als String angesehen werden.
Anscheinend wird beim Addieren der "String" als Zahl betrachtet, aber bei >= wird das erste Zeichen des Strings mit dem ersten Zeichen des anderen Strings verglichen.

Das ist nicht gerade die optimale Lösung, denn es ist ja des öfteren nötig mit Zahlen zu arbeiten, welche in einem String enthalten sind, z.B.

Code: Select all

vier := SubStr("Nummer4",7,2)
zehn := SubStr("Nummer10",7,2)
vierzehn := vier + zehn
if (vier >= zehn)
	MsgBox vier >= zehn`nvierzehn = %vierzehn%

...und da funktioniert z.B. >= nicht ...im Gegensatz zum addieren

[jNizM]

Funktioniert bei mir mit 1.1 und v2

Code: Select all

vier := 4
zehn := 10
vierzehn := vier + zehn
if (vier >= zehn)
    MsgBox vier >= zehn`nvierzehn = %vierzehn%
MsgBox % vier "`n" zehn "`n" vierzehn

[Holle]

Ja, aber nicht wenn man "" nutzt.
Sobald es als String angesehen wird geht es nicht mehr (aber das addieren geht trotzdem).
Schau dir mal das untere Beispiel an.


Edit:
Ok, man kann den Fehler umgehen, indem man einfach 0 dazu addiert (dann wird der String zum Integer):

Code: Select all

vier := SubStr("Nummer4",7,2)
zehn := SubStr("Nummer10",7,2)
vier += 0 , zehn += 0   ; ohne diese Zeile bleibt es ein String ist dann ist "vier >= zehn"
vierzehn := vier + zehn
if (vier >= zehn)
	MsgBox vier >= zehn`nvierzehn = %vierzehn%
else
	MsgBox Vier ist < Zehn

[jNizM]

Merkwürdig.. das geht unter v2

Code: Select all

x := 1
y := 2
z := x + y
if (x >= y)
    MsgBox, % x " >= " y " | " z

a := "1"
b := "2"
c := a + b
if (a >= b)
    MsgBox, % a " >= " b " | " c

Das geht nicht:

Code: Select all

x := 4
y := 10
z := x + y
if (x >= y)
    MsgBox, % x " >= " y " | " z

a := "4"
b := "10"
c := a + b
if (a >= b)
    MsgBox, % a " >= " b " | " c

Sobald die Zahl 2-Stellig ist geht es unter v2 nicht

[Holle]

Sobald die Zahl mehrstellig ist UND es ein String ist geht es nicht, weil in dem Fall nicht die Zahl verglichen wird, sondern nur das erste Zeichen (anstatt "4 > 10 = False" wird "4 > 1 = True" verwendet).
Sobald man += 0 benutzt wird aus dem String ein Integer, uns somit wird die Zahl wieder als ganze vergleichen (und nicht nur das erste "Zeichen").

[HotKeyIt]

V2 Help (Greater (>), less (<), greater-or-equal (>=), and less-or-equal (<=))

If either of the inputs is not numeric (or both are strings), they are compared alphabetically. For example, 2 < "10" is true whereas "2" < "10" is false. The comparison is case sensitive only if StringCaseSense has been turned on. See also: Sort

[Holle]

Danke.
Eine Sache verstehe ich aber nicht:

For example, 2 < "10" is true whereas "2" < "10" is false
Die "Zahl" 2 ist kleiner als der "Text" 10 , aber der "Text" 2 ist NICHT kleiner als der "Text" 10 ???
Kann mir das jemand erklären?

[HotKeyIt]

"2"<"10" macht einen Textvergleich und 1 kommt vor 2 daher falsch, ähnlich wie bei "b"<"ab".

[Holle]

Ja, das verstehe ich, aber wenn ich eine Zahl mit einem Buchstabe vergleiche, wie wird da dann vorgegangen?
Ich dachte, sobald ein String vorhanden ist wird nach String verglichen. Also müsste ja auch bei 2<"10" die 2 mit der 1 verglichen werden. Das ist aber anscheinend nicht der Fall.

[just me]

If either of the inputs is not numeric ...

könnte man so übersetzen:

Wenn eine der Eingaben nicht als Zahl dargestellt werden kann ...

Bei If 2 < "10" ist der erste Eingabewert numerisch und der zweite kann in einen numerischen Wert überführt werden. AHK geht wegen des ersten numerischen Eingabewerts davon aus, dass dann auch ein numerischer Vergleich gewünscht ist.

[Holle]

Ja, das macht Sinn.
Danke, nun habe ich das auch kapiert

[Holle]

OK, neuer Bug:

MsgBox % StrLen(12.8) = 9 ??? Sollte die Stringlänge nicht eigentlich 4 sein?


Edit: Ein weiterer Bug:

Code: Select all

text := "Enabled Timers: 1 of 1 (test)"
RegExMatch(text, "Enabled\sTimers\:\s(\d+)\sof\s(\d+)\s\((.*)\)",hit)
MsgBox hit1 = %hit1%`nhit2=%hit2%`nhit3=%hit3%

Im alten AHK ging das noch

Spoiler

---------------------------
AutoHotkey_alt.ahk
---------------------------
hit1 = 1
hit2=1
hit3=test
---------------------------
OK
---------------------------

In v2 klappt das nicht mehr.

Spoiler

---------------------------
temp3.ahk
---------------------------
hit1 =
hit2=
hit3=
---------------------------
OK
---------------------------

[HotKeyIt]

Code: Select all

MsgBox % 12.9 ;12.9 = 12.900000

Code: Select all

MsgBox % StrLen(Round(12.8,1))

[Holle]

Code: Select all

MsgBox % 12.9 ;12.9 = 12.900000

Ja, aber das war in der alten Version nicht der Fall. Warum werden nun zig Nullen angehängt?
Round ist auch keine Lösung, da ich ja vorher nicht weiß wieviele Stellen hinter dem Komma einen Wert haben.

[HotKeyIt]

Dafür gibt es die format funktion von Lexikos.

Code: Select all

MsgBox % StrLen(format("{1:g}",12.9))

[Holle]

Wo finde ich Lexikos Format-Funktion?

[jNizM]

im Lib ordner wenn du v2 runterlädst

[Holle]

Danke.
Dann sollte ich meine Format-Funktion wohl besser umbenennen

Ich habe auch eine kleine Funktion geschrieben, welche die Nullen am Ende entfernt:

Code: Select all

MsgBox % "Original = " 12.9 "`nOhne Nullen = " RemZero(12.9)
ExitApp

RemZero(value) { ; removes zeros (and the point, if no value follow) at the END from float numbers
	; Example: RemZero("12.000") --> 12   ,   RemZero("12.001") --> 12.001
return RegExReplace(value,"(\.0*$|\.$)|(^-?\d+)|0*$","$2")
}

[just me]

Edit: Ein weiterer Bug:

Code: Select all

text := "Enabled Timers: 1 of 1 (test)"
RegExMatch(text, "Enabled\sTimers\:\s(\d+)\sof\s(\d+)\s\((.*)\)",hit)
MsgBox hit1 = %hit1%`nhit2=%hit2%`nhit3=%hit3%

Im alten AHK ging das noch

Spoiler

---------------------------
AutoHotkey_alt.ahk
---------------------------
hit1 = 1
hit2=1
hit3=test
---------------------------
OK
---------------------------

In v2 klappt das nicht mehr.

Spoiler

---------------------------
temp3.ahk
---------------------------
hit1 =
hit2=
hit3=
---------------------------
OK
---------------------------

RegExMatch() arbeitet in v2 im Objektmodus, d.h. die Funktion gibt ein Objekt zurück. RegExMatch().

[Holle]

Vielen Dank, das habe ich übersehen.
So funktioniert das nun auch:

Code: Select all

text := "Enabled Timers: 1 of 1 (test)"
RegExMatch(text, "Enabled\sTimers\:\s(\d+)\sof\s(\d+)\s\((.*)\)",hit)
MsgBox % "hit1 = " hit.1 "`nhit2 = " hit.2 "`nhit3 = " hit.3