Java if文

Question

現在JavaでApache Clickを使用し開発を行っています。

ラジオボタンの1を選択した場合、セレクトリストから何か選択をすることが必須となるというソースを書いたのですが、上手くいきません。
ラジオボタン2を選択した場合はセレクトリストを選ばなくても大丈夫であるという処理は実装されているのですが、ラジオボタン1を選んだ際に、セレクトリストを何か選択していてもfalseが帰ってきてしまいます。
select箇所のif文に何か記述が足りないと思われるのですが、よく分からなず停滞してしまっています。。
どなたかご教授お願いいたします。
ソースは↓このようになっております。

	String radio = inRadio.getValue();
	if ( "1".equals(radio)) {
	  // 1のときの処理
	  String select = inSelect.getValue();
	  if ( select == null || "".equals(select)) {
		  return false;
		  }
		} else {
		  // 2のときの処理


		return true;

		}

※追記

フォームとはセレクトリストなどを作成している部分のソースでよいでしょうか。勉強不足で歯切れの悪い追記になってしまい申し訳ございません。

	//利用停止 全取引
	inRadio = new ClickRadioGroup("radio", "利用停止 全取引");
	inRadio.setVerticalLayout(false);
	inRadio.add(new Radio("1", "利用停止  "));
	inRadio.add(new Radio("2", "解除"));
	inRadio.setValue("1");
	form.add(inRadio, 5);

	//利用停止理由 全取引
	ClickSelect inSelect = new ClickSelect("select", "利用停止理由 全取引", false);
	inSelect.add(Constants.SELECT);
	inSelect.add("未収による一時利用停止");
	inSelect.add("紛失・盗難による一時利用停止");
	inSelect.add("不正利用による一時利用停止");
	inSelect.add("口座凍結による一時利用停止");
	form.add(inSelect, 5);

このような記述をしております。よろしくお願いいたします。
radioと異なりinSelectにvalueを定めたような記述が見当たらないため、そもそもinSelectの取得方法自体が誤っているのでしょうか。

Comments

[moonphase]

inSelectのValueが "" になっているとかではないですか？フォームを作成しているコードも掲載してください。

[megu0209]

すみません、追記いたします。

Answer 1

org.apache.click.control Class Selectにある例とは使い方が異なってるようですね。

一旦、inSelect.add()のパラメータに文字列ではなくnew Option(String, String)したものを渡してみてはいかがでしょうか。

inSelect.add(new Option("1", "未収による一時利用停止"));
inSelect.add(new Option("2", "紛失・盗難による一時利用停止"));
inSelect.add(new Option("3", "不正利用による一時利用停止"));
inSelect.add(new Option("4", "口座凍結による一時利用停止"));

Comments

[megu0209]

ありがとうございます。
inRadioと同じよう、ご指摘いただいたソースに修正し、new Optionを使用した形にいたしました。
フォームに追記いたしました、inSelectは、セレクトリストの初期表示を無しにしたいため("");のままでよいのでしょうか。
またそうすると修正すべきはやはりif文の方となりますでしょうか。

[moonphase]

初期表示をなしにしたいなら
inSelect.add(new Option("", ""));
をコードに追加すればいけませんか？

[megu0209]

そのような記述で初期表示をなしにすることができました。ありがとうございます。
しかしセレクトリストから項目を選択してもfalseがまだ帰ってきてしまいました。。

String radio = inRadio.getValue();
if ( "1".equals(radio)) {
// 1のときの処理
String select = inSelect.getValue();
if ( select == null || "".equals(select)) {
return false;
}
} else {
// 2のときの処理

return true;

}

こ記述だけでは、radioが1の時に、selectがnullだとエラーとなる処理しか書けておらず、if文にさらに続けて、selectがnullじゃない場合にtrueを返す記述が必要なのでしょうか。
理解がなかなか及ばず申し訳ございません。。