java メンバ変数について

Question

javaのメンバ変数についての質問です。
keyboardクラスにて、メンバ変数のkeycodeにkeyPressedメソッドから値を代入したいと思っています。keycodeはstaticにして、別クラスにてkeyboard.keycodeと書いて取得したキーによっていろいろな動作を行いたいと思っています。
ただ、keyPressedメソッドでkeycodeに値を入れることはできるのですが、そこからメンバ変数に入れることができません。
解決方法をお教え頂ければ幸いです。

追加
keyPressedメソッドでkeycodeに値を入れることはできていると思うのですが、
testメソッドにてkeycodeの値に応じて文字を出力するようにしているのですが、スペースキーを押しても文字が出力されません。

class keyboard implements KeyListener {
    static int keycode;

    public void keyReleased(KeyEvent e) {}
    public void keyTyped(KeyEvent e) {}
    public void keyPressed(KeyEvent e){
	this.keycode = e.getKeyCode();    
    }
     public void test() {
	 if(keycode == KeyEvent.VK_SPACE) {
	     System.out.println("a");
	 }
     }
}

public class aaaa{
    public static frame frame;
    public static void main(String[] args) {
	frame = new frame();
    }
}

class frame extends JFrame {
    public panel panel;    
    public frame() {

	panel = new panel();
	this.addKeyListener(new keyboard());
    }
}

Comments

[momon-ga]

コンパイルエラーだと思うのですが、それを直したいってことですか？

[white_]

コンパイルエラーは表示されませんでした。値が代入できていないのが問題です。

[momon-ga]

失礼。Warningでした。
んー。
this.keycode = e.getKeyCode(); が呼ばれてる時点で、keycodeには値が設定されているはずなのですが。

[white_]

他の要素としてJFrameを作成。そこでKeyListenerをaddしているのですがここに問題があるのですか。

[YT0014]

現在のコードでは、keyboardクラスのstatic変数keycodeに値が正常にセットされているのですよね？
で、希望されている、メンバ変数というのは、どのクラスの何という変数なのでしょうか？
それとも、static変数keycodeに値がセットされない、という状況なのでしょうか？

[swordone]

> ただ、keyPressedメソッドでkeycodeに値を入れることはできるのですが、そこからメンバ変数に入れることができません。

あなたの言う「メンバ変数」ってどれの事ですか？

[white_]

keyboardクラス直下のkeycodeのことです。構造的にはkeycodeに値がセットされるとは思うのですが・・・コンパイルエラーも出ていません。

[momon-ga]

> keyPressedメソッドでkeycodeに値を入れることはできるのですが、そこからメンバ変数に入れることができません。

は、以下のように読みかえればよいですか？

keyPressedメソッドでkeycodeに値を入れることはできるのですが、そこからkeyboardクラス直下のkeycodeに入れることができません。

とは、どういう状況なのでしょうか？

[white_]

このコード以外でもいろいろと試していました。まずkeyboardクラス直下のkeycodeを宣言せず、直接keyPressedメソッドにてint keycode = e.getKeyCode();を行う。ただ、この場合keycodeを他クラスに渡す時(自分の場合はメインクラスです)インスタンス生成して(例：aaaa a = new aaaa();)クラス内のメソッドに値を渡す(a.main(keycode);)。ただ、public static void main(String[] args)の中でkeycodeをしたいので、aaaaクラス内のmainメソッド内に値を渡せません。(引数が違うので)

話を戻します。自分の知識不足で申し訳ないのですが、keyPressedメソッドでthis.keycodeはkeyboardクラス直下のメンバ変数keycodeを指すことだと思っています。

つまり、this.keycode=e.getKeyCode();とするこでクラス直下のメンバ変数keycodeにも値を代入が入ると認識しています。

しかし、同クラス内のtestメソッドにてkeycodeの値に応じて文字を出力するようにしているのですが、スペースキーを押しても文字が出力されません。つまり、クラス直下のメンバ変数keycodeに値が代入されていないということになります。

>keyPressedメソッドでkeycodeに値を入れることはできるのです(1)が、そこからkeyboardクラス直下のkeycodeに入れることができません(2)。

