javafxでメソッドの中でtheStgaeを使えるようにしたい

Question

###前提・実現したいこと
NetBeans上でメソッドだけページを分けて書きたい
自作関数（メソッド）Startup()内でtheStgaeを使いたい
です
プログラム自体初心者なもので初歩的な質問すみません

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

package novelgame;

import java.util.ArrayList;
import javafx.animation.AnimationTimer;
import javafx.application.Application;
import javafx.event.EventHandler;
import javafx.scene.Group;
import javafx.scene.Scene;
import javafx.scene.input.KeyEvent;
import javafx.scene.input.MouseEvent;
import javafx.stage.Stage;

public class Setup extends Application 
{
    int num = 1;
    double mw,mv,pw,pv= 0;
    ArrayList<String> keyarry = new ArrayList<String>();
    public Stage theStage = new Stage();
    String a = "q";
    
    public static void main(String[] args) 
    {
             launch(args);
    }
    
    @Override public void start(Stage theStage)
    {
        Group root = new Group();
        Scene theScene = new Scene(root);
 
        new AnimationTimer(){
				@Override
				public void handle(long now) 
                                {

                                                    switch (num)
                                                    {
                                                      case 1:
                                                        num = num + 1;
                                                        Startup();
                                                        System.out.println("1");
                                                        theStage.setScene(theScene);
                                                        
                                                        break;
                                                      case 2:

                                                        break;
                                                      case 3:

                                                        break;
                                                      case 4:

                                                        break;
                                                      case 5:

                                                        break;
                                                    }
                                }
                            }
                
//マウス
                    theScene.setOnMouseMoved(
            new EventHandler<MouseEvent>()
            {
                public void handle(MouseEvent e)
                {
                    mw=e.getX();
                    mv=e.getY();
                    System.out.println("Move  X=" + mw + "  Y=" + mv);
                }
            });
                
                    theScene.setOnMouseClicked(
            new EventHandler<MouseEvent>()
            {
                public void handle(MouseEvent ce)
                {
                    pw=ce.getX();
                    pv=ce.getY();
                    System.out.println("Push  X=" + pw + "  Y=" + pv);
                }
            });
                    
//キー
                    theScene.setOnKeyPressed(
            new EventHandler<KeyEvent>()
            {
                public void handle(KeyEvent pe)
                {
                    String code = pe.getCode().toString();
                    if ( !keyarry.contains(code) )
                        keyarry.add(code);
                    
                    System.out.print("push");
                    System.out.println(code);
                }
            });
        
                    theScene.setOnKeyReleased(
            new EventHandler<KeyEvent>()
            {
                public void handle(KeyEvent re)
                {
                    String code1 = re.getCode().toString();
                    keyarry.remove( code1 );
                    
                    System.out.print("remove");
                    System.out.println(code1);
                    
                }
            });
                                                
        theStage.show();
    }
    
    void Startup()
    {
        //画像のロードなど
        theStage.setTitle("testseikou");
        
    }
        
}

window名が"testseikou"に変更されませんでした
###試したこと
public Stage theStage = new Stage();と先にインスタンスを作成しました
###補足情報(言語/FW/ツール等のバージョンなど)
macでのNetBeans 8.2環境

Answer 1

原因は、タイトルをセットしているStageインスタンスと表示しているStageインスタンスが別のものだからです。ご質問のコードの肝心な部分だけを抜き出すと次のようになります。

public class Setup extends Application {
  ...
  public Stage theStage = new Stage(); //***(2)***
  ...
  @Override public void start(Stage theStage /***(1)***/) {
    ...
    new AnimationTimer() {
      @Override
      public void handle(long now) {
        switch (num) {
          case 1:
            ...
            Startup();
            ...
            break;
          ...
        }
      }
    }

    //マウスイベントのハンドリング
    //キーイベントのハンドリング

    theStage.show(); //***(1)***
 }

 void Startup() {
   //画像のロードなど
   theStage.setTitle("testseikou"); //***(2)***
 }
}

コードに***(1)***、***(2)***で示したStageインスタンスはそれぞれ異なるStageインスタンスになります。

• ***(1)***

アプリケーション開始時にJavaFXが生成してくれたStageインスタンスです。startメソッドの引数で渡されることになっています。

• ***(2)***

アプリケーションクラス(Setup)のフィールドとして定義した方で、アプリケーションインスタンス生成時に独自にStageインスタンスを生成しており***(1)***とは別物のインスタンスになります。

startメソッドの引数theStage、フィールドのtheStageはたまたま名前が同じですが、あくまで別の変数ですので混乱しないように注意してください。

メソッドStartup()内でtheStgaeを使いたい

Startupメソッドの引数としてStageインスタンスを渡すのが素直な方法だと思います。

もしフィールドtheStageを通じてStageインスタンスにアクセスしたいなら、以下のようにしてstartメソッドの引数に渡されたインスタンスをフィールドへ設定してやるとよいでしょう。

class Setup extends Application {
  Stage theStage;
  ...

  @Override
  public void start(Stage theStage) {
    this.theStage = theStage;
    ...
  }
  ...
}

---

蛇足：

• Javaの標準命名規則になるべく従いましょう。

メソッドや定数でないフィールド、ローカル変数の名前は小文字で始めるのが標準的な習慣です。C#などでは逆で大文字で始める習慣です。こうした習慣は言語ごとに違いますがなるべく標準的な習慣に合わせた方がよいと思います。

• 字下げ

コードをcopy & pasteした関係でずれているのかも知れませんが、switch文の字下げが不自然に深くなっています。質問の際には「閲覧者が見づらくならないよう」適切な字下げとなるよう注意するとよいと思います。

Comments

[nununuso]

解決できそうです
わざわざありがとうございます
観やすいように意識したいと思います