[SOLVED][1.18.2] Implementing player left clicks and right clicks

[FantaLaTone]

Hello, 

I have a simple Minecraft mod. I want to add a system that can track my mouse clicks and run commands when I left click or right click.

I tried some solutions but I couldn't find any good resource about it. I tried some methods but no help at all. Can someone just tell which event I need to subscribe?

Edited July 20, 2022 by FantaLaTone

[Luis_ST]

  On 7/20/2022 at 3:14 PM, FantaLaTone said:

Can someone just tell which event I need to subscribe?

Expand  

InputEvent.RawMouseEvent, depends on what the command should do you need to send a custom network packet
to the server which performs there the action of the command

 

Edited July 20, 2022 by Luis_ST
Correcting

[FantaLaTone]

  On 7/20/2022 at 3:29 PM, Luis_ST said:

InputEvent.MouseButton.Pre, depends on what the command should do you need to send a custom network packet
to the server which performs there the action of the command

 

Expand  

i meant methods by saying "commands". like just print something

i tried this code but it doesnt seem to be working

@SubscribeEvent
    public void a(InputEvent.MouseButton.Pre event) {
        System.out.println(event);
    }

 

[Luis_ST]

  On 7/20/2022 at 3:39 PM, FantaLaTone said:

i meant methods by saying "commands". like just print something

i tried this code but it doesnt seem to be working

Expand  

please post the full class

Edited July 20, 2022 by Luis_ST

[FantaLaTone]

  On 7/20/2022 at 3:49 PM, Luis_ST said:

please post the full class

Expand  

package com.fantalatone.soccermod.event;

import com.fantalatone.soccermod.SoccerMod;
import net.minecraftforge.api.distmarker.Dist;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;

import java.awt.event.InputEvent;

@Mod.EventBusSubscriber(modid = SoccerMod.MOD_ID, bus = Mod.EventBusSubscriber.Bus.FORGE, value = Dist.CLIENT)
public class SoccerModEvents {

    @SubscribeEvent
    public void a(InputEvent.MouseButton.Pre event) {
        System.out.println(event);
    }

}

 

[warjort]

Does that even compile?

You are using the AWT event when you should be using the forge event.

[Luis_ST]

take a look at this image then try again

[Luis_ST]

  On 7/20/2022 at 3:29 PM, Luis_ST said:

InputEvent.MouseButton.Pre

Expand  

Correction, this is for 1.19 in 1.18 you need to use InputEvent.RawMouseEvent

[FantaLaTone]

  On 7/20/2022 at 4:03 PM, Luis_ST said:

Correction, this is for 1.19 in 1.18 you need to use InputEvent.RawMouseEvent

Expand  

ok, i got it working with this code but i have simple question how i can test if it is in world not gui?

@SubscribeEvent
    public static void a(InputEvent.RawMouseEvent event) {
        System.out.println("3131313!");

        if (event.getButton() == 0) {
            System.out.println("SOl CLİK!");
        }
    }

 

[warjort]

Ironically, you can do the opposite with the ScreenEvent.MouseClickedEvent.Pre

The only way I can think of doing this to use Minecraft.getScreen() and test if it is null (i.e. no gui screen)?

[FantaLaTone]

  On 7/20/2022 at 6:01 PM, warjort said:

Ironically, you can do the opposite with the ScreenEvent.MouseClickedEvent.Pre

The only way I can think of doing this to use Minecraft.getScreen() and test if it is null (i.e. no gui screen)?

Expand  

this only detects when im on gui it is not detecting my normal clicks 

@SubscribeEvent
    public static void a(ScreenEvent.MouseClickedEvent event) {
        if (event.getScreen() == null) {
            System.out.println("boıiş!");
        }
        if (event.getButton() == 0) {
            System.out.println("SOl CLİK!");
        }
    }

(i tried ScreenEvent.MouseClickedEvent.Pre too same thing)

[warjort]

I didn't say use ScreenEvent, that is only for guis.

You should use the RawMouseEvent and add an additional check in your handler.

public static void a(InputEvent.RawMouseEvent event) {
    if (Minecraft.getInstance().getScreen() != null) return; // GUI screen

    // Your code here
}

If that doesn't work, I don't know how to do it.

[FantaLaTone]

  On 7/20/2022 at 6:21 PM, warjort said:

I didn't say use ScreenEvent, that is only for guis.

You should use the RawMouseEvent and add an additional check in your handler.

public static void a(InputEvent.RawMouseEvent event) {
    if (Minecraft.getInstance().getScreen() != null) return; // GUI screen

    // Your code here
}

If that doesn't work, I don't know how to do it.

Expand  

theres no command like .getScreen() i just used .screen instead.

latest code for 1.18.2 forge click detection

package com.fantalatone.soccermod.event;

import com.fantalatone.soccermod.SoccerMod;
import net.minecraft.client.Minecraft;
import net.minecraftforge.api.distmarker.Dist;
import net.minecraftforge.client.event.InputEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;

@Mod.EventBusSubscriber(modid = SoccerMod.MOD_ID, bus = Mod.EventBusSubscriber.Bus.FORGE, value = Dist.CLIENT)
public class SoccerModEvents {

    @SubscribeEvent
    public static void a(InputEvent.RawMouseEvent event) {
        if (Minecraft.getInstance().screen != null) return;
        if (event.getButton() == 0) {
            System.out.println("Left Click Detected!");
        }
    }

}

 

[FantaLaTone]

@Luis_ST@warjort

i am sorry for pinging but can you tell me can i get the player with this click event? how i can get the player?

[FantaLaTone]

  On 7/20/2022 at 7:08 PM, diesieben07 said:

RawMouseEvent is client side only, there is only one player.

Expand  

i know, i am asking for how i can get player's x and other things like 

LivingEntity p = event.clicker;

[FantaLaTone]

  On 7/20/2022 at 7:21 PM, diesieben07 said:

Minecraft#player is the client side player

Expand  

ok, thanks it worked