常連

注目の的

社会貢献

質問力

自己紹介

###前提
[オブジェクト指向を用いたテトリスの設計](https://teratail.com/questions/56188)
ここで質問させていただいた者です。
この続きの質問です。これは見ても見なくても結構です。

ブロック４つを一まとまりにしたものをテトリミノと呼びます。
テトリミノは7種類あります。
テトリミノには回転軸がブロックの中心に存在するものと、交差点に存在する物の２種類あり、以下の図の赤い部分が回転軸です。
![イメージ説明](30c6885fcda504317218bea73069de10.png)

###質問内容
テトリミノのクラスの設計はどの様にするのが上手な設計でしょうか。

私は以下の様にテトリミノが存在する座標と、ミノの形を持つ二次配列を持たせる設計をしていました。
```java
int x = 3, y = 1;
boolean[][] shape = new boolean[][]
	{ { true , true , false },			//	■■□
	  { false, true , true  },			//	□■■
	  { false, false, false } };		//	□□□
```
この設計だと、例えば右回転をさせたいなら以下の様なアルゴリズムで簡単に処理することができます。
```java
//  Boolean2次配列をコピー
private boolean[][] copy( boolean[][] args )
{
	boolean[][] ret = new boolean[args.length][args[0].length];
	for( int x=0 ; x< args   .length ; x++ ){
	for( int y=0 ; y< args[0].length ; y++ ){
		ret[y][x] = args[y][x];
	}
	}

	return ret;
}

//  右回転
void rotateRight()
{
	boolean[][] temp = copy( shape );
	for( int y = 0 ; y < shape.length ; y++ ){
	for( int x = 0 ; x < shape.length ; x++ ){
		shape[y][x] = temp[shape.length -x -1]y];
	}
	}
}
```
しかし、オブジェクト指向的な設計を目指すなら、ブロックを４つ持っていると言う方が良いですよね。

ブロックが保持する内容は以下の二通りが考えられますが、いずれも問題点があり、私には良い解決方法が見当たりません。

1.フィールドの中のどこに位置しているかを表す座標
この場合、回転させるという処理を書くのがすごく大変です。
2.回転軸からの相対座標(この場合、テトリミノが回転軸の座標を保持)
この場合、回転させる処理は簡潔に書けそうですが、回転軸がブロックの中心にあるものと、交差点にあるものにより、保持する内容が異なってしまいます。

いずれかの良い問題解決の方法、それがないならこの二つ以外に良い方法があれば教えてください。

テトリス クラス設計の考え方

前回と似た質問で申し訳ないですが、わからないので質問させていただきます。
下のプログラムはシューティングゲームを作っている途中のプログラムです。

```ここに言語を入力
コード
public class Part01 extends JFrame{

	/**
	 * メインクラス
	 */
	public static void main(String args[]){
		new Part01();
	}

	/**
	 * コンストラクタ
	 */
	public Part01(){

		/**フレームの初期化*/
		super("Shooting Game");
		setBounds(0, 0, 500, 500);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(true);

		/**パネルの生成*/
		MyPanel myPanel = new MyPanel();
		add(myPanel);

		/**初期化を行う*/
		myPanel.initialization();
	}

}


public class MyPanel extends JPanel implements Runnable{

	Title title;		//Titleクラス変数
	Score score;		//Scoreクラス変数

	/**
	 * コンストラクタ
	 */
	public MyPanel(){
		/*title = new Title();
		score = new Score();*/

		Thread thread = new Thread(this);
		thread.start();
	}

	/**
	 * 初期化
	 */
	public void initialization(){
		//score.initMyScore();
	}

	/**
	 * メインループ
	 */
	@Override
	public void run(){

		while(true){
            System.out.println("12345");	
			repaint();

			try{
				Thread.sleep(500);
			}
			catch(InterruptedException e){
			}
		}
	}

	/**
	 * 描画
	 */
	@Override
	protected void paintComponent(Graphics g){
		super.paintComponent(g);
			
     System.out.println("6789");	
		g.setColor(Color.RED);
		g.fillRect(100, 100, 100, 100);
	}

}


```
プログラム内容
Part01クラスでフレームを作り、そこにパネルを貼り付けている。
0.5秒ごとに再描画をして、赤い四角形を描画している。


今MyPanelクラスのコンストラクタでScoreクラスとTitleクラスをインスタンス化したのをコメントアウトしています。
これで実行すると四角形がきちんと描画され、0.5秒ごとに”12345”と”6789”も表示されます。

次にコメントアウトをやめて
title = new Title();
score = new Score();
を記述した状態で実行すると四角形と”6789”は表示されず、”12345”だけ0.5秒ごとに表示されました。つまりpaintComponentメソッドが動いていませんでした。
フレームをいじると四角形も”6789”も表示されます。

下にScoreクラスとTitleクラスを載せておきますが、これをインスタンス化しただけでpaintComponentメソッドが呼ばれなくなるのはどうしてなのでしょうか？
ScoreクラスとTitleクラスにpaintComponentメソッドを妨げるような記述があるからだと予想はできるのですが原因が何かわかりません。
わかる方がいればよろしくお願いします!





ScoreクラスとTitleクラス
```ここに言語を入力
コード
public class Score{

	int myScore;		//スコア
	int highScore;		//ハイスコア

	Color scoreColor;	//スコア表示の色
	Font scoreFont;		//スコア表示のフォント

	/**
	 * コンストラクタ
	 */
	public Score(){
		myScore = 0;
		highScore = 0;

		scoreColor = Color.BLACK;
		scoreFont = new Font(Font.MONOSPACED, Font.BOLD, 10);
	}

	/**
	 * MyScoreのセッター
	 */
	public void setMyScore(int myScore){
		this.myScore = myScore;
	}

	/**
	 * MyScoreのゲッター
	 */
	public int getMyScore(){
		return myScore;
	}

	/**
	 * 得点加算
	 * @param point		追加する得点
	 */
	public void addMyScore(int point){
		myScore += point;
	}

	/**
	 * highScore更新
	 */
	public void cmpareScore(){
		if(myScore > highScore){
			highScore = myScore;
		}
	}

	/**
	 * MyScore描画
	 * @param g	グラフィック変数
	 */
	public void drawMyScore(Graphics g){
		g.setColor(scoreColor);
		g.setFont(scoreFont);
		g.drawString("score:" + myScore, 30, 30);
	}

	/**
	 * highScore描画
	 * @param g	グラフィック変数
	 */
	public void drawHighScore(Graphics g){
		g.setColor(scoreColor);
		g.setFont(scoreFont);
		g.drawString("highScore:" + highScore, 420, 30);
	}

	/**
	 * MyScoreの初期化
	 */
	public void initMyScore(){
		myScore = 0;
	}

}

public class Title{

	Color titleColor;		//タイトル表示の色
	Font mainTitleFont;		//メインタイトル表示のフォント
	Font subTitleFont;		//サブタイトル表示のフォント

	int blinkCount;			//アニメーション用点滅カウンタ

	/**
	 * コンストラクタ
	 */
	public Title(){
		titleColor = Color.BLACK;
		mainTitleFont = new Font(Font.MONOSPACED, Font.BOLD, 30);
		subTitleFont = new Font(Font.MONOSPACED, Font.BOLD, 15);
	}

	/**
	 * ゲームスタートタイトル表示
	 * @param g	グラフィック変数
	 */
	public void drawGameStartTitle(Graphics g){
		g.setColor(titleColor);
		g.setFont(mainTitleFont);
		g.drawString("Shooting", 150, 150);

		blinkCount++;

		/**点滅処理(偶数の場合のみ表示)*/
		if(blinkCount % 2 == 0){
			g.setFont(subTitleFont);
			g.drawString("hit SPACE key", 200, 350);
		}
	}

	/**
	 * ゲームオーバータイトル表示
	 * @param g	グラフィック変数
	 */
	public void drawGameOverTitle(Graphics g){
		g.setColor(titleColor);
		g.setFont(mainTitleFont);
		g.drawString("GAMEOVER", 150, 150);
	}

}


```

paintComponentメソッドについて

オブジェクト指向を意識したことがないのでオブジェクト指向の設計をしようと学習していますが、なんだかよくわからなくなってきました。

例えば一月分の予定を入力するカレンダーのようなプログラムを作る場合、
一ヶ月分のカレンダーに、日付ごとの入力フォームをつくり、入力された内容をDBに保持するプログラムとします。
DBのレコードは1日ひとつ、項目には1日単位の年月日、イベント名、起床時間、持ち物、予算などがあり今後も増える可能性があるとします。


このケースでは入力画面からDBに書き込むSQL文を発行するまでに、日付単位で管理するべき情報をもったオブジェクトが毎月30個ほど作られることになりますよね？
となるとそれらのオブジェクトを生成するために引数として入力画面から5つのデータを渡すことになると思います。

引数5つでもプログラムとして多すぎる気がしますし、今後も増えていくとしたらただ羅列するのは可読性が悪いので他の方法を取るべきだと思います。
でもセッターでフィールドに逐一値を入れていくのも不自然で無駄が多い気がしますし、
配列などに入れてデータを渡すのはオブジェクト指向らしくない気がして、どうすればいいのかわからなくなってきました。

この例は素人が適当に作ったので根本的に考え方がおかしいところもあるかもしれませんが、
オブジェクトが管理するべき引数が多くなることってありますよね？
そういったときはどう対処すればいいのでしょうか。






また、DBからデータを読み込んで表示するときには、この日付ごとのデータを保持したオブジェクトから画面描画オブジェクトにインスタンス変数を渡すことになると思うのですが、
この場合にはフィールドをpublicにして直接参照するか、全ての変数にgetterを設定して逐一渡していくか、配列などに入れてまとめて渡す方法しか浮かびません。

フィールドがpublicなのは良くないと聞きますし、getterを使うべきではないという意見も見ます。
配列は求めているデータがどこに格納されているのか識別できず、配列を作ったクラスを閲覧しなければいけないのでカプセル化に失敗している気がします。

何かいいやり方はないのでしょうか。

javaで、引数が多すぎるときの対処法

Javaで実装するときに、以下のようにフィールドをprivateにしてsetterとgetterを実装するということがあるかと思います。

```lang-＜Java＞
public class Product {
	private String name;
	private Integer price;
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public Integer getPrice() {
		return price;
	}
	public void setPrice(Integer price) {
		this.price = price;
	}
}
```

最初にこれを見たときは、わざわざフィールドをprivateにしてsetterとgetterを実装する意味が分からず、以下のようにフィールドをpublicで実装してしまった方がコードも短くなって読みやすいのではないかと思いました。

```lang-＜Java＞
public class Product {
	public String name;
	public Integer price;
}
```

その後も、Javaでsetterとgetterを実装する理由について、周りのエンジニアの方々に質問したり、ググったりしてみたのですが、「Javaの仕様だから」、「オブジェクト指向だから」というような回答が多く、なかなか納得できる答えに出会えませんでした。

今では、私自身もなんとなくsetterとgetterを実装する習慣が付いてしまっているのですが、それでもsetterとgetterを実装する実質的なメリットはよく分かっていません。

この辺りの話題に詳しい方がいらっしゃったら、setterとgetterの意義について教えて頂けるとありがたいです。

よろしくお願い致します。