[Solved] Can the new Input system be used without the Player Input Component?

I am learning from some old tutorials that do not use the player input component, but as I am trying for now I have no luck using only the generated C# scripting without using the Player Input Component.

Code (CSharp):

1. using System.Collections;
2. using System.Collections.Generic;
3. using UnityEngine;
4. using UnityEngine.InputSystem;
5.  
6. public class Player_Input : MonoBehaviour
7. {
8.  
9.     public GameObject unitSelected;
10.  
11.     public Controls controls;
12.  
13.     private void Awake()
14.     {
15.  
16.         controls = new Controls();
17.        
18.     }
19.  
20.     private void OnEnable()
21.     {
22.  
23.         controls.Enable();
24.         controls.General.Attack.performed += ShootButton;
25.         controls.General.Move.performed += MoveButton;
26.  
27.     }
28.  
29.     private void OnDisable()
30.     {
31.  
32.         controls.General.Attack.performed -= ShootButton;
33.         controls.General.Move.performed -= MoveButton;
34.         controls.Disable();
35.  
36.     }
37.  
38.     public void ShootButton(InputAction.CallbackContext context)
39.     {
40.  
41.         if (context.started && !context.performed)
42.         {
43.             //Shoot Here
44.             Debug.Log("Shoot");
45.         }
46.  
47.     }
48.  
49.     public void MoveButton(InputAction.CallbackContext context)
50.     {
51.  
52.         if (context.started && !context.performed)
53.         {
54.             //Movement here
55.             unitSelected.GetComponent<Movement_Controller>().Move();
56.         }
57.  
58.     }
59. }
60.  

 

I could be mistaken, but is the issue not that you're registering for the "performed" callbacks, but then in your handlers you're checking "!context.performed"?

 

I am assuming you are saying this

1. if (context.started && !context.performed)
2. {
3. //Shoot Here
4. Debug.Log("Shoot");
5. }

I put that there because for some reason when I use the component to make the input system work it call the function 4 times. Adding the ifs prevented it.

 

Ok, but look at how you're preventing it. I strongly suspect you're filtering out the wrong calls.

 

Thanks it works now, it is the if's statements. I think I should not have left them there because I change into the component midway and then came back to try using scripts again. Thanks again!

 

how we can use the c# generated class this way for local multiplayer?

 

thanks, @Rene-Damm the way of doing using the Player Input manager and manually spawning player on join using the InputUser or Player Input is for me the better option. thanks so much.

 

The input system is now at version 1.0.0, why do i still have to find some forum post for essential documentation?

Honestly the PlayerInputManager is making things more confusing rather than helping out, i cant understand why it was added. It fits a very specific case, where players join in at any moment and might have split screen, and even that not very well. Any other form of having local multiplayer, such as pre-selecting input devices in a menu, seems utterly unsupported.

 

Hi, I've just tried to implement your solution and it throws a memory leak error that seems to be coming from the user.ActivateControlScheme("Gamepad"); line: "A native collection has not been disposed, resulting in a memory leak". Is there some extra line to force it to dispose of the native collection?

 

Oh and if it's of any help the error only registers starting from the second play after compiling the script for whatever reason (even if the script itself wasn't changed), so the first play after returning to unity from visual studio never registers an error.

 

Hi, I'm trying to figure out how to get this information from the PlayerInput module. Maybe something like this:

Code (csharp):

1.  
2. // you have a reference to the PlayerInput module called p1Input
3. var p1Controls = new MyControls();
4. p1Controls.devices = p1Input.devices;
5. p1Controls.bindingMask = InputBinding.MaskByGroup("p1Input.controlScheme");
6.  

It would be nice if you could just pass an InputControlScheme struct to the script since that would have all the necessary information, right? Or couldn't it just look for a PlayerInput module?

 

Apologies if I am mistaken. From what I know any MyControls would need to be disposed at the end. so it would be something like p1Controls.Dispose();.
Also always desubscribe all methods from the input actions, then disable the action map before using p1Controls.Dispose().
Though this reply might be too late to help you. It's for any1 else out there that stumbles onto it.