質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

コードの改良余地について。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,564

score 17

下記にあるコードは
四角形を9個作り、点が存在する四角形を青で塗り潰すコードなんですが、
一応これで思った通りに実行されます。

ですが、オブジェクト指向ではないような気がします。
何かもっと簡潔にコードをかけないでしょうか?

初心者なのでお手柔らかにお願いします。

package 問題;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.Rectangle;
import javax.swing.JFrame;



public class CD591216Fujikake118 extends JFrame{

    private static final long serialVersionUID = 1L;
    Rectangle rec = new Rectangle(70,70,70,70);    // 四角形
    Point p = new Point(0,0);                    // 座標

    public CD591216Fujikake118(){
        setTitle("課題4");
        setSize(360,340);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
    }

    /*** paint():ウィンドウに描画するコードを記述する ***/
    public void paint(Graphics g){
        super.paint(g);
        //円の座標の設定(x,y)
        int ran1=(int)(Math.random()*210+70);
        int ran2=(int)(Math.random()*210+70);
        p.setLocation(ran1, ran2);
        
        //九つの 四角形の描画
        for(int j=0;j<210;j+=70){
            for(int i=0;i < 210;i += 70){
                
                g.drawRect(rec.x+i, rec.y+j, rec.width, rec.height);
                
                
                //該当する四角を塗りつぶす
                if(rec.x+i < p.x & rec.x+i+70 > p.x & 
                   rec.y+j < p.y & rec.y+j+70 > p.y){
                    g.setColor(Color.blue);
                    g.fillRect(rec.x+i, rec.y+j, rec.width, rec.height);
                    g.setColor(Color.black);
                }//Point!もう一度黒に戻す!    
            }
        }
        //円の描写
        g.fillOval(p.x, p.y, 10, 10);
    }
    
    /*** main ***/
    public static void main(String[] args){
        JFrame f = new CD591216Fujikake118();
        f.setVisible(true);
    }
}
ちなみにこの部分です。
//該当する四角を塗りつぶす
                if(rec.x+i < p.x & rec.x+i+70 > p.x & 
                   rec.y+j < p.y & rec.y+j+70 > p.y){
                    g.setColor(Color.blue);
                    g.fillRect(rec.x+i, rec.y+j, rec.width, rec.height);
                    g.setColor(Color.black);
                }//Point!もう一度黒に戻す!    
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+2

このままで良いと思います。
オブジェクト指向の「指向」はあくまでも目指す方向であり、
目的が達せられているなら無理に実装することはない・・・と昔言われました。

オブジェクト指向とは別ですが、四角形の辺の長さの70という数字は
いわゆるマジックナンバーですので定数としてもいいかなと思います。

private static final int SIDE_LENGTH = 70;

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

このままでもよいとは思います。

もっとオブジェクト指向っぽくするなら、自分なら
「四角をつくって塗る処理をクラス化・メソッド分割」
して、外から呼び出すようにするかな。
そしたら、呼び出し側の記述はすっきりするし、
他の場所で必要になったときに、インスタンスをつくって呼び出すだけでできるから。

あくまで、個人的な考え・・・ってことで参考程度に思っていただけたら幸いです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

オブジェクト指向ではありませんが、コードを改良してみました。
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.Rectangle;
import javax.swing.JFrame;


public class NineBox extends JFrame{
    
    private static final long serialVersionUID = 1L;
    private static final Point origin = new Point(70, 70);
    private static final Dimension size = new Dimension(70, 70);
    private final Rectangle[] recs = new Rectangle[9];
    private Point p = new Point(0,0);
    
    
    public NineBox(){
        setTitle("課題4");
        setSize(360,340);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        
        for(int i = 0; i < 9; i++) {
            int x = i%3*size.width+origin.x;
            int y = i/3*size.height+origin.y;
            int width = size.width;
            int height = size.height;
            recs[i] = new Rectangle(x, y, width, height);
        }
        
        randomMove();
    }
    
    
    public void randomMove() {
        int x=(int)(Math.random()*size.width*3+origin.x);
        int y=(int)(Math.random()*size.height*3+origin.y);
        p.setLocation(x, y);
    }
    
    
    public void paint(Graphics g){
        super.paint(g);
        
        for (Rectangle rec : recs) {
            g.setColor(Color.black);
            g.drawRect(rec.x, rec.y, rec.width, rec.height);
            if (rec.contains(p)) {
                g.setColor(Color.blue);
                g.fillRect(rec.x, rec.y, rec.width, rec.height);
            }
        }
        
        g.setColor(Color.black);
        g.fillOval(p.x, p.y, 10, 10);
    }
    
    
    public static void main(String[] args){
        JFrame f = new NineBox();
        f.setVisible(true);
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/07/16 11:46

    回答ありがとうございます!
    すごいきれいになっていて驚きました!
    参考にさせて頂きます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る