C#•10mo ago

SDL2 Joystick issues

Hello. I need help regarding my post from StackOverflow. https://stackoverflow.com/questions/78100724/inconsitent-detection-of-joystick-button-input-using-sdl-in-c-sharp-with-the-use Summary: Code is not working properly when 2 or more joysticks are connected. Only one joystick connected and it works well.

Stack Overflow

Inconsitent detection of joystick button input using SDL in C# with...

So I made a code that will detect joystick button input in a TPL where all joysticks' inputs are detected simultaneously. Good thing is that if there's only one joystick connected, it works consist...

174 Replies

TheRanger•10mo ago

This part should be in the main method

          while (!quit) // beginning of a frame
            {
                SDL.SDL_Event e;
                while (SDL.SDL_PollEvent(out e) != 0)
                {
                    foreach(var joystick in joysticks)
                    {
                           joystick.Handle(e);
                    }
                }
              // end of a frame
            }

          while (!quit) // beginning of a frame
            {
                SDL.SDL_Event e;
                while (SDL.SDL_PollEvent(out e) != 0)
                {
                    foreach(var joystick in joysticks)
                    {
                           joystick.Handle(e);
                    }
                }
              // end of a frame
            }

the first while loop is to keep the game running

jcaishereOP•10mo ago

main method?

TheRanger•10mo ago

on every frame, you handle ur input, a character's physics, rendering, etc... yeah the static void Main(string[] args) one for now your game is a console game, it only handles input, the while (SDL.SDL_PollEvent(out e) != 0) loops on every key just got pressed/unpressed

jcaishereOP•10mo ago

how about the product ID? because i do need it so that i can make sure that it's actually following the right joystick

TheRanger•10mo ago

you could say it acts like

foreach(var e in sdl_keyevents)
{
}

foreach(var e in sdl_keyevents)
{
}

but SDL was made in C, C has no foreach afaik do you get what i mean? now lets say you pressed X on one of the 8 joysticks the while loop here will only loop once, and the e has the data of the key pressed are you following?

jcaishereOP•10mo ago

still a bit confused because i used to have everything thrown at me and think it'd work

TheRanger•10mo ago

that's your problem, you should exactly understand everything ur writing dont assume it will automatically work

jcaishereOP•10mo ago

using System;
using SDL2;
using System.Threading.Tasks;
using SDLCSharp;
using System.Collections.Concurrent;
using static SDL2.SDL;

class Program
{
    static void Main(string[] args)
    {
        SDL.SDL_Init(SDL.SDL_INIT_JOYSTICK);
        Dictionary<int, Tuple<int, int>> joystickInfo = new Dictionary<int, Tuple<int, int>>();

        int numJoysticks = SDL.SDL_NumJoysticks();
        for (int i = 0; i < numJoysticks; ++i)
        {
            IntPtr joystick = SDL.SDL_JoystickOpen(i);
            if (joystick != IntPtr.Zero)
            {
                int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
                int instanceID = SDL.SDL_JoystickInstanceID(joystick);

                joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);

                SDL.SDL_JoystickClose(joystick);
            }
        }

        Console.WriteLine("Connected Joysticks:");
        foreach (var entry in joystickInfo)
        {
            int productID = entry.Key;
            int joystickIndex = entry.Value.Item1;
            int instanceID = entry.Value.Item2;
            Console.WriteLine($"Product ID: {productID}, Joystick Index: {joystickIndex}, Instance ID: {instanceID}");
        }


    }

}

using System;
using SDL2;
using System.Threading.Tasks;
using SDLCSharp;
using System.Collections.Concurrent;
using static SDL2.SDL;

class Program
{
    static void Main(string[] args)
    {
        SDL.SDL_Init(SDL.SDL_INIT_JOYSTICK);
        Dictionary<int, Tuple<int, int>> joystickInfo = new Dictionary<int, Tuple<int, int>>();

        int numJoysticks = SDL.SDL_NumJoysticks();
        for (int i = 0; i < numJoysticks; ++i)
        {
            IntPtr joystick = SDL.SDL_JoystickOpen(i);
            if (joystick != IntPtr.Zero)
            {
                int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
                int instanceID = SDL.SDL_JoystickInstanceID(joystick);

                joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);

                SDL.SDL_JoystickClose(joystick);
            }
        }

