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

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

ただいまの
回答率

90.51%

  • JavaFX

    397questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。

JavaFX CombBoxに値をsetしたい

解決済

回答 1

投稿 編集

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

katare

score 8

JavaFX CombBoxにTableViewから取得した値や、Databaseからの値をもとにレコードを選択したいと考えております。
現在は、cmbCombBox.getSelectionModel().select(i);
これを、AccessなどのCombBoxのように値をセットすると該当のレコードが表示されるようにできないでしょうか?

開発環境は、Luna 4.4.2です。 

// CombBoxクラス
public class cmbSyainID {
    private final SimpleIntegerProperty iSyainID = new SimpleIntegerProperty();
    private final SimpleStringProperty sSyainName = new SimpleStringProperty();

    public cmbSyainID()    {
        this (0,"");
    }
    public cmbSyainID (Integer id, String name)    {
        setSyainID(id);
        setSyainName(name);
    }
//    ―――――――――――――――――――――――――――――――――――――――――――――――――――――
    public Integer getSyainID()    {
        return iSyainID.get();
    }
    public void setSyainID(Integer id){
        iSyainID.set(id);
    }    
    public String getSyainName()    {
        return sSyainName.get();
    }
    public void setSyainName(String name)    {
        sSyainName.set(name);
    }
    public void setSelectedItem(Integer id)    {
        iSyainID.set(id);
    }
    public Integer getSelectedItem()    {
        return iSyainID.get();
    }
//    ―――――――――――――――――――――――――――――――――――――――――――――――――――――
    public int getValue()    {
        return iSyainID.get();
    }
    public void setValue(Integer id)    {
        iSyainID.set(id);
     }
    public String getText()    {
        return sSyainName.get();
    }
    public void setText(String name)    {
        sSyainName.set(name);
    }
    public String toString()    {
        return sSyainName.get();
    }
}
// Controllerクラス
public class SyainController implements Initializable {
    @FXML ComboBox<cmbSyainID> cSyainID;
    @FXML TableView<LstSyain> LstSyain;
    @FXML TextField tSyainName;

    Integer iBusyoID = 10;
    PreparedStatement pstmt = null;
    ResultSet rset = null;

    @Override
    public void initialize(URL url , ResourceBundle rb)    {
//     * CombBox List
//    ―――――――――――――――――――――――――――――――――――――――――――――――――――――
        setSyainList();
        cSyainID.getSelectionModel().select(0);

//     * TableView List
//    ―――――――――――――――――――――――――――――――――――――――――――――――――――――
        LstSyain.getItems().setAll(returnSyainList());

        LstSyain.getSelectionModel().selectedItemProperty().addListener(new ChangeListener<LstSyain> ()    {
        @Override
        public void changed(ObservableValue<? extends LstSyain> value, LstSyain oldValue, LstSyain newValue)    {
            if (LstSyain.getSelectionModel().getSelectedItem() != null)    {
            //getSyainName()は、LstSyainクラスにて宣言
                tSyainName.setText((String) newValue.getSyainName());
            // ※※※※※※※※※※※※※※※※※※※※※※※※※※※
            //Listの動きに伴って、CombBoxの値も変更したい
            //今回指摘して頂いた ⇒ select(T item)    (B)選択すべきアイテムのvalueを指定をしてるつもり
                cSyainID.getSelectionModel().select(newValue.getSyainID());
            // ※※※※※※※※※※※※※※※※※※※※※※※※※※※
            }
        }
    });
}
    // * CombBox List.
    //──────────────────────────────
        public void setSyainList()    {
            String strSQL    = "Execute    dbo.PStad"
                            + "  @NN    = 'CBoxSource'"
                            + ", @BusyoID    = " + Integer.toString(iBusyoID);

            PreparedStatement pstmt = null;
            ResultSet rset = null;

            List<cmbSyainID> ll = new LinkedList<cmbSyainID>();
            cmbSyainID roww = new cmbSyainID();

        //    ──────────────────────────────
            try    {
                Connection con = DBConnect.connect();    //DBConnec Connectionクラス
                pstmt = con.prepareStatement(strSQL);
                rset  = pstmt.executeQuery();

                while (rset.next())    {
                    roww = new cmbSyainID();
                    roww.setSyainID(rset.getInt(1));
                    roww.setSyainName(rset.getString(2));
                    ll.add(roww);
                }
                rset.close();
                pstmt.close();
            }    catch    (Exception e)    {
                e.printStackTrace();
            }
            cSyainID.getItems().setAll(ll);
        }

