C# Keybind for Autoclicker

[Aaudiio]

Hey. I'm looking for help with my autoclicker. I got it working with buttons, but I need keybinds instead.

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace auto_click
{

public partial class Form1 : Form
{

    
    \[System.Runtime.InteropServices.DllImport("user32.dll")\]
    public static extern void mouse\_event(int dwFlags, int dx, int dy, int cButtons, int dwExtraInfo);

    public const int MOUSEEVENTF\_LEFTDOWN = 0x02;
    public const int MOUSEEVENTF\_LEFTUP = 0x04;
    public const int MOUSEEVENTF\_RIGHTDOWN = 0x08;
    public const int MOUSEEVENTF\_RIGHTUP = 0x10;



    
    public void MouseClick()
    {
        mouse\_event(MOUSEEVENTF\_LEFTDOWN, 0, 0, 0, 0);
        mouse\_event(MOUSEEVENTF\_LEFTUP, 0, 0, 0, 0);
    }
    public Form1()
    {
        InitializeComponent();
    }


    private void timer1\_Tick(object sender, EventArgs e)
    {
        MouseClick();
    }

    private void Form1\_KeyDown(object sender, KeyEventArgs e)
    {
        if (e.KeyCode.ToString() == "F") ;
        timer1.Enabled = true;
    }
}

}

Any idea why it doesn't work?

REEEEEEEEEEEEH

[Zain Ul Abidin]

first confirm whether your timer is starting or not some times timer itself does not starts and people start messing with the code inside timer tick event handler

[OmegaExtern]

First off all, you are dealing with deprecated method (mouse_event). Second, I think you will need KeyPreview enabled/set to true for your form (look on Form1 properties). Third, "if (e.KeyCode.ToString() == "F") ;" wtf? That "if" branch statement does nothing, why? You assigned it to empty (;) and by not using "{" and "}" it basically mean "if" is limited to single line.. code style.. Fourth, your if statement comparison should be made from "e.KeyCode.ToString() == "F" to "e.KeyCode == Keys.F". Fifth, are you sure your events (timer1_Tick/Form1_KeyDown) are assigned, that could be a main cause, "this code does nothing", as I don't see anything in your constructor, "InitializeComponent();", and you didn't posted code of your (*.Designer.cs) Form1.Designer.cs file in this case. Sixth, it is a lot of bad practice in here so I will jump to seventh. Seventh, you can't just copy/paste into C# always double-check everything such as Win32 native API you deal with, it maybe have wrong signature from my experience there are a lot of examples also 90% of all Windows native APIs from pinvoke.net are NOT correctly defined, so sad... I have fixed your code, and it works fine for me, all you need to do is keep up with my friendly warnings. If you are not that smart, well, here is my smart suggestion for you, why not do: "timer1.Enabled = Keys.F;" in Form1_KeyDown event (Keys.F will be 1/true when hold down; otherwise 0/false). And yeah, do not forget mouse_event() is deprecated method and is no longer officially supported (so I should not be even giving you away these nice warnings). Switch to SendInput function with structures, it is not that hard. Best regards, OmegaExtern.

[Aaudiio]

Thanks a lot. As you can tell, I know almost nothing. I'm just trying to pick up on the basics by trying to make things.

[OmegaExtern]

Re-Edited my post, refresh your page. I glad I can help you :)