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

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

ただいまの
回答率

90.38%

  • Java

    16465questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Swing

    251questions

    SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。

マウスクリックのたびに少しずつ大きい円を描画したいのですが…

解決済

回答 1

投稿 編集

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

kazz0117

score 10

実現したいこと

Javaでマウスクリックのたびにマウス座標を中心として円を描くプログラムを作成しています。static変数を使って描かれる円が少しずつ大きくなっていくようにしたいのです(イメージとしては・→ o → O → 〇)、

発生している問題・エラーメッセージ

ところが過去に描いた円も一緒に大きくなってしまいます。
どこを修正すればいいか分からなくなってしまいました。

ソースコード

Java
Circle.java::
import java.awt.Color;
import java.awt.Graphics;

public class Circle
{
    private int x;
    private int y;

    private static int radius = 0;

    public Circle(int x, int y)
    {
        this.x = x;
        this.y = y;
        radius++;
    }
    public void draw(Graphics g)
    {
        g.setColor(Color.red);
        g.drawOval(x - radius / 2, y - radius / 2, radius, radius);
    }
}

Sample4.java::
import java.awt.BorderLayout;
import java.awt.Graphics;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.ArrayList;
import java.util.Iterator;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class Sample4 extends JFrame
{
    private SamplePanel sp;

    public static void main(String[] args)
    {
        Sample4 sm = new Sample4();
        sm.setVisible(true);
    }
    public Sample4()
    {
        super("円を描画");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(500, 500);
        sp = new SamplePanel();
        add(sp, BorderLayout.CENTER);
    }
    public class SamplePanel extends JPanel
    {
        private ArrayList<Circle> circlelist = new ArrayList<Circle>();

        public SamplePanel()
        {
            addMouseListener(new SampleMouseListener());
        }
        public void paint(Graphics g)
        {
            super.paint(g);;
            Iterator<Circle> it = circlelist.iterator();
            while(it.hasNext()) {
                Circle c = it.next();
                c.draw(g);;
            }
        }
        public class SampleMouseListener extends MouseAdapter
        {
            public void mousePressed(MouseEvent e)
            {
                circlelist.add(new Circle(e.getX(), e.getY()));
                repaint();
            }
        }
    }
}

補足情報(言語/FW/ツール等のバージョンなど)

ご回答ありがとうございます。うまくいきました。
Circle.java::
import java.awt.Color;
import java.awt.Graphics;

public class Circle
{
    private int x;
    private int y;
    private int radius;

    public Circle(int x, int y, int radius)
    {
        this.x = x;
        this.y = y;
        this.radius = radius;
    }
    public void draw(Graphics g)
    {
        g.setColor(Color.red);
        g.drawOval(x - radius / 2, y - radius / 2, radius, radius);
    }
}

Sample4.java::
import java.awt.BorderLayout;
import java.awt.Graphics;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.ArrayList;
import java.util.Iterator;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class Sample4 extends JFrame
{
    private SamplePanel sp;

    public static void main(String[] args)
    {
        Sample4 sm = new Sample4();
        sm.setVisible(true);
    }
    public Sample4()
    {
        super("円を描画");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(500, 500);
        sp = new SamplePanel();
        add(sp, BorderLayout.CENTER);
    }
    public class SamplePanel extends JPanel
    {
        private int radius = 0;

        private ArrayList<Circle> circlelist = new ArrayList<Circle>();

        public SamplePanel()
        {
            addMouseListener(new SampleMouseListener());
        }
        public void paint(Graphics g)
        {
            super.paint(g);;
            Iterator<Circle> it = circlelist.iterator();
            while(it.hasNext()) {
                Circle c = it.next();
                c.draw(g);
            }
        }
        public class SampleMouseListener extends MouseAdapter
        {

            public void mousePressed(MouseEvent e)
            {
                circlelist.add(new Circle(e.getX(), e.getY(), radius++));
                repaint();
            }
        }
    }
}

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

過去に描画した円はそのままのサイズで、新しく追加した円だけを少しずつ大きくする。

ということであれば、過去の円のサイズは保持し続けなければなりませんよね。static で持ってしまうと、(個別ではなく)今まで作ったのも含めて全ての Circle クラスで共通した値として扱われてしまいます。そのため、今回のような目的においては Circle クラスの radius は static で持つべきではありません。

解決策としては Circle クラスのフィールドとしてサイズを持たせて、x や y と同様にコンストラクタに初期値を渡して設定されるようにすると良いでしょう。

次に Sample4 クラスあるいは SamplePanel クラスのフィールドとして円に設定するサイズを持っておき、Circle クラスを追加する度に値を大きくしていけば良いでしょう。別段ここでもサイズは static で持つ必要はありません。

以上、ご参考になれば幸いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/01/18 22:57

    明快なご回答ありがとうございます。すぐに修正版を作成でき、動作も思い通りになりました!!

    キャンセル

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

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

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

  • Java

    16465questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Swing

    251questions

    SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。