論プロエディタを使ったコードをメソッドに分けたい

Question

前提

論プロエディタを利用してゲームを作っています。ネズミがプレイヤーでチーズに当たると得点が増えて、猫に当たると増えない、という風にしています。

実現したいこと

コード自体はすべて完成してコンパイルにも成功しています。このコードを可能な限りメソッドに分けて視覚的に簡潔にしたいです。

ウィンドウのサイズとワープのメソッド化。BGM作成のメソッド化。
上の二つについてはメソッドに分けられました。（以下、ここまでのコード）

public class CatAndRat extends Turtle {
	public static void main(  String[] args){
		Turtle.startTurtle(new CatAndRat(),args);
	}
	public void start() {
		System.out.println("CatAndRat: Version 8");
		hide();
//得点版生成
		int score = 0;
		TextTurtle scoreBoard = new TextTurtle("");
		scoreBoard.warp(600,50);
//画面サイズ設定
		window();
//キャラ生成
		ImageTurtle nezumi = new ImageTurtle("nezumi.png");
		ImageTurtle cat1 = new ImageTurtle("cat.png");
		ImageTurtle cat2 = new ImageTurtle("cat.png");
		ImageTurtle cat3 = new ImageTurtle("cat.png");
		ImageTurtle cat4 = new ImageTurtle("cat.png");
		ImageTurtle cheese1 = new ImageTurtle("cheese.png");
		ImageTurtle cheese2 = new ImageTurtle("cheese.png");
		ImageTurtle cheese3 = new ImageTurtle("cheese.png");
		ImageTurtle cheese4 = new ImageTurtle("cheese.png");
		ImageTurtle cheese5 = new ImageTurtle("cheese.png");
		ImageTurtle cheese6 = new ImageTurtle("cheese.png");
		ImageTurtle cheese7 = new ImageTurtle("cheese.png");
//音楽
		makesound();
		SoundTurtle hitc = new SoundTurtle("hitc.mp3");
		hitc.loadOnMemory();
		SoundTurtle hitcat = new SoundTurtle("hitcat.mp3");
		hitcat.loadOnMemory();
//初期値のワープ
		cat1.warp(random(640),0);
		cat2.warp(random(640),0);
		cat3.warp(random(640),0);
		cat4.warp(random(640),0);
		cheese1.warp(250,0);
		cheese2.warp(320,0);
		cheese3.warp(420,0);
		cheese4.warp(520,0);
		cheese5.warp(150,0);
		cheese6.warp(80,0);
		cheese7.warp(600,0);
		nezumi.warp(320,350);

		while(true){

			if(key() == 37){
				nezumi.warp(nezumi.x() - 5,nezumi.y());
			}else if(key() == 39){
				nezumi.warp(nezumi.x() + 5,nezumi.y());
			}
//待機
			sleep(0.025);
//ワープ先の設定
			cat1.warp(cat1.x(),cat1.y() + 5);
			cat2.warp(cat2.x(),cat2.y() + 5);
			cat3.warp(cat3.x(),cat3.y() + 5);
			cat4.warp(cat4.x(),cat4.y() + 5);
			cheese1.warp(cheese1.x(),cheese1.y() + 5);
			cheese2.warp(cheese2.x(),cheese2.y() + 5);
			cheese3.warp(cheese3.x(),cheese3.y() + 5);
			cheese4.warp(cheese4.x(),cheese4.y() + 5);
			cheese5.warp(cheese5.x(),cheese5.y() + 5);
			cheese6.warp(cheese6.x(),cheese6.y() + 5);
			cheese7.warp(cheese7.x(),cheese7.y() + 5);
//下まで行ったら上に行く
			if(cat1.y() > 480){
				cat1.warp(cat1.x(),0);
			}
			if(cat2.y() > 600){
				cat2.warp(cat2.x(),0);
			}
			if(cat3.y() > 500){
				cat3.warp(cat3.x(),0);
			}
			if(cat4.y() > 700){
				cat4.warp(cat4.x(),0);
			}
			if(cheese1.y() > 600){
				cheese1.warp(cheese1.x(),0);
			}
			if(cheese2.y() > 500){
				cheese2.warp(cheese2.x(),0);
			}
			if(cheese3.y() > 700){
				cheese3.warp(cheese3.x(),0);
			}
			if(cheese4.y() > 800){
				cheese4.warp(cheese4.x(),0);
			}
			if(cheese5.y() > 480){
				cheese5.warp(cheese5.x(),0);
			}
			if(cheese6.y() > 900){
				cheese6.warp(cheese6.x(),0);
			}
			if(cheese7.y() > 550){
				cheese7.warp(cheese7.x(),0);
			}

//ネズミが当たった時
			if(cat1.intersects(nezumi)){
				hitcat.play();
			}
			if(cat2.intersects(nezumi)){
				hitcat.play();
			}
			if(cat3.intersects(nezumi)){
				hitcat.play();
			}
			if(cat4.intersects(nezumi)){
				hitcat.play();
			}
			if(cheese1.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese2.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese3.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese4.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese5.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese6.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese7.intersects(nezumi)){
				score++;
				hitc.play();
			}
			scoreBoard.text(score);
			update();
		}
	}
	public	void window() {
		window.size(640,480);
		window.warp(100,100);
	}
	void makesound() {
		SoundTurtle bgm = new SoundTurtle("pakkuwoman.mp3");
		bgm.play();
	}
	
}

