ゲームパッドが反映されない

Question

jinputを使用してゲームパッドをコントローラ代わりにしてクライアントサーバ通信でロボットを操作するということを行おうとしています.ゲームパッドを使用するのはクライアント側でjavafx8を使用してアプリケーションを作成しました.問題点はアプリを起動してスティックを動かすとアプリが応答停止になってしまいます.

応答停止になる原因にはどのようなものがあるのでしょうか？
下記にソースコードを載せますのでどこが悪いのか教えてはくれないでしょうか?

package ieexp3;

import java.util.function.Consumer;

import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.scene.control.Button;
import javafx.scene.control.TextField;



public class operateRobot_with_GamepadController   {
	
	//オブジェクト
	@FXML private Button buttonConnect;
	@FXML private Button buttonDisconnect;
	@FXML private Button buttonRun;
	@FXML private TextField inputIPaddress;
	@FXML private TextField inputPort;
	@FXML private static TextField viewLog;
	
	//通信クラス
	static CommunicationTask task;
	
	static robotController task2;
	
	
	
	
	
	//イベントハンドラの処理
	@FXML protected void handleButtonConnect(ActionEvent e){
		//接続処理（タスク起動）
		String server = inputIPaddress.getText();
		int port = Integer.parseInt(inputPort.getText());
		task = new CommunicationTask(server,port);
		
		viewLog.textProperty().bind(task.messageProperty());
		Thread thread = new Thread(task);
		thread.setDaemon(true);
		thread.start();
	}
	@FXML protected void handleButtonDisconnect(ActionEvent e){
		//タスク終了処理
		task.makeMessage("disconnect");
		task.cancel();
		viewLog.textProperty().unbind();
	}
	@FXML protected void handleButtonRun(ActionEvent e){
		task2 = new robotController();
		Thread thread = new Thread(task2);
		thread.setDaemon(true);
		task.makeMessage("run");
		if(ControllerInput.State.x == 1){
			task.makeMessage("right");
		}else if(ControllerInput.State.x == -1){
			task.makeMessage("left");
		}else if(ControllerInput.State.y == 1){
			task.makeMessage("forward");
		}else if(ControllerInput.State.y == -1){
			task.makeMessage("back");
		}else if(ControllerInput.State.catchButtonPushed){
			task.makeMessage("catch");
		}else if(ControllerInput.State.downButtonPushed){
			task.makeMessage("down");
		}else if(ControllerInput.State.upButtonPushed){
			task.makeMessage("up");
		}
	}

	
}

package ieexp3;

import java.util.Optional;


import net.java.games.input.Controller;
import net.java.games.input.ControllerEnvironment;

public interface ControllerInput  {
	
	//コントローラーの入力状態を取得する抽象メソッド
	Optional<ControllerInput.State> getState();
	
	
	boolean available();
	
	//コントロールオブジェクトを取得する抽象メソッド
	static Controller detectController(Controller.Type type){
		Controller[] controllers = ControllerEnvironment.getDefaultEnvironment().getControllers();
		for(Controller controller : controllers){
			if(controller != null && controller.getType() == type){
				return controller;
			}
		}
		return NullController.Instance;
	}
	
	//コントローラーの入力状態
	static final class State {
		static int x;
		static int y;
		static boolean upButtonPushed;
		static boolean downButtonPushed;
		static boolean catchButtonPushed;
		
		
		State(int x,int y,boolean upButtonPushed,boolean downButtonPushed,boolean catchButtonPushed){
			this.x = x;
			this.y = y;
			this.upButtonPushed = upButtonPushed;
			this.downButtonPushed = downButtonPushed;
			this.catchButtonPushed = catchButtonPushed;
		}
	}
}

