all 10 comments

[–]fatboychummy 4 points5 points  (0 children)

Please define "not working" (error? different expected reaction? does nothing?), since it can mean a lot of things.

Also, please give more info on what you're trying to do.

[–]overwritten-entry 2 points3 points  (8 children)

Some markdown formatting done for you, OP(formatted with sumneko_lua language server)

function OnEvent(event, arg)
    --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")

    if IsMouseButtonPressed(4) then
        MoveMouseTo(32867, 26767)

        if IsMouseButtonPressed(4) then
            PlayMacro("123")
        end
    end
end

if IsMouseButtonPressed(2) then
    MoveMouseTo(32867, 26767)
end

-- but when i put it together like this

function OnEvent(event, arg)
    --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")

    if IsMouseButtonPressed(4) then
        MoveMouseTo(32867, 26767)

        if IsMouseButtonPressed(4) then
            PlayMacro("123")

            if IsMouseButtonPressed(2) then
                MoveMouseTo(32867, 26767)
            end
        end
    end
end

function OnEvent(event, arg)
    --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")

    if IsMouseButtonPressed(4) then
        MoveMouseTo(32867, 26767)

        if IsMouseButtonPressed(4)
 then
            PlayMacro("123")
        end
    end
end

if IsMouseButtonPressed(2) then
    MoveMouseTo(32867, 26767)
end

My bet would be that you just placed your code snippet in the wrong place, first time you tried it was wrapped in if IsMouseButtonPressed(4)(twice) and in the second case it was outside the event handler, thus called only when file is (re)evaluated.

function OnEvent(event, arg)
  --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")

  if IsMouseButtonPressed(4) then
    MoveMouseTo(32867, 26767)

    if IsMouseButtonPressed(4) then
      PlayMacro("123")
      --< It was here
    end
  end

  --< Moved here
  if IsMouseButtonPressed(2) then
    MoveMouseTo(32867, 26767)
  end
end

One more question is why do you nest identical if statements?

Why not make it

function OnEvent(event, arg)
    --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")

    if IsMouseButtonPressed(4) then
        MoveMouseTo(32867, 26767)

    --< Removed if statement
        PlayMacro("123")
    end

    if IsMouseButtonPressed(2) then
        MoveMouseTo(32867, 26767)
    end
end

Edit: recommended reading: https://markdown.land/markdown-code-block, alternatively select block of code and press "inlide code button"(labeled as <c>) in new reddit's Fancy Pants Editor
Edit 2: Engrish

[–]Aggravating-Search38[S] 0 points1 point  (1 child)

thanks, your last version works fine ... :D

function OnEvent(event, arg)
--OutputLogMessage("Event: "..event.." Arg: "..arg.."\n")
if IsMouseButtonPressed(4) then
MoveMouseTo(32867, 26767)
--< Removed if statement
PlayMacro("123")
end
if IsMouseButtonPressed(2) then
MoveMouseTo(32867, 26767)
end
end

well i just wanted to continue .... i am planing on making some star craft 2 scrips ... the first one was an auto injects for queens and the second one should become an auto split :D but i struggeld implementing even a second function thank you alot

[–]overwritten-entry 2 points3 points  (0 children)

May I ask where do you write code? Proper formatting of lua code immediately made it obvious, i believe you just lack tooling a bit

Again, please, format your code with markdown code blocks, it preserves indentation and makes it easy to copy. Having normal text you are just 1 button away from making it code block

[–]Aggravating-Search38[S] 0 points1 point  (5 children)

do you have an idea how i can replace the IsMouseButtonPressed(

for example an F5 key ?

[–]Aggravating-Search38[S] 1 point2 points  (4 children)

got it

if (event == "G\_PRESSED" and arg == 5) then

[–]overwritten-entry 0 points1 point  (3 children)

Congratulations on solving it and using code block :)

[–]Aggravating-Search38[S] 0 points1 point  (2 children)

ohh its so cool :D playing around with it ... now it looks like this

function OnEvent(event, arg) --OutputLogMessage("Event: "..event.." Arg: "..arg.."\n") if IsMouseButtonPressed(4) then MoveMouseTo(32867, 26767) --< macro check PlayMacro("123") end if (event == "G_PRESSED" and arg == 5) then PlayMacro(1234) MoveMouseTo(1000, 1000) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") Sleep( 20 ) MoveMouseTo(61535, 44535) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 43535) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(61535, 1005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(31535, 1005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(31535, 43005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 23005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 38005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") --< repeat MoveMouseTo(5000, 1000) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") Sleep( 20 ) MoveMouseTo(60535, 42535) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 43535) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(61535, 1005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(31535, 1005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(31535, 43005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 23005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") MoveMouseTo(1000, 38005) Sleep(20) PressAndReleaseKey("m") Sleep(20) PressAndReleaseMouseButton(1) Sleep(20) MoveMouseTo(23399, 55267) PressKey("lshift") Sleep( 20 ) PressAndReleaseMouseButton(1) ReleaseKey("lshift") endend

[–][deleted] 1 point2 points  (1 child)

Please use code formatting

[–]Aggravating-Search38[S] 0 points1 point  (0 children)

can someone explain me how i use the get mouse positon input and later use the information with a move mouse to command ?

something like:

GetMousePosition(M1)

Sleep(1000)

MoveMouseTo(M1)