これ以外のコードをメソッドにできるだけ分けたいのですが、とりあえず以下のコードをメソッドに分けたくてやってみたところエラーが出ました。変数がうまくメソッドに渡せていないのだろうと思い、調べると「変数のデータ型　変数名」と書いてあったのでやってみたのですがエラーになりました。
この解決方法と、ほかにもメソッドに分けられる部分があれば教えていただきたいです。何か追記しなければならない部分等あればよろしくお願いします。
論プロエディタの物を質問されている方を見つけられなかったのですが、教えていただけると幸いです。初心者でわけわからないようなコードかもしれませんがよろしくお願いします。

該当のソースコード

メソッドに分けたい部分↓

if(key() == 37){
	nezumi.warp(nezumi.x() - 5,nezumi.y());
}else if(key() == 39){
	nezumi.warp(nezumi.x() + 5,nezumi.y());
}

変更したもの↓

public class CatAndRat extends Turtle {
	public static void main(  String[] args){
		Turtle.startTurtle(new CatAndRat(),args);
	}
	public void start() {
		System.out.println("CatAndRat: Version 8");
		hide();
//得点版生成
		int score = 0;
		TextTurtle scoreBoard = new TextTurtle("");
		scoreBoard.warp(600,50);
//画面サイズ設定
		window();
//キャラ生成
		ImageTurtle nezumi = new ImageTurtle("nezumi.png");
		ImageTurtle cat1 = new ImageTurtle("cat.png");
		ImageTurtle cat2 = new ImageTurtle("cat.png");
		ImageTurtle cat3 = new ImageTurtle("cat.png");
		ImageTurtle cat4 = new ImageTurtle("cat.png");
		ImageTurtle cheese1 = new ImageTurtle("cheese.png");
		ImageTurtle cheese2 = new ImageTurtle("cheese.png");
		ImageTurtle cheese3 = new ImageTurtle("cheese.png");
		ImageTurtle cheese4 = new ImageTurtle("cheese.png");
		ImageTurtle cheese5 = new ImageTurtle("cheese.png");
		ImageTurtle cheese6 = new ImageTurtle("cheese.png");
		ImageTurtle cheese7 = new ImageTurtle("cheese.png");
//音楽
		makesound();
		SoundTurtle hitc = new SoundTurtle("hitc.mp3");
		hitc.loadOnMemory();
		SoundTurtle hitcat = new SoundTurtle("hitcat.mp3");
		hitcat.loadOnMemory();
//初期値のワープ
		cat1.warp(random(640),0);
		cat2.warp(random(640),0);
		cat3.warp(random(640),0);
		cat4.warp(random(640),0);
		cheese1.warp(250,0);
		cheese2.warp(320,0);
		cheese3.warp(420,0);
		cheese4.warp(520,0);
		cheese5.warp(150,0);
		cheese6.warp(80,0);
		cheese7.warp(600,0);
		nezumi.warp(320,350);

		while(true){
			ke(ImageTurtle nezumi);
			//if(key() == 37){
				//nezumi.warp(nezumi.x() - 5,nezumi.y());
			//}else if(key() == 39){
				//nezumi.warp(nezumi.x() + 5,nezumi.y());
			//}
//待機
			sleep(0.025);
//ワープ先の設定
			cat1.warp(cat1.x(),cat1.y() + 5);
			cat2.warp(cat2.x(),cat2.y() + 5);
			cat3.warp(cat3.x(),cat3.y() + 5);
			cat4.warp(cat4.x(),cat4.y() + 5);
			cheese1.warp(cheese1.x(),cheese1.y() + 5);
			cheese2.warp(cheese2.x(),cheese2.y() + 5);
			cheese3.warp(cheese3.x(),cheese3.y() + 5);
			cheese4.warp(cheese4.x(),cheese4.y() + 5);
			cheese5.warp(cheese5.x(),cheese5.y() + 5);
			cheese6.warp(cheese6.x(),cheese6.y() + 5);
			cheese7.warp(cheese7.x(),cheese7.y() + 5);
//下まで行ったら上に行く
			if(cat1.y() > 480){
				cat1.warp(cat1.x(),0);
			}
			if(cat2.y() > 600){
				cat2.warp(cat2.x(),0);
			}
			if(cat3.y() > 500){
				cat3.warp(cat3.x(),0);
			}
			if(cat4.y() > 700){
				cat4.warp(cat4.x(),0);
			}
			if(cheese1.y() > 600){
				cheese1.warp(cheese1.x(),0);
			}
			if(cheese2.y() > 500){
				cheese2.warp(cheese2.x(),0);
			}
			if(cheese3.y() > 700){
				cheese3.warp(cheese3.x(),0);
			}
			if(cheese4.y() > 800){
				cheese4.warp(cheese4.x(),0);
			}
			if(cheese5.y() > 480){
				cheese5.warp(cheese5.x(),0);
			}
			if(cheese6.y() > 900){
				cheese6.warp(cheese6.x(),0);
			}
			if(cheese7.y() > 550){
				cheese7.warp(cheese7.x(),0);
			}

//ネズミが当たった時
			if(cat1.intersects(nezumi)){
				hitcat.play();
			}
			if(cat2.intersects(nezumi)){
				hitcat.play();
			}
			if(cat3.intersects(nezumi)){
				hitcat.play();
			}
			if(cat4.intersects(nezumi)){
				hitcat.play();
			}
			if(cheese1.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese2.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese3.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese4.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese5.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese6.intersects(nezumi)){
				score++;
				hitc.play();
			}
			if(cheese7.intersects(nezumi)){
				score++;
				hitc.play();
			}
			scoreBoard.text(score);
			update();
		}
	}
	public	void window() {
		window.size(640,480);
		window.warp(100,100);
	}
	void makesound() {
		SoundTurtle bgm = new SoundTurtle("pakkuwoman.mp3");
		bgm.play();
	}