    // * TableView List.
    //──────────────────────────────
        private List<LstSyain> returnSyainList()    {
            Integer iID = 100;

            PreparedStatement pstmt = null;
            ResultSet rset = null;
            List<LstSyain> ll = new LinkedList<LstSyain>();

            try {
                Connection cnn = DBConnect.connect();
                String strSQL    = "Execute dbo.PStad"
                            + "  @nn    = 'ListReturn'"
                            + ", @ID    = " + Integer.toString(iID);

                pstmt = cnn.prepareStatement(strSQL);
                rset = pstmt.executeQuery();

                int i = 0;
                while (rset.next())    {
                    i = i + 1;
                //──────────────────────────────                
                    Integer id = rset.getInt(1);
                    String name = rset.getString(2);

                //━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                // 一番上のレコードの値ををコントロールに表示
                    if (i == 1)    {
                //※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※
                // select(i) indexではなくValueでSelectしたい
                // cmbCombBox.select(T item)
                        cSyainID.getSelectionModel().select(id);
                //※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※※
                        tSyainName.setText(name);    
                    }
                //━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

                    LstSyain roww = new LstSyain();
                    roww.setSyainID(id);
                    roww.setSyainName(name);

                    ll.add(roww);
                }
                rset.close();
                pstmt.close();
            }
            catch (Exception e)    {
                e.printStackTrace();
            }
            return ll;
        }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

結論を申し上げると「できます」なのですが・・・ご質問のコードを拝見すると少々混乱します。

// select(i) Row_Count()でなくValueでSelectしたい
// cmbCombBox.setSelectItem(nameid)な感じ
cmbCombBox.getSelectionModel().select(nameid);

はっきりわかりませんが、勘違いされている可能性もあるのでその点を指摘しておきます。ComboBox<T>には選択状態を変更するためのオーバーロードされたメソッドが2つあります。

select(int index) // (A)選択すべきアイテムのindexを指定
select(T item)    // (B)選択すべきアイテムのvalueを指定

コードではIntegerをselectに渡しておられます。JavaではintとIntegerはboxing/unboxingにより暗黙変換されるのですが、オーバーロードされたメソッドの解決においてはint/Integerは区別して扱われます。ご質問のコードではcmbCombBoxをどのような型として宣言しているのか不明ですが、宣言のしかたによって意味が変わる可能性があります。

ComboBoxの型パラメータ 起動されるメソッド
Integer,Number,Objectなど (B):valueにより選択
?, Longなど (A):indexにより選択

さて、ComboBoxは「あらかじめitemsプロパティーに設定した値のリストの中から特定の値を選ぶ」ためのコントロールです。ご質問のコードにはitemsプロパティーに何が設定されているのか不明ですが、DBのresultsetをitemsの内容と仮定するなら、以下のようにすべきです。(nameidは重複がないと仮定しています)

// cmbCombBoxの型はComboBox<Integer>であると仮定
...
int i = 0;
while (rset.next()) {
  i++;
  ...
  Integer nameid = rset.getInt(3);
  ...
  cmbCombBox.getItems().add(nameid); // 各行を読み込む度にitemsへ追加
  if (i == 1) {
    tfName.setText(nama);
    cmbCombBox.getSelectionModel().select(nameid); // valueにより選択
    dtfKeiyakuDay.setValue(DateTimes.toLocalDate(date));    
  }
  ...
}
...

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/09 10:03

    今回も丁寧な回答ありがとうござました。
    KSwordOfHaste様がおっしゃるとおり、理解が浅いままwebで調べて機能実装を優先し開発をすすめています。
    今回もCombBoxのselectをitemとindexで行っているのですが、理解なく行っていたようです。

    質問文を少し変更させて頂きました。
    自分なりにも調査しまして、今回はChoiceBoxでやりたいことができるか調べてみようと考えています。
    根本的な解決ではないのですが、select(index)をid一致まで回して取得するよりはスマートかと・・・。

    キャンセル

  • 2017/03/09 12:32 編集

    質問文にある「レコードを選択」というのはTableViewの行とComboBoxの選択アイテムを連携することだとは思ったのですが、回答に書いたようにComboBoxの操作について混乱があったようなのでとりあえず単一のComboBoxのアイテムの選択にしぼって回答しました。
    さて変更されたコードを拝見しました。コードの一部を省略するのはいいのですが例えばローカル変数のタイプミスとおぼしき部分があったりしてコンパイルが通らない状態のソースになっているように見えます。比較的長いソースなのでコンパイルエラーがあるとあなたの意図が間違って読者に伝わる恐れがありますし回答に必要以上の手間がかかります。

    余計な論理の部分をとりさってやりたい点を説明するのに最小限必要なだけの短い(かつコンパイル&実行可能な)ソースにするとベターです。

    キャンセル

  • 2017/03/09 14:03

    ソースの中途半端な修正申し訳ありませんでした。
    質問する側の最低限な礼儀がなってなく申し訳ないです。

    今回は、SQL側でCombBoxの一番上にselectしたいデータがくるようにすることで逃れたいと思います。
    ChoiceBoxの回答でもKSwordOfHaste様のものを拝見しましたが、現状私が実装するには理解が足りないようです。
    次回挑戦したいと思います。

    今回は、不躾な回答依頼に対応頂きありがとうございました。

    キャンセル

  • 2017/03/09 14:44

    本件は内容的によくないわけではないと思います。むしろ回答者や同様の課題に取り組んでおられる方々にも参考になり得る題材だと感じますし、不躾などとは思っておりません。
    ある程度複雑なものは質問も回答もどこまで書くのがよいか難しいですね。自分の回答は今一つ役立ってない気がするのでBAを付けていただくのは却って心苦しいです。
    なお、「コンパイルが通るコードを書こう」というのは気を付けた方がよいでしょう。質問・回答双方にいえる話だと思いますw;

    キャンセル

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

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

関連した質問

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

  • JavaFX

    397questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。