        Console.WriteLine("Connected Joysticks:");
        foreach (var entry in joystickInfo)
        {
            int productID = entry.Key;
            int joystickIndex = entry.Value.Item1;
            int instanceID = entry.Value.Item2;
            Console.WriteLine($"Product ID: {productID}, Joystick Index: {joystickIndex}, Instance ID: {instanceID}");
        }


    }

}

let's say this is my main for now and then i'll have to add the things from the other file that has the namespace SDLCSharp

TheRanger•10mo ago

wait why are you closing the joysticks?

jcaishereOP•10mo ago

this is pretty much just displaying the joysticks that i have that are connected

TheRanger•10mo ago

you haven't answered my question u are opening the joystick on every frame, you dont need that once is enough

jcaishereOP•10mo ago

well, im opening multiple joysticks

TheRanger•10mo ago

yeah thats good, then you close them I never used GetProduct tho

jcaishereOP•10mo ago

yeah that's the product ID of the joystick it's important for my project

TheRanger•10mo ago

the IntPtr from JoystickOpen is enough for me anyway

jcaishereOP•10mo ago

i remove the closing of joysticks, okay... what's next, adding the while loop for checking the state of joysticks buttons

TheRanger•10mo ago

no need to use dictionary, just use List<Joystick>

jcaishereOP•10mo ago

I'll just keep it that way

TheRanger•10mo ago

if ull keep it that way how are you gonna access their instances? you have a class called Joystick1, why even store the info of joystick in a dictionary if you can just store it in a list of Joystick1

jcaishereOP•10mo ago

List<int> productIDs = new List<int> { 12345, 12346, 12347 };
Dictionary<int, Tuple<int, int>> joystickInfo = new Dictionary<int, Tuple<int, int>>();

int numJoysticks = SDL.SDL_NumJoysticks();
for (int i = 0; i < numJoysticks; ++i)
{
    IntPtr joystick = SDL.SDL_JoystickOpen(i);
    if (joystick != IntPtr.Zero)
    {
        int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
        int instanceID = SDL.SDL_JoystickInstanceID(joystick);

        if (productIDs.Contains(getProductIDs))
        {
            joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);
        }

    }
}

Console.WriteLine("Connected Joysticks:");
foreach (var entry in joystickInfo)
{
    int productID = entry.Key;
    int joystickIndex = entry.Value.Item1;
    int instanceID = entry.Value.Item2;
    Console.WriteLine($"Product ID: {productID}, Joystick Index: {joystickIndex}, Instance ID: {instanceID}");
}

List<int> productIDs = new List<int> { 12345, 12346, 12347 };
Dictionary<int, Tuple<int, int>> joystickInfo = new Dictionary<int, Tuple<int, int>>();

int numJoysticks = SDL.SDL_NumJoysticks();
for (int i = 0; i < numJoysticks; ++i)
{
    IntPtr joystick = SDL.SDL_JoystickOpen(i);
    if (joystick != IntPtr.Zero)
    {
        int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
        int instanceID = SDL.SDL_JoystickInstanceID(joystick);

        if (productIDs.Contains(getProductIDs))
        {
            joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);
        }

    }
}

Console.WriteLine("Connected Joysticks:");
foreach (var entry in joystickInfo)
{
    int productID = entry.Key;
    int joystickIndex = entry.Value.Item1;
    int instanceID = entry.Value.Item2;
    Console.WriteLine($"Product ID: {productID}, Joystick Index: {joystickIndex}, Instance ID: {instanceID}");
}

this is my original code.

TheRanger•10mo ago

List<int> productIDs = new List<int> { 12345, 12346, 12347 }; arent the joystick product ids 0, 1, 2 and so on?

jcaishereOP•10mo ago

no, they're number indices 12345, 12346, 12347 are just for display the product IDs that i have are for, not public

TheRanger•10mo ago

