日々の破片

_ こないだはまったおばかなバグ

public String foo(String[] array) {
  StringBuffer sb = new StringBuffer();
  for (int i = 0; i < array.length; i++) {
    sb.append(array[0]);
  }
  return new String(sb);
}
....
assertEquals("ABCD", o.foo(new String[] { "A", "B", "C", "D" }));

と抽出すると、これで気づかなきゃおかしいだろう。AAAAとなるわけだから。

実際は、

public String foo(String[] classNames) {
  for (int i = 0; i < classNames.length; i++) {
    try {
        Class.forName(classNames[0]);
    } catch (Exception e) {
      IllegalArgumentException ia = new IllegalArgumentException("bad class name:" + classNames[i]);
      ia.initCause(e);
      throw ia; 
    }
  }
}
....
try {
  o.foo(new String[] { "jp.co.foo.BAAAA", "jp.co.foo.Bar" });
  fail("no exception");
} catch (Exception e) {
  assertTrue(e.printStackTrace(), e instanceof IllegalArgumentException); // 追記（これ第1引数はそのままは書けないよ）
}
try {
  o.foo(new String[] { "jp.co.foo.Bar", "jp.co.foo.BAAAAA" });
  fail("no exception");
} catch (Exception e) {
  assertTrue(e.printStackTrace(), e instanceof IllegalArgumentException);
}

で、何度やっても2番目の呼び出しでno exceptionとなる。素直に自分を疑えばよいのに（というか疑っているのだが、例外処理のほうに気を取られてclassNames[0]としていることにまったく気づかず。

考察1：APIがたとえば、foo(List classNames)だったらどうだろうか？

for (Iterator i = classNames.iterator(); i.hasNext();) {
  Class.forName((String)i.next());

とするから、少なくてもここでストールするこたないだろう。(実際には長い文字列をsplitした結果だから配列を利用するほうが手数が少ないので配列を利用しているので、それなりの必然性はある）

考察2：ソースコードレベルデバッガを使う場合は面倒がらずにかっちりと使う。（forNameの次の文とcatchの内側に仕掛けていたので与えている文字列がclassNames[0]だとは思いもしなかった）

考察3：こういうときはprintfデバッグでも良いかも。

  for (int i = 0; i < classNames.length; i++) {
    try {
        System.out.println("start check:" + classNames[i]);
        Class.forName(classNames[0]);  // で気づかないとますますはまるだけだけど。
    } catch (Exception e) {

_ J2SE 1.5のfor-each

あらためて見てみたら、この例はおもしろいな。

for (Iterator i = suits.iterator(); i.hasNext(); )
    for (Iterator j = ranks.iterator(); j.hasNext(); )
        sortedDeck.add(new Card(i.next(), j.next()));

これはバグ。でも、

for (Suit suit : suits)
    for (Rank rank : ranks)
        sortedDeck.add(new Card(suit, rank));
}

こっちはOK。でも、なんかな……単純なIterator#nextの置換と考えて逆に痛い目に合いそうな気もするのだが。