	void ke(ImageTurtle nezumi) {
		if(key() == 37){
			nezumi.warp(nezumi.x() - 5,nezumi.y());
		}else if(key() == 39){
			nezumi.warp(nezumi.x() + 5,nezumi.y());
		}
	}
}

以下、エラーメッセージ
CatAndRat.java:53: エラー: ')'がありません
ke(ImageTurtle nezumi);
^
CatAndRat.java:53: エラー: 文ではありません
ke(ImageTurtle nezumi);
^
CatAndRat.java:53: エラー: ';'がありません
ke(ImageTurtle nezumi);
^
エラー3個

Comments

[jimbe]

>論プロエディタの物を質問されている方を見つけられなかった

論プロエディタ自体がプログラミング教育環境として開発されたもので、使われているのも一部の大学だけなのではないでしょうか。
フリーソフトみたいですが、 google で探しても数件しか記事が出てきませんでしたし。

Answer 1

ke(ImageTurtle nezumi);

メソッドの呼び出しに変数の型は書きませんが…。

ke(nezumi);

では？

---

一番行数を取っている cat や cheese の処理をまずメソッドにするべきでしょう。
その際、「//キャラ生成」「//初期値のワープ」とか「//ワープ先の設定」「//下まで行ったら上に行く」といった、処理毎に各オブジェクトを使うのではなく、 cat1 の生成とワープの設定、 cat2 の生成とワープの設定、・・・、cat1 のワープ先の設定と下まで行ったら上に行く処理、 cat2 のワープ先の設定と下まで行ったら上に行く処理、・・・と各オブジェクト毎に処理を纏めると、それがメソッドとして分ける単位に出来ます。
そして、配列を習っていれば、 cat や cheese を配列にして for で回すことが出来ます。
以下は、こちらに実行環境が無いのでコンパイルだけ通したコードです。

public class CatAndRat extends Turtle {
  public static void main(String[] args) {
    Turtle.startTurtle(new CatAndRat(), args);
  }