これは、自分の前提です。keycode = e.getKeyCode();とすることで値は代入できると思います(1)。ただ、this.と置いてもメンバ変数keycodeに値が代入されていません(2)。

[momon-ga]

質問を正しくしたほうがよいです。（質問を以下のように修正してください）
---
keyPressedメソッドでkeycodeに値を入れることはできていると思うのですが、
testメソッドにてkeycodeの値に応じて文字を出力するようにしているのですが、スペースキーを押しても文字が出力されません。
---
ですね。と、この質問だけでも理由はうすうすわかる気がしますが・・・

[white_]

すみません、文章コピペして修正しました。変数keycodeはオブジェクト生成したときに作成されると思います。frameクラスにて this.addKeyListener(new keyboard());とすることでオブジェクト生成がなされていると思うんですけど勘違いですか。

[momon-ga]

インスタンスは作られてますが、test()を呼ぶ処理がないです。keyPressで自動で呼ばれるようなものじゃないです。

[white_]

test()を呼ぶ処理については理解しました。ありがとうございます。ただmainメソッド内にてkeycodeを参照して動作を行う場合(if(keyboard.keycode == KeyEvent.VK_SPACE) {・・・ )としても、スペースキーを押しているにも関わらず動作が行われません。

[momon-ga]

イベントドリブンを理解されていないと思うのですが、mainにif文を書いてもキーを押したらmainの中身が呼ばれるわけじゃないのは理解されてますか？
単純なものなら、test()を keyPressの中に書けばいいと思いますけど。

Answer 1

momon-gaさんの回答のように、test()が呼ばれてないのと、
同じようにaddActionLisner()しているyachtframe()がmain()内で呼ばれてないですね。
あと、panelクラスの実装が不明ですが、Jpanelの継承でよいのかな。

その前提でいくと、下記のような感じですかね。

import java.awt.Container;

public class Main{
    public static MyFrame frame;
    public static MyPanel panel;
    public static void main(String[] args) {
        frame = new MyFrame();
        panel = new MyPanel();

        Container container = frame.getContentPane();
        container.add(panel);
        frame.yachtframe();
        frame.setVisible(true);
    }
}

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

class Keyboard implements KeyListener {
    static int keycode;

    public void keyReleased(KeyEvent e) {}

    public void keyTyped(KeyEvent e) {}

    public void keyPressed(KeyEvent e){
        this.keycode = e.getKeyCode();
        test();
    }
    public void test() {
        if(keycode == KeyEvent.VK_SPACE) {
            System.out.println("a");
        }
    }
}

import javax.swing.*;

class MyFrame extends JFrame {
    public MyPanel panel;
    public void yachtframe() {
        panel = new MyPanel();
        this.addKeyListener(new Keyboard());
    }
}

import javax.swing.*;
public class MyPanel extends JPanel{

}

最後に、必ず強制されるものではないですが、
Javaではクラス名の先頭の文字は大文字にする慣習があります。
大抵のエディタやIDEではwarningが出るくらい浸透している慣習です。

メソッドなら先頭小文字、
クラス名とコンストラクタ（クラスと同名）は先頭大文字です。
それに従ってクラス名は少し直しました。

Answer 2

keyPressedメソッドでkeycodeに値を入れることはできていると思うのですが、

testメソッドにてkeycodeの値に応じて文字を出力するようにしているのですが、スペースキーを押しても文字が出力されません。

keyuPressedイベントの発生とtestメソッドは関連がないので、明示的に呼び出す必要があります。

public void keyPressed(KeyEvent e){
    this.keycode = e.getKeyCode();    
    test();
}     

じゃあ、他のクラスで使ってるkeyを判定するには、どうすればよいか？
いわゆる通知用のクラスを使うか、keyboardクラスの内部に持てばよいと思います。
※なんちゃらLisnerを自前でやってる感じです

class keyboard implements KeyListener {
    static int keycode;
    SomeClass action;

    keyboard (SomeClass action) {
        this.action = action;
    }
    public void keyPressed(KeyEvent e){
        this.keycode = e.getKeyCode();    
        action.someMethod(keycode);
    }
}