some_aoi 2014-04-19 03:12:09

[Java] Java5とJava1.4の夢のコラボレーション このエントリーをはてなブックマークに追加

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

Listが大好きでJavaBeanを作るのは嫌だったんでしょうか。

担当者が途中でギブアップして作れなくなってしまい、続きをなんとか作ってくれとお願いされ 渡されたソースがこんな感じだったそうです。

public class GyoumGamen1 {
    
    public List<List> getData() {
        
        List<List> list = new ArrayList<List>();

        List subList1 = new ArrayList();
        subList1.add(1);
        subList1.add("ColumnDataA1");
        subList1.add(2);
        subList1.add("ColumnDataB1");
        list.add(subList1);
               
        List subList2 = new ArrayList();
        subList2.add(1);
        subList2.add("ColumnDataA2");
        subList2.add(2);
        subList2.add("ColumnDataB2");
        list.add(subList2);
        
        /* 中略 */
    
        return list;
    }
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(2)

#1 InternalServerE 2014-07-25 20:45:21  

すみません。ふたつ質問
・夢のコラボレーションをしているのはどの辺ですか?
・貴方なら、どう設計するのが「正解」だと思いますか?
 例えば、「行・列方向ともに可変長なCSVデータ」の簡易な表現方法として、List<List>は割とよく使われるわけで。
 要は何をしたいか次第なので、一概に「これが正しい」という答えは決まらないんじゃないかなぁ、という印象を受けました。

#2 sieghart55 2015-01-17 14:22:52  

subList1とsubList2の要素を取得するときに、型がIntegerだったりStringだったりするのがNGということでは。

コメント投稿には、twitter認証が必要です。

Twitter認証

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

[Java] is禁止令

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

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »

[Java] 内部的にunicodeだから日本語つかえ...

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

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...

鑑賞する »

[Java] もうコメント書くなお前

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

改修案件でぶち当たったソース マジでこ...

/**
  *
  *   〇〇の値を取得する
  *
  */
pu...

鑑賞する »