  private int score = 0;

  public void start() {
    System.out.println("CatAndRat: Version 8");
    hide();
    //得点版生成
    //int score = 0;
    TextTurtle scoreBoard = new TextTurtle("");
    scoreBoard.warp(600,50);
    //画面サイズ設定
    window();
    //キャラ生成
    ImageTurtle nezumi = new ImageTurtle("nezumi.png");
    nezumi.warp(320,350);

    int[] catYMaxs = { 480, 600, 500, 700 };
    ImageTurtle[] cats = new ImageTurtle[catYMaxs.length];
    for(int i=0; i<cats.length; i++) {
      cats[i] = createCat();
    }

    int[] cheesesInitXs = { 250, 320, 420, 520, 150, 80, 600 };
    int[] cheesesYMaxs = { 600, 500, 700, 800, 480, 900, 550 };
    ImageTurtle[] cheeses = new ImageTurtle[cheesesInitXs.length];
    for(int i=0; i<cheeses.length; i++) {
      cheeses[i] = createCheese(cheesesInitXs[i]);
    }
    //音楽
    makesound();
    SoundTurtle hitc = createSound("hitc.mp3");
    SoundTurtle hitcat = createSound("hitcat.mp3");

    while(true){
      ke(nezumi);

      //待機
      sleep(0.025);

      for(int i=0; i<cats.length; i++) {
        moveCat(cats[i], catYMaxs[i], nezumi, hitcat);
      }

      for(int i=0; i<cheeses.length; i++) {
        moveCheese(cheeses[i], cheesesYMaxs[i], nezumi, hitc);
      }

      scoreBoard.text(score);
      update();
    }
  }

  public void window() {
    window.size(640, 480);
    window.warp(100, 100);
  }

  void makesound() {
    SoundTurtle bgm = new SoundTurtle("pakkuwoman.mp3");
    bgm.play();
  }

  void ke(ImageTurtle nezumi) {
    if (key() == 37) {
      nezumi.warp(nezumi.x() - 5, nezumi.y());
    } else if (key() == 39) {
      nezumi.warp(nezumi.x() + 5, nezumi.y());
    }
  }

  private ImageTurtle createCat() {
    ImageTurtle cat = new ImageTurtle("cat.png"); //キャラ生成
    cat.warp(random(640),0); //初期値のワープ
    return cat;
  }

  private ImageTurtle createCheese(int x) {
    ImageTurtle cheese = new ImageTurtle("cheese.png"); //キャラ生成
    cheese.warp(x,0); //初期値のワープ
    return cheese;
  }

  private void moveCat(ImageTurtle cat, int ymax, ImageTurtle nezumi, SoundTurtle hitcat) {
    cat.warp(cat.x(),cat.y() + 5); //ワープ先の設定
    if(cat.y() > ymax){ //下まで行ったら上に行く
      cat.warp(cat.x(),0);
    }
    if(cat.intersects(nezumi)){ //ネズミが当たった時
      hitcat.play();
    }
  }

  private void moveCheese(ImageTurtle cheese, int ymax, ImageTurtle nezumi, SoundTurtle hitc) {
    cheese.warp(cheese.x(),cheese.y() + 5); //ワープ先の設定
    if(cheese.y() > ymax){ //下まで行ったら上に行く
      cheese.warp(cheese.x(),0);
    }
    if(cheese.intersects(nezumi)){ //ネズミが当たった時
      score++;
      hitc.play();
    }
  }

  private SoundTurtle createSound(String filename) {
    SoundTurtle sound = new SoundTurtle(filename);
    sound.loadOnMemory();
    return sound;
  }
}

Comments

[yuyy]

呼び出しに変数の型は書かないのですね！修正してコンパイルしたところ成功しました。ありがとうございました！
複数の変数を渡す場合はかっこの中を何かで区切らないといけないのでしょうか。

[jimbe]

複数の引数は "," 区切りです。

cat1.warp(random(640),0);

等と使っていますよね。

これらは java の基本中の基本であって、入門書・サイト等を見れば必ず書いてあると思います。
わざわざ時間の掛かるQAサイトよりも、入門サイトを検索して読みまくるほうが他のことも知られて良いのではないでしょうか。

[yuyy]

すみません、サイトなどを見ても理解できなくて質問しました。回答ありがとうございました。

[jimbe]

Turtle のメソッドの呼び出しは問題無く出来ているのですから、何か難しく考えすぎているのかもしれませんね。