```package ieexp3;

import java.util.Optional;

import net.java.games.input.Component.Identifier;
import net.java.games.input.Controller;


public final class GamepadControllerInput implements ControllerInput {
	
	private Controller controller;
	
	
	private boolean button2Released;
	private boolean button3Released;
	private boolean button6Released;
	
	//ゲームコントローラーコンストラクタ生成
	GamepadControllerInput(){
		this.controller = ControllerInput.detectController(Controller.Type.GAMEPAD);
		this.button2Released = true;
		this.button3Released = true;
		this.button6Released = true;
	}
	public Controller getController(){
		return this.controller;
	}
	@Override
	public boolean available(){
		return controller != NullController.Instance;
	}
	
	@Override
	public Optional<ControllerInput.State> getState(){
		//コントローラの入力が受け付けられた場合
		if(controller.poll()){
			//スティックの状態を取得
			float x0 = controller.getComponent(Identifier.Axis.X).getPollData();
			float y0 = controller.getComponent(Identifier.Axis.Y).getPollData();
			int x = 0;
			int y = 0;
			
			//x軸
			if(x0 > 0.0f){
				x = 1;
			}else if(x0 < 0.0f){
				x = -1;
			}else{
				x = 0;
			}
			//y軸
			if(y0 > 0.0f){
				x = 1;
			}else if(y0 < 0.0f){
				x = -1;
			}else{
				x = 0;
			}
			//ボタン判定
			boolean button2Pushed = controller.getComponent(Identifier.Button._1).getPollData()>0.0f;
			boolean upButtonPushed = button2Released && button2Pushed;
			button2Released = !button2Pushed;
			
			boolean button3Pushed = controller.getComponent(Identifier.Button._2).getPollData()>0.0f;
			boolean downButtonPushed = button3Released && button3Pushed;
			button3Released = !button3Pushed;
			
			boolean button6Pushed = controller.getComponent(Identifier.Button._5).getPollData()>0.0f;
			boolean catchButtonPushed = button6Pushed && button6Released;
			button6Released = !button6Pushed;
			
			if(x != 0 || y != 0 || upButtonPushed || downButtonPushed || catchButtonPushed){
				return Optional.of(new ControllerInput.State(x, y, upButtonPushed, downButtonPushed, catchButtonPushed));
			}
			
		}
		return Optional.empty();
	}
	
}

package ieexp3;

import net.java.games.input.*;
import net.java.games.input.Component.Identifier;

public final class NullController implements Controller {
	static final Controller Instance = new NullController();
	
	@Override
	public Controller[] getControllers(){
		return new Controller[0];
	}
	  @Override
	    public Type getType() {
	        return Controller.Type.UNKNOWN;
	    }

	    @Override
	    public Component[] getComponents() {
	        return new Component[0];
	    }

	    @Override
	    public Component getComponent(Identifier id) {
	        return null;
	    }

	    @Override
	    public Rumbler[] getRumblers() {
	        return new Rumbler[0];
	    }

	    @Override
	    public boolean poll() {
	        return false;
	    }

	    @Override
	    public void setEventQueueSize(int size) {
	    }

	    @Override
	    public EventQueue getEventQueue() {
	        return null;
	    }

	    @Override
	    public PortType getPortType() {
	        return Controller.PortType.UNKNOWN;
	    }

	    @Override
	    public int getPortNumber() {
	        return -1;
	    }

	    @Override
	    public String getName() {
	        return "NullController";
	    }
}

package ieexp3;

public class robotController implements Runnable{
	
	private ControllerInput gamepadController;
	
	robotController(){
		this.gamepadController = new GamepadControllerInput();
	}
	public void run(){
		//ゲームパッド入力
		if(gamepadController.available()){
			while(true){
				gamepadController.getState();
				sleep(15L);
			}
		}
	}
	//コントローラ入力の状態を反映
	public void sleep(long millis){
		try{
			Thread.sleep(millis);
		}catch(InterruptedException e){
			throw new RuntimeException(e);
		}
	}
}

Answer 1

おそらく参考にされているページは同じものを見ていると思いますので、そのページを見ながらもう一度以下の点を確認してみてください。

• ネイティブライブラリーにパスを通す
• ゲームパッドをONにしてからJavaプログラムを起動する
• エラー（スタックトレース）が出ていたらそれをチェックする

それと、「ゲームパッドの信号が反映されない問題」について、もう少し調査をお願いします。
デバッグprintなどを活用して、どの時点で処理が動いていないのかを確認してみて下さい。

Comments

[kt3302y]

信号が反映されていないというのほRunボタンのイベントハンドラが設定されていなかったからでした
それを直したら6 28, 2016 8:08:19 午後 net.java.games.input.DefaultControllerEnvironment getControllers
警告: Found unknown Windows version: Windows 10
6 28, 2016 8:08:19 午後 net.java.games.input.DefaultControllerEnvironment getControllers
警告: Attempting to use default windows plug-in.
このエラーが出たのですがどうすればよいのでしょうか

[argius]

> 信号が反映されていないというのほRunボタンのイベントハンドラが設定されていなかったからでした

なるほど。
で、結局のところ動作はしたのでしょうか？

> このエラーが出たのですがどうすればよいのでしょうか

最新のjinput2.0.6が2年前なので、Windows10に対応していないのでしょう。
既知のバージョンであればそれ専用の対応を行うが、バージョンが既知でないため、
とりあえずWindowsとして動作する、というメッセージを警告として出しているんだと思います。
ログにおける警告というのは、エラーではないが問題があるかも知れないような場合に出力されます。

その警告を抑制する方法は分かりません。
jinputの新しいバージョンが（出ることは期待できませんが）出るまで待つしか無いかもです。

[kt3302y]

ゲームパッドを接続しているのにコンストラクタでNullControllerを返していましたので動作しませんでした

[argius]

Windows10で（参考サイトの方のコードを）動作確認をしてみたところ、
同じ警告メッセージが出ましたが動作しました。
よって、Windows10だから動作しないというわけではなさそうです。

念のため確認ですが、
最初の回答に書いたことは全てクリアーしているということで良いんですよね？
（これを書いていただかないと、どこまでが上手く行っているのか回答者側で判断できません。）
それと、jinput以外ではそのゲームパッドは正しく動作しているのですよね？

[kt3302y]

クリアしてると思います.
一つ質問なのですがネイティブライブラリーのパスを通していないとビルドエラーが発生するはずですよね？
ゲームパッドはオンラインゲームで使用していたのを使用しているため動作するはずです.

[argius]

> クリアしてると思います.
> ゲームパッドはオンラインゲームで使用していたのを使用しているため動作するはずです.

了解です。ありがとうございます。

> ネイティブライブラリーのパスを通していないとビルドエラーが発生するはずですよね？

いえ、ネイティブライブラリーはランタイムなので、実行時にパスが通っている場所に存在すれば大丈夫です。
ビルド時と実行時に必要なのは、jinput.jarだけです。
jutils.jarも実行時（のクラスパス）だけでOKです。

ちなみに、ネイティブライブラリーにパスが通っていない場合は、
java.lang.UnsatisfiedLinkError: no jinput-dx8_64 in java.library.path
などのエラーが出ます。

GamePadが検出できないのは、jinput内部とWindows側やDirectX側の連携まで
追いかけないといけないので、
せめて現物がないと調査が難しいです。
あっても難航しそうですが...

あとはダメ元で、JavaFXを使わずに、
参考にされたサイトのコードをそのまま動かしてみて（Windows10 64ビット版では確認済み）、
それでも動かなければ、
ちょっとお手上げかも知れません。

[kt3302y]

jUtilsというのも追加してクラスパスを通さなければ実行できないのですか？

[argius]

Jutilsは、GradleでJinputを設定したときにGradleが追加してくれています。
プロジェクトの「参照ライブラリー」というところを見ると、
jutils-1.0.0.jarが入っているはずです。
前回の質問の回答はこれのことを言っています。

[kt3302y]

ゲームパッドを取得する際の定数をGamepadからStickに変更したら接続することができました.
次に発生したのがプログラムが応答停止する問題なのですが応答停止になる原因には何が考えられるのですか？

[argius]

ゲームパッドではなく、ジョイスティックタイプだったのでしょうか？

応答停止というのは具体的にどのような状態でしょうか。

[argius]

いま不自由な環境なので小出しになってしまいすみません。

Stateクラスのフィールドはstaticにしてはダメです。
入力状態はgamepadController.getState()の値を使って判定してください。

[kt3302y]

アプリケーションがフリーズしたという表現でいいのですかね
statinフィールドにしてはいけない理由はなんでなのでしょうか？

[argius]

ご質問については後半で返答します。
先にコードの改善すべき箇所の説明から書きます。

まず、ControllerInputの入力状態のチェックですが、
これはループの中で行う必要があります。
なので、operateRobot_with_GamepadControllerのhandleButtonRunメソッドに
書いているif文を、robotControllerクラスのrunメソッドのwhile文の中に
書くようにした方がよいです。

あと、handleButtonRunメソッドのthreadはstartされていないので、
処理が開始しないと思います。

フリーズの件は、
Windowsなら「応答なし」状態になるのがフリーズです。
そうでなければ、単に処理が空振りしているだけの可能性があります。
デバッグを行って、どこが動いていないのかを確認してみて下さい。

staticフィールドの件は、
ControllerInput.Stateというのは入力をチェックした
その「時点」の「状態」を表すオブジェクトなので、
staticにしてしまうと状態が次のチェックと混ざってしまう恐れがあります。
ただ問題が発生する頻度がとても低いので、
正しく動作しているように見えてしまうと思います。
とにかく、クラスの使い方としては正しくないです。

[kt3302y]

修正したら正常に動作させることができました。
いろいろとお答えしてくれてありがとうございました。
最後にひとつだけお尋ねしたいのですが、
ifPresent(this::changestate)
これのthis::changestateはthis.changestateと同じ意味なのでしょうか？

[argius]

this::changestateというのは「メソッド参照」を表します。
this.changestateだとその場でメソッドを実行してしまいますが、
メソッド参照は条件を満たした場合のみメソッドを後から実行するような場合に使用します。

具体的には、
gamepadInput.getState().ifPresent(this::changeState);
はgetState()がOptionalを返しますが、値がemptyでない場合にだけ
changeState()を実行するようにしています。