math_neko 2013-05-13 12:30:15

[Java] 返り値で返り血 このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

何で値が保存されないんだよぉぉぉ…と考え込むこと半日…ヒントはタイトル。JSF の Managed Bean での一幕でした。

@EJB
private CustomerDAO dao;
private Customer item;
private HtmlInputText id;

// 中略

public void search() {
    int id = Integer.parseInt((String)this.id.getValue());
    item = dao.find(id);
}

コメント(2)

#1 kogaken1 2013-06-09 01:51:46  

10行目の意図が分からなかったんだけど、idのスコープの問題?

#2 math_neko 2013-06-09 01:53:57  

いや、search メソッドの返り値が拙かったんです。正しくは String 型を返して、このケースだと空文字列を返す必要があったんですが、横着して void にしたのがいけませんでした orz


このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] 曰く、「getterアクセサを呼ぶ時にn...

このエントリーをはてなブックマークに追加

**これは或るメンバに対するGetter...

// なんか20000行くらいあるクラス

// ...

// 600...

鑑賞する »

[Java] ClassCastException上等

このエントリーをはてなブックマークに追加

つい昨日見たコードです。

...

public doSomething(int userId, Object pa...

鑑賞する »

[Java] そういうオーバーライドの使い方はやめろ

このエントリーをはてなブックマークに追加

以前客先で出くわしたコードを思い出しなが...

public class MyClass extends Object{

...

鑑賞する »