yeah but do u know what this code does?

   if (productIDs.Contains(getProductIDs))
        {
            joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);
        }

   if (productIDs.Contains(getProductIDs))
        {
            joystickInfo[getProductIDs] = Tuple.Create(i, instanceID);
        }

jcaishereOP•10mo ago

yeah, if the productID of a joystick matches the one in the productIDs, it will be added to the Dictionary Otherwise, it won't be added

TheRanger•10mo ago

so wait, u know the productId from the start? alright fair enough ig anyway please delete this Dictionary<int, Tuple<int, int>> joystickInfo = new Dictionary<int, Tuple<int, int>>(); classes are a thing, store the data of each joystick in each instance of Joystick1

List<int> productIDs = new List<int> { 12345, 12346, 12347 };
List<Joystick> joysticks = new List<Joystick>();

int numJoysticks = SDL.SDL_NumJoysticks();
for (int i = 0; i < numJoysticks; ++i)
{
    IntPtr joystick = SDL.SDL_JoystickOpen(i);
    if (joystick != IntPtr.Zero)
    {
        int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
        int instanceID = SDL.SDL_JoystickInstanceID(joystick);

        if (productIDs.Contains(getProductIDs))
        {
            joysticks.Add(new Joystick(i, getProductIDs, instanceId));
        }

    }
}

List<int> productIDs = new List<int> { 12345, 12346, 12347 };
List<Joystick> joysticks = new List<Joystick>();

int numJoysticks = SDL.SDL_NumJoysticks();
for (int i = 0; i < numJoysticks; ++i)
{
    IntPtr joystick = SDL.SDL_JoystickOpen(i);
    if (joystick != IntPtr.Zero)
    {
        int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);
        int instanceID = SDL.SDL_JoystickInstanceID(joystick);

        if (productIDs.Contains(getProductIDs))
        {
            joysticks.Add(new Joystick(i, getProductIDs, instanceId));
        }

    }
}

let the class's constructor take the values and store them into its fields There is a better way and cleaner way than using Dictionary in your case also you dont need to make many classes for your joysticks

jcaishereOP•10mo ago

okay

TheRanger•10mo ago

because you can make many instances out of a single class

jcaishereOP•10mo ago

so i made a class Joystick that's good, what's next...

TheRanger•10mo ago

show me ur class

jcaishereOP•10mo ago

internal class Joystick
{
    private int i;
    private int getProductIDs;

    public Joystick(int i, int getProductIDs)
    {
        this.i = i;
        this.getProductIDs = getProductIDs;
    }
}

internal class Joystick
{
    private int i;
    private int getProductIDs;

    public Joystick(int i, int getProductIDs)
    {
        this.i = i;
        this.getProductIDs = getProductIDs;
    }
}

TheRanger•10mo ago

great but u could change private int getProductIDs; to private int _productId;

jcaishereOP•10mo ago

alright

TheRanger•10mo ago

in C# its recommended to name private fields like that to begin with an underscore

jcaishereOP•10mo ago

i see

TheRanger•10mo ago

ok now you can put the Handle method

jcaishereOP•10mo ago

in the Joystick class, right?

TheRanger•10mo ago

yeah

jcaishereOP•10mo ago

Well, this happened

TheRanger•10mo ago

ops

public void Handle(SDL_Event e)
{
   switch (e.type)
  {
       case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
           bool pressed = SDL.SDL_JoystickGetButton(_yourJoystickId,  e.jbutton.button);
          break;
  }
}

public void Handle(SDL_Event e)
{
   switch (e.type)
  {
       case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
           bool pressed = SDL.SDL_JoystickGetButton(_yourJoystickId,  e.jbutton.button);
          break;
  }
}

jcaishereOP•10mo ago

TheRanger•10mo ago

you missed something break; what does SDL_JoystickGetButton return? int?

jcaishereOP•10mo ago

nint on joystick and int on button

TheRanger•10mo ago

then you know what to do use int not bool if its 0 it indicates the button index isnt pressed, 1 if it is

jcaishereOP•10mo ago

now ill go back to main and try your while loop

TheRanger•10mo ago

good but do u understand this error?

jcaishereOP•10mo ago

i did

TheRanger•10mo ago

