6
本当にMainFrame frameが使用されていない(使用しない)変数であれば、変数に受ける必要はありません。
下記のように書けます。
public class Main {
public static void main(String[] args){
new MainFrame();
}
}
(つまりIDEの指摘は正しいです)
4
Swingのコンポーネントを使用する場合、描画の処理はすべてイベントディスパッチスレッド(AWTスレッド)で行うようにすべきです。そうでない場合、ただしく動作しない可能性があります。
mainメソッドだけで言えば、下記のように、SwingUtilities.invokeLaterを使って、JFrameを表示するコードがAWTスレッドで実行されるようにします。
public static void main(String[] args) {
final Main app = new Main();
SwingUtilities.invokeLater(new Runnable() {
public void run() {
// TODO ...
3
各TextFieldのどれにファイルがドロップされたかを判別したいのです。
TransferHandler.TransferSupport#getComponent()メソッドで、転送のターゲット・コンポーネントを取得することができます。
import java.awt.*;
import java.awt.datatransfer.*;
import java.io.*;
import java.util.Objects;
import java.util.List;
import javax.swing.*;
public class Hoge2 {
public JComponent makeUI() {
int n = 4;
JPanel p = new JPanel(new ...
3
すでに argius さんの回答にあるイベントディスパッチスレッド関係の問題と、Threadを使うより、javax.swing.Timerを使用したほうが良いのでは?という二点が気になりますが、JPanelが黒のままになっている直接の原因は、作成したbufferをJPanelに描画していないからだと思います。
以下のように、JPanel#paintComponent(...)をオーバーライドしてテストしてみてください。
public class Main extends JPanel implements Runnable{
@Override public void paintComponent(Graphics g) {
super.paintComponent();
g....
2
BI2DPv#repaint()がどう呼ばれているのかが分からないので推測ですが、そのJPanelがJFrame、またはJAppletなどのトップレベルコンテナに追加されていないのではないでしょうか。
参考: Top-Level Containers and Containment Hierarchies (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)
例えば、以下のサンプルコードでは、一秒ごとにbi2dpv.repaint();を実行していますが、return bi2dpv;をコメントアウトして、return new JPanel();を実行し、JFrameにbi2dpvを追加しないようにした場合、...
2
class Main{
public static void main(String[] args){
@SuppressWarnings("unused")
MainFrame frame = new MainFrame();
}
}
のように@SuppressWarnings("unused")をつければ大概のIDE(eclipse、ideaなど)では抑止できると思います。
2
以下で述べる事項で全て解決するわけではありませんが、取り敢えずJava的に指摘できる事項を2点記載します。(要件的な話は分からないのでここでは記載していません。)
1.
「ゲームオーバー画面」を表示させようとしている直接のコード、MyPanel2#setPanel3()
public void setPanel3(){
MyPanel3 p3 = new MyPanel3();
add(p3);
}
ですが、ここで呼んでいるaddメソッドのJavaDocはこちらになります。
(前略)このメソッドではレイアウトに関連する情報が変更されるため、コンポーネント階層が無効になります。コンテナがすでに表示されている場合は、追加されたコンポーネントを表示するために、...
1
(このコードはSwingでなくVaadinではないでしょうか?)
省略されているコードの方に問題があるように見えるので、現在の質問文からは何が起こっているのか分かりません。
(ですので質問文を見直す必要があるかと考えます。)
getCmbMode()の中でaddValueChangeListenerで新しいValueChangeListenerを毎回追加しているように見えますが、これは明らかにおかしいです。
1
frame.setVisible(true); を最後にもってくれば描画された状態で表示されると思います。
また、frame.pack() の実行も望まれているかと思われます。
ただし根本的な原因としては、イベントディスパッチスレッドで実行していないことです。
//This is a method to stick png file to JFrame
SwingUtilities.invokeLater(() -> {
ZTestCalculation frame = new ZTestCalculation();
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame....
1
MouseAdapterと無名クラスを使うのがシンプルです。
import java.awt.CardLayout;
import java.awt.Container;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
public class Layout2 extends JFrame {
JLabel label1;
JLabel label2;
JLabel label3;
CardLayout layout;
...
1
動くコード
MainFrame.java
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.*;
public class MainFrame extends JFrame {
PlayView pv = new PlayView();
public MainFrame() {
this.add(pv.panel);
pv.setVisible(true);
this.setBounds(200, 100, 800, 600);
this.addKeyListener(pv);
}
...
1
まず、 paintComponent(Graphics g) で渡された Graphics オブジェクトを保存しておいて、paintComponent(Graphics g) を抜けた後に、呼び出してはいけません。ほかのクラスに描画処理を委譲したい場合は paintComponent() から Graphics オブジェクトを渡してメソッドを呼び出しましょう。
timer は Circle の状態(アニメーションのフレーム)を更新して repaint() を呼び出す
Circle のアニメーション処理が終わったら timer を停止
repaint() の呼び出しにより paintComponent(Graphics g)が呼び出される
paintComponent(Graphics g) から Circle#...
1
JTextField (Java Platform SE 8 )の冒頭から引用:
JTextComponentベースのコンポーネントでは、DocumentEvent経由でモデルからDocumentListenersに変更が送られます。DocumentEventは、必要に応じて、変更位置と変更の種類を提供します。この部分のコードは次のようになります。
DocumentListener myListener = ??;
JTextField myArea = ??;
myArea.getDocument().addDocumentListener(myListener);
とあるように、JTextComponent#getDocument()メソッドを使用すると良さそうです。
//...
1
//セルの描画範囲を計算
startRow = visibleRect.y / getCellHeight();
endRow = startRow + (visibleRect.height / getCellHeight());
startCol = visibleRect.x / getCellWidth();
endCol = startCol + (visibleRect.width / getCellWidth());
おそらく、/で割った時の余りが切り捨てられることで、誤差が出ているのではないかと思います。割り切れない場合(visibleRect.height % getCellHeight() != 0)は、+1するとかで誤差は無くなりませんか?
1
以下で引用しているSwingのチュートリアルに記述されているように、イベント・ディスパッチ・スレッド(EDT)内でSwingオブジェクトのメソッドを呼ぶというルールを無視していると、たまに変な挙動(JFrameのサイズが0になったり、表示位置がズレたりなど)をしたり、再現不可なエラーが発生したりします。
Lesson: Concurrency in Swing (The Java™ Tutorials > Creating a GUI With JFC/Swing)
The Event Dispatch Thread
Swing event handling code runs on a special thread known as the event dispatch thread. ...
1
ActionListener(addProduct)内で、新規の別JFrame(registration)を作成して、getTextField()を呼び出しているせいで、常に初期値が表示されています。
例えば、addProductクラスをregistrationクラスの内部クラスにするなどして、現在表示しているregistrationのgetTextField()メソッドを呼び出すように修正してはどうでしょうか。
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.border.EmptyBorder;
public class registration extends JFrame {
...
1
/**
* キーワードハイライトを範囲ハイライトで上書きする
* @param content
* @param offset
* @param length
*/
private void overrideHighlighting(String content, int offset) {
ざっと見た印象では、overrideHighlighting(...)のwhileループが重そうなので、未使用?の@param lengthを利用したり、文字入力された位置がコメントの範囲ハイライト外かどうかなどを判断して、中にあるmultiTokenHighlight(...)を呼ぶ回数を減らせば速くなりそうな気がします。
上位に投票された、最小文字数のコミュニティ wiki 以外の回答のみが対象となります