1

私は CanvasPaint ライブラリをいじっていましたが、ユーザー入力に基づいて円を動かすことができる素敵な小さなゲームを作りたいと思っています。

私のアプローチは、同時に実行される 2 つの While ループを作成することです。1 つはユーザーからの新しい入力を探し、2 番目は円を移動します。

この種のコードを投稿した経験がなくて申し訳ありませんが、Java コーディングを始めたばかりです。

問題: 私の「方向」変数は常に 0 です。値を取得できません :(

package canvas;

import java.awt.Color;
import java.util.Scanner;

import TestPackage.CanvasPaint;

public class BilSpil extends Thread {

Scanner input = new Scanner (System.in);

private int direction;

public void run() { //First array, checking user-input
    while(true) {
            System.out.print("Direction: ");
            direction = input.nextInt();
        }
    }

//Getter for user-input
public int getDirection() {
    return direction;
}

public static void main(String[] args) {
    (new BilSpil()).start();

    BilSpil spilObject = new BilSpil(); //Create object of extended Thread.

    int commands = (spilObject.direction); //Raw command-data for circle


    //Game background
    int yBg = 10;

    CanvasPaint cp = new CanvasPaint(500,500);
    cp.setBackground(Color.black);

    for(int i = 0; i<50; i++) {
        cp.setColor(Color.DARK_GRAY);
        cp.paintLine(yBg, 500, yBg, 0); //Y-streger
        cp.paintLine(500, yBg, 0, yBg); //X-streger

        yBg += 10;
    }

    boolean gameIsLive = true;

    int x = 245;
    int y = 245;

    cp.setColor(Color.CYAN);
    cp.paintOval(x, y, 5, 5);

        while(gameIsLive) { //Second while-loop, moving the circle

            while(commands == 2) { //Down direction
                cp.paintOval(x, y+10, 5, 5);
                y += 10;
                try{Thread.sleep(1000);}
                catch (InterruptedException e) {}
                cp.repaintFrame();
            }
            while(commands == 4) { //Left direction
                cp.paintOval(x-10, y, 5, 5);
                x -= 10;
                try{Thread.sleep(1000);}
                catch (InterruptedException e) {}
                cp.repaintFrame();
            }
            while(commands == 8) { //UP direction
                cp.paintOval(x, y-10, 5, 5);
                y-= 10;
                try{Thread.sleep(1000);}
                catch (InterruptedException e) {}
                cp.repaintFrame();
            }
            while(commands == 6) { //RIGHT direction
                cp.paintOval(x+10, y, 5, 5);
                x += 10;
                try{Thread.sleep(1000);}
                catch (InterruptedException e) {}
                cp.repaintFrame();
            }
        }

}

}

4

3 に答える 3

0

while ブロックで最初に行うべきことは、ユーザー入力を取得することです。複数のオプションがあり、そのうちの 1 つを決定する必要があります: キーボード、ボタンなど...

ユーザーが何をしたいのかがわかったら、switch を使用し、while を使用しないでください (それらから出ないため、これは非常に正しくありません)。

while(gameIsLive) { //Second while-loop, moving the circle
    commands = getUserInput();
    switch (commands) {
    case 2: { //Down direction
            cp.paintOval(x, y+10, 5, 5);
            y += 10;
            try{Thread.sleep(1000);}
            catch (InterruptedException e) {}
            cp.repaintFrame();
        } break;
    case 4: { //Left direction
            cp.paintOval(x-10, y, 5, 5);
            x -= 10;
            try{Thread.sleep(1000);}
            catch (InterruptedException e) {}
            cp.repaintFrame();
        } break;
    case 8: { //UP direction
            cp.paintOval(x, y-10, 5, 5);
            y-= 10;
            try{Thread.sleep(1000);}
            catch (InterruptedException e) {}
            cp.repaintFrame();
        } break;
    case 6: { //RIGHT direction
            cp.paintOval(x+10, y, 5, 5);
            x += 10;
            try{Thread.sleep(1000);}
            catch (InterruptedException e) {}
            cp.repaintFrame();
        } break;
    }

ユーザー入力をどのように聞き取り、ゲームを表示するために何を使用しているかについて詳しく教えていただければ、より多くのガイドラインを提供できるかもしれません。

于 2013-10-01T21:46:22.757 に答える
0

を使用する場合はwhile(true) { //do stuff };、一部に//do stuffa を含めbreak;てループを終了する必要があります。

于 2013-10-01T21:41:25.983 に答える
0

方向が変化しない理由は、2 つのインスタンスがBilSpilあり、変化していないインスタンスから方向を取得しているためです。

 (new BilSpil()).start();

BilSpil spilObject = new BilSpil(); //Create object of extended Thread.

int commands = (spilObject.direction); //Raw command-data for circle

この部分を機能させるには、これを行う必要があります。

    BilSpil spilObject = new BilSpil(); //Create object of extended Thread.
    spilObject.start();

    int commands = (spilObject.direction); //Raw command-data for circle

へのアクセスdirectionはスレッドセーフではなく、同時実行例外が発生する可能性があると思います。

代わりにjava.util.concurrent.Callable、何かを返したいので実装することをお勧めします

/**
* The Callable<V> interface allows you to define a Object V that you will
* return. In this case we are returning the integer object wrapper Integer
*/
public class BilSpil implements Callable<Integer> {
Scanner input = new Scanner(System.in);
/**
*Since we defined our class to implement Callable<Integer> we have to override
* the abstract method V call() throws Exception, and in our case it is of 
*the type Integer
*/
@Override
public Integer call() throws Exception {
     /**
     *input.nextInt() will block and not return untill a integer is entered 
     * java autoboxes the int primitive to its wrapper class Integer automagically
     */
    return input.nextInt();
}

すべてを 1 つのループに移動し、switch ステートメントを使用します。

 public static void main(String[] args) {
     //creates a executor service with a fixed number of threads
     //that this ExecutorService can access
    ExecutorService executor = Executors.newFixedThreadPool(10);
    //submitting a callable runs it asynchronously, executor returns
    //a Future object we can use later to get the returned value
    //once the callable is done executing.
    Future<Integer> userInput = executor.submit(new BilSpil());
    int commands = 0; 

    ...

    while (gameIsLive) { 
        if(userInput.isDone()){
            try {
                commands = userInput.get().intValue();
            } catch (InterruptedException ex) {
              commands = 0;
            } catch (ExecutionException ex) {
               commands = 0;
            }
            userInput = executor.submit(new BilSpil());
        }

        switch(commands){
            case 2:
                y += 10;
                break;
            case 4:
                x -= 10;
                break;
            case 8:
                y -= 10;
                break;
            case 6:
                x += 10;
                break;
        }
        cp.paintOval(x, y, 5, 5);
        cp.repaintFrame();
    }

}

あなたができるもう1つの改善は、マジックナンバーを取り除くことです

public static final int MOVE_SPEED = 10;
public static final int DOWN = 2;
public static final int LEFT = 4;
public static final int UP = 8;
public static final int RIGHT = 6;
public static final int NO_INPUT = 0;
public static final long GAME_CLOCK = 1000L;
public static final int BODY_WIDTH = 5;
public static final int BODY_HEIGHT = 5;

... 

       while (gameIsLive) { 
        if(userInput.isDone()){
            try {
                commands = userInput.get().intValue();
            } catch (InterruptedException ex) {
              commands = NO_INPUT;
            } catch (ExecutionException ex) {
               commands = NO_INPUT;
            }
            userInput = executor.submit(new BilSpil());
        }

        switch(commands){
            case DOWN:
                y += MOVE_SPEED;
                break;
            case LEFT:
                x -= MOVE_SPEED;
                break;
            case UP:
                y -= MOVE_SPEED;
                break;
            case RIGHT:
                x += MOVE_SPEED;
                break;
        }
        cp.paintOval(x, y, BODY_WIDTH, BODY_HEIGHT);
        cp.repaintFrame();
        try{Thread.sleep(GAME_CLOCK);}
            catch (InterruptedException e) {}
    }
}

読み物:
http://docs.oracle.com/javase/tutorial/java/data/autoboxing.html http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ExecutorService .html
http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/Future.html

于 2013-10-01T22:10:41.987 に答える