jcaishereOP•10mo ago

i fixed it

TheRanger•10mo ago

ok now but GetButton doesnt take a product id

jcaishereOP•10mo ago

yeah it takes i instead

TheRanger•10mo ago

jcaishereOP•10mo ago

really?

TheRanger•10mo ago

yeah it needs the value that JoyStickOpen returns for the first parameter

jcaishereOP•10mo ago

which is i

TheRanger•10mo ago

jcaishereOP•10mo ago

unless i misunderstood

TheRanger•10mo ago

https://wiki.libsdl.org/SDL2/SDL_JoystickGetButton

SDL2/SDL_JoystickGetButton

The Simple Directmedia Layer Wiki

TheRanger•10mo ago

it requires an instance of SDL_Joystick which SDL_JoystickOpen returns but its IntPtr in your case because that's how C# see's every pointer from C/C++ like that

jcaishereOP•10mo ago

so i have to change it ?

TheRanger•10mo ago

ofcourse it wants the value that SDL_JoystickOpen returns

jcaishereOP•10mo ago

well, im trying to find what can be a replacement for IntPtr now

TheRanger•10mo ago

why would you? nint is alias for IntPtr btw so, same thing

jcaishereOP•10mo ago

oh yeah i came from C++ but i saw C# fitting better for my project so i got confused anyways, now i have to get joystick in as well for Joystick too, right?

TheRanger•10mo ago

no you need to figure out how to pass the value of IntPtr joystick into your class because you need that value

jcaishereOP•10mo ago

joysticks.Add(new Joystick(i, getProductIDs, joystick));

joysticks.Add(new Joystick(i, getProductIDs, joystick));

does this not work?

TheRanger•10mo ago

it should, if you have the correct parameters in your constructor

jcaishereOP•10mo ago

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

how's this?

TheRanger•10mo ago

looks perfect

jcaishereOP•10mo ago

alright, now onto complex part

TheRanger•10mo ago

yeah i would put Console.WriteLine under int pressed to print the info of the joystick and the pressed button you have a big problem tho in ur main method

jcaishereOP•10mo ago

hm i think i need to get like, specific buttons

TheRanger•10mo ago

jcaishereOP•10mo ago

like, 0, 1, 2, 3, etc.

TheRanger•10mo ago

not now well, did you test? what did you find so far?

jcaishereOP•10mo ago

i did test i have 2 joysticks it seems to only detect the first one instead of both

TheRanger•10mo ago

i dont think the 2nd joystick was detected yes because ur while(!quit`) is in the for loop which means i will never be 1 or more the for loop will always be stuck in the first loop

jcaishereOP•10mo ago

mhm

TheRanger•10mo ago

the while loop should be outside of the for loop

jcaishereOP•10mo ago

i removed it and it does work

TheRanger•10mo ago

cool

jcaishereOP•10mo ago

now onto the more complex part i need to know which is which now like, i have different names from these joysticks that i have and the buttons too i need to know which pressed button came from any of my joysticks

TheRanger•10mo ago

u have i, and productid of each joystick u could differentiate with their values

jcaishereOP•10mo ago

should i also add e.jbutton.which too?

TheRanger•10mo ago

i never heard of that

jcaishereOP•10mo ago

because it does display both, but it feels like the other button is getting triggered because this joystick of mine has 2 buttons

TheRanger•10mo ago

SDL does not have which that in jbutton did u mean e.jbutton.button ?

jcaishereOP•10mo ago

hm let's say i have this joystick that has 2 buttons and it's on and off from a single switch. up is on, down is off

TheRanger•10mo ago

what do u mean?

jcaishereOP•10mo ago

and i have another joystick that has 3 buttons from a single switch, if it's at 1, 1 is on, 2 and 3 is off. wait

TheRanger•10mo ago

realize that

jcaishereOP•10mo ago

so im pushing buttons from 8 but once i do that, 9 does trigger too

TheRanger•10mo ago

a button event is only called once when u press, it doesnt constantly call it while ur holding a button what are u censoring 😅

jcaishereOP•10mo ago

actual product ID of my joysticks gotta hide em

TheRanger•10mo ago

button 8, button 9?

jcaishereOP•10mo ago

TheRanger•10mo ago

jcaishereOP•10mo ago

joystick 8 and joystick 9

TheRanger•10mo ago

:bigthonk: show ur joystick class and ur main

jcaishereOP•10mo ago

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                Console.WriteLine($"Joystick {_productId} is pressed!");
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                Console.WriteLine($"Joystick {_productId} is pressed!");
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

TheRanger•10mo ago

show ur main too oh wait like i said

jcaishereOP•10mo ago

using System;
using SDL2;
using static SDL2.SDL;

class Program
{
    static void Main(string[] args)
    {
        SDL.SDL_Init(SDL.SDL_INIT_JOYSTICK);

        List<int> productIDs = new List<int> { /*hidden for reasons*/ };
        List<Joystick> joysticks = new List<Joystick>();

        int numJoysticks = SDL.SDL_NumJoysticks();
        for (int i = 0; i < numJoysticks; ++i)
        {
            IntPtr joystick = SDL.SDL_JoystickOpen(i);
            if (joystick != IntPtr.Zero)
            {
                int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);

                if (productIDs.Contains(getProductIDs))
                {
                    joysticks.Add(new Joystick(i, getProductIDs, joystick));
                }

            }
        }

        bool quit = false;

        while (!quit)
        {
            SDL.SDL_Event e;
            while (SDL.SDL_PollEvent(out e) != 0)
            {
                foreach (var joy in joysticks)
                {
                    joy.Handle(e);
                }
            }
        }
    }
}

using System;
using SDL2;
using static SDL2.SDL;

class Program
{
    static void Main(string[] args)
    {
        SDL.SDL_Init(SDL.SDL_INIT_JOYSTICK);

        List<int> productIDs = new List<int> { /*hidden for reasons*/ };
        List<Joystick> joysticks = new List<Joystick>();

        int numJoysticks = SDL.SDL_NumJoysticks();
        for (int i = 0; i < numJoysticks; ++i)
        {
            IntPtr joystick = SDL.SDL_JoystickOpen(i);
            if (joystick != IntPtr.Zero)
            {
                int getProductIDs = SDL.SDL_JoystickGetProduct(joystick);

                if (productIDs.Contains(getProductIDs))
                {
                    joysticks.Add(new Joystick(i, getProductIDs, joystick));
                }

            }
        }

        bool quit = false;

        while (!quit)
        {
            SDL.SDL_Event e;
            while (SDL.SDL_PollEvent(out e) != 0)
            {
                foreach (var joy in joysticks)
                {
                    joy.Handle(e);
                }
            }
        }
    }
}

TheRanger•10mo ago

yeah i know why the SDL_Event doesnt know what joystick button u pressed

jcaishereOP•10mo ago

yeah

TheRanger•10mo ago

it just knows the button index that is pressed

jcaishereOP•10mo ago

i mean, that's another thing

TheRanger•10mo ago

wrap ur console.write line in an if condition of pressed if (pressed == 1)

jcaishereOP•10mo ago

the other thing is that if i press something from 8, since joystick 9 has also a pressed button too, it displays both

TheRanger•10mo ago

do that, and it should work do the if condition the reason for ur bug was

     while (SDL.SDL_PollEvent(out e) != 0)
            {
                // oh look, a button was pressed
                foreach (var joy in joysticks) // let every joystick handle e
                {
                    joy.Handle(e);
                }
            }

     while (SDL.SDL_PollEvent(out e) != 0)
            {
                // oh look, a button was pressed
                foreach (var joy in joysticks) // let every joystick handle e
                {
                    joy.Handle(e);
                }
            }

then here

        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN: // its a joystick button
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button); // returns true for joystick 8, but false for joystick 9
                Console.WriteLine($"Joystick {_productId} is pressed!"); //doesnt matter, a joystick button is pressed, ill just call this
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }

        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN: // its a joystick button
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button); // returns true for joystick 8, but false for joystick 9
                Console.WriteLine($"Joystick {_productId} is pressed!"); //doesnt matter, a joystick button is pressed, ill just call this
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }

jcaishereOP•10mo ago

yeah it should be like, i pressed a button from a specific joystick, then it should display which button did i press and which joystick did it come from.

TheRanger•10mo ago

thats SDL_JoystickGetButton's job it detects if the pressed button is coming from that specific joystick

jcaishereOP•10mo ago

yeah question is, which button was pressed that's what im trying to find

TheRanger•10mo ago

its stored in e.jbutton.button

jcaishereOP•10mo ago

okay that works now but what about where it's only detecting a specific joystick regardless of another joystick having a pressed button too.

jcaishereOP•10mo ago

for example, im only pressing 8 here but 9 shows up as well because it has a button down too.

jcaishereOP•10mo ago

i only want to get from 8.

TheRanger•10mo ago

huh, show ur joystick class again

jcaishereOP•10mo ago

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                if (pressed == 1)
                {
                    Console.WriteLine($"Joystick {_productId} button {e.jbutton.button} is pressed!");
                }
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

using SDL2;
using static SDL2.SDL;

internal class Joystick
{
    private int i;
    private int _productId;
    private nint _joystick;

    public Joystick(int i, int _productId, nint _joystick)
    {
        this.i = i;
        this._productId = _productId;
        this._joystick = _joystick;
    }

    public void Handle(SDL_Event e)
    {
        switch (e.type)
        {
            case SDL.SDL_EventType.SDL_JOYBUTTONDOWN:
                int pressed = SDL.SDL_JoystickGetButton(_joystick, e.jbutton.button);
                if (pressed == 1)
                {
                    Console.WriteLine($"Joystick {_productId} button {e.jbutton.button} is pressed!");
                }
                break;
            case SDL.SDL_EventType.SDL_JOYAXISMOTION:
                int motion = SDL.SDL_JoystickGetAxis(_joystick, e.jaxis.axis);
                break;
            default:
                break;
        }
    }
}

TheRanger•10mo ago

is this an old image? it doesnt match ur code

jcaishereOP•10mo ago

ill show again

jcaishereOP•10mo ago

im only pressing joystick 6 here, but 8 shows up although im not doing anything with it

TheRanger•10mo ago

:bigthonk:

jcaishereOP•10mo ago

it does have a pressed button (there's no other way around it with my physical joystick)

TheRanger•10mo ago

can u print the value of _joystick

jcaishereOP•10mo ago

i mean, if it's not possible, i guess that's fine it's random numbers

TheRanger•10mo ago

i know, show them those are addresses, like the ones you see in C/C++

jcaishereOP•10mo ago

TheRanger•10mo ago

and ur only pressing buttons from only 1 joystick?

jcaishereOP•10mo ago

yep

TheRanger•10mo ago

It doesn't make sense tbh

jcaishereOP•10mo ago

hm i think i can just leave it at that as long as it stays there, it's fine atleast im not touching it

TheRanger•10mo ago

maybe ur other joystick is lagging?

jcaishereOP•10mo ago

TheRanger•10mo ago

or its very sensitive

jcaishereOP•10mo ago

just that it always has one button down just think of switches a lightswitch it's pretty much on the firmware that's it's like this

TheRanger•10mo ago

All i can think of is that ur joystick is very sensitive

jcaishereOP•10mo ago

actually it's just the firmware's built like this so it's all fine

TheRanger•10mo ago

If you say so

jcaishereOP•10mo ago

anyways, on to more complex part:

byte button0 = SDL.SDL_JoystickGetButton(joy1, 0);
byte button1 = SDL.SDL_JoystickGetButton(joy1, 1);
byte button2 = SDL.SDL_JoystickGetButton(joy1, 2);
byte button3 = SDL.SDL_JoystickGetButton(joy1, 3);

if (button0 == 1) { Console.Write("\nOff"); }

if (button1 == 1) { Console.Write("\nLeft"); }

if (button2 == 1) { Console.Write("\nCenter"); }

if (button3 == 1) { Console.Write("\nRight"); }

byte button0 = SDL.SDL_JoystickGetButton(joy1, 0);
byte button1 = SDL.SDL_JoystickGetButton(joy1, 1);
byte button2 = SDL.SDL_JoystickGetButton(joy1, 2);
byte button3 = SDL.SDL_JoystickGetButton(joy1, 3);

if (button0 == 1) { Console.Write("\nOff"); }

if (button1 == 1) { Console.Write("\nLeft"); }

if (button2 == 1) { Console.Write("\nCenter"); }

if (button3 == 1) { Console.Write("\nRight"); }

byte button4 = SDL.SDL_JoystickGetButton(joy2, 4);
byte button5 = SDL.SDL_JoystickGetButton(joy2, 5);
byte button6 = SDL.SDL_JoystickGetButton(joy2, 6);
byte button7 = SDL.SDL_JoystickGetButton(joy2, 7);

if (button4 == 1) { Console.Write("\nSouth"); }

if (button5 == 1) { Console.Write("\nWest"); }

if (button6 == 1) { Console.Write("\nNorth"); }

if (button7 == 1) { Console.Write("\nEast"); }

byte button4 = SDL.SDL_JoystickGetButton(joy2, 4);
byte button5 = SDL.SDL_JoystickGetButton(joy2, 5);
byte button6 = SDL.SDL_JoystickGetButton(joy2, 6);
byte button7 = SDL.SDL_JoystickGetButton(joy2, 7);

if (button4 == 1) { Console.Write("\nSouth"); }

if (button5 == 1) { Console.Write("\nWest"); }

if (button6 == 1) { Console.Write("\nNorth"); }

if (button7 == 1) { Console.Write("\nEast"); }

im thinking of doing something like this

TheRanger•10mo ago

you could start with creating a method called OnPressed(int button);

jcaishereOP•10mo ago

it also has to be on a specific joystick, as it shows there

TheRanger•10mo ago

     if (pressed == 1)
                {
                    OnPressed(e.jbutton.button);
                    Console.WriteLine($"Joystick {_productId} button {e.jbutton.button} is pressed!");
                }

     if (pressed == 1)
                {
                    OnPressed(e.jbutton.button);
                    Console.WriteLine($"Joystick {_productId} button {e.jbutton.button} is pressed!");
                }

well you have ur productId

jcaishereOP•10mo ago

OnPressed has to be those { Console.Write("\nSouth"); } stuff, right?

TheRanger•10mo ago

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     if (button == 0) { Console.Write("\nOff"); }
     if (button == 1) { Console.Write("\nLeft"); }
     if (button == 2) { Console.Write("\nCenter"); }
     if (button == 3) { Console.Write("\nRight"); }
  }
  else if (_productId == 2222)
  {
     if (button == 5) { Console.Write("\nSouth"); }
     if (button == 6) { Console.Write("\nWest"); }
     if (button == 7) { Console.Write("\nNorth"); }
     if (button == 8) { Console.Write("\nEast"); }
  }

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     if (button == 0) { Console.Write("\nOff"); }
     if (button == 1) { Console.Write("\nLeft"); }
     if (button == 2) { Console.Write("\nCenter"); }
     if (button == 3) { Console.Write("\nRight"); }
  }
  else if (_productId == 2222)
  {
     if (button == 5) { Console.Write("\nSouth"); }
     if (button == 6) { Console.Write("\nWest"); }
     if (button == 7) { Console.Write("\nNorth"); }
     if (button == 8) { Console.Write("\nEast"); }
  }

something like that

jcaishereOP•10mo ago

alright thanks for your big help

TheRanger•10mo ago

jcaishereOP•10mo ago

@R another thing, is there a way to make sure that im getting all the button state from a joystick? because from this code, it only shows when im pressing a button but it's not getting all button states

TheRanger•10mo ago

button state as in if its pressed or not?

jcaishereOP•10mo ago

yep let's say, i have a joystick that has a lot of buttons some of them are pressed and some of them are not

TheRanger•10mo ago

yea well u have SDL_JoystickGetButton it gets the state of a button loop on all of the buttons

jcaishereOP•10mo ago

in my current code, how do you do that?

TheRanger•10mo ago

use a for loop?

jcaishereOP•10mo ago

oh right

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     for (int i = 0; i < 27; i++)
    {
        int pressed = SDL.SDL_JoystickGetButton(_joystick, i);
    }
  }
  else if (_productId == 2222)
  {
     if (button == 5) { Console.Write("\nSouth"); }
     if (button == 6) { Console.Write("\nWest"); }
     if (button == 7) { Console.Write("\nNorth"); }
     if (button == 8) { Console.Write("\nEast"); }
  }
}

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     for (int i = 0; i < 27; i++)
    {
        int pressed = SDL.SDL_JoystickGetButton(_joystick, i);
    }
  }
  else if (_productId == 2222)
  {
     if (button == 5) { Console.Write("\nSouth"); }
     if (button == 6) { Console.Write("\nWest"); }
     if (button == 7) { Console.Write("\nNorth"); }
     if (button == 8) { Console.Write("\nEast"); }
  }
}

@R now the issue here idk how would i get that i or do i just write 0-26 here?

TheRanger•10mo ago

wdym how u would get that i

jcaishereOP•10mo ago

hm oh wait this is wrong i was trying to get all button states i decided to write 0-26. rather than getting the i

TheRanger•10mo ago

jcaishereOP•10mo ago

okay let me try to clear something up this is kind of what i did:

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     for (int i = 0; i < 27; i++)
    {
        byte button0 = SDL.SDL_JoystickGetButton(_joystick, 0);
        byte button1 = SDL.SDL_JoystickGetButton(_joystick, 1);
        byte button2 = SDL.SDL_JoystickGetButton(_joystick, 2);
        byte button3 = SDL.SDL_JoystickGetButton(_joystick, 3);
        
        if (button0 == 1) { Console.Write("\nOff"); }
        if (button1 == 1) { Console.Write("\nLeft"); }
        if (button2 == 1) { Console.Write("\nCenter"); }
        if (button3 == 1) { Console.Write("\nRight"); }

    }
  }

}

void OnPressed(int button)
{
  if (_productId = 1111)
  {
     for (int i = 0; i < 27; i++)
    {
        byte button0 = SDL.SDL_JoystickGetButton(_joystick, 0);
        byte button1 = SDL.SDL_JoystickGetButton(_joystick, 1);
        byte button2 = SDL.SDL_JoystickGetButton(_joystick, 2);
        byte button3 = SDL.SDL_JoystickGetButton(_joystick, 3);
        
        if (button0 == 1) { Console.Write("\nOff"); }
        if (button1 == 1) { Console.Write("\nLeft"); }
        if (button2 == 1) { Console.Write("\nCenter"); }
        if (button3 == 1) { Console.Write("\nRight"); }

    }
  }

}

@R does this work fine?

TheRanger•10mo ago

ofcourse not, u never used i

jcaishereOP•10mo ago

well, this is like im using specific buttons only

TheRanger•10mo ago

then loop on those specific buttons

jcaishereOP•10mo ago

it did somehow work but i have a bad feeling that SDL.SDL_JoystickGetButton(_joystick, 0); will be also detected by other joysticks

TheRanger•10mo ago

it shouldnt

jcaishereOP•10mo ago

hm let me try to put it on the test.

TheRanger•10mo ago

+ now the variable button is useless here since you never used it

jcaishereOP•10mo ago

yep

TheRanger•10mo ago

which would make the method's purpose pointless

jcaishereOP•10mo ago

hm yeah i got back to that old error 2+ joysticks just don't work but i learned something here feels like i have to make a custom event-driven system in my program so that i can try reducing cpu usage

TheRanger•10mo ago

use Thread.Sleep Games that render 60 frames per sec, they just divide 1000 / 60, which is 16.66666666667 ms

Buddy•10mo ago

Note that Thread.Sleep is not precise it is far off from precise

TheRanger•10mo ago

What would you recommend?

Buddy•10mo ago

Timers? Either roll your own high resolution timer or use a library If you need exact or close to 60 fps

TheRanger•10mo ago

well i use Thread.Sleep with 16.66667 ms minus the amount of ms that passed since the previous frame @ajcdev sorry, turns out it does, lol no idea how i missed it

Gaming

Programming

SDL2 Joystick issues