[表示 : 全て 最新50 1-99 101- 201- 301- 401- 501- 601- 2chのread.cgiへ]
Update time : 07/13 07:04 / Filesize : 160 KB / Number-of Response : 657
[このスレッドの書き込みを削除する]
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧] [類似スレッド一覧]


↑キャッシュ検索、類似スレ動作を修正しました、ご迷惑をお掛けしました

★★Java質問・相談スレッド162★★



406 名前:デフォルトの名無しさん mailto:sage [2013/06/29(土) 19:05:40.12 ]
Tableクラスに関するとりあえずのコメントとして

・カラムの型や名前をバラバラのMapで管理しているのはあまり良くない。素直にカラムの
 定義を扱うTableの内部クラスを用意して、
 public static class Column{public final int index; public final String name; public final String type;}

 これのリストをスキーマとして定義する。必要であればルックアップテーブルも。
 private final List<Column> schema = new ArrayList<Column>();
 private final Map<String, Column> nameToColumn = new HashMap ...

 スキーマ情報を提供する公開メソッドも用意すると親切だよね。
 public List<Column> getSchema() {return Collections.unmodifiableList(schema);}
 public List<Column> getColumn(int index) {return schema.get(index);}
 public List<Column> getColumn(String name) {return nameToColumn,get(name);}

・検索結果に一個結果を追加する度にサイズを+1した配列を生成して内容を全件コピーする
 凄いコードを見たような気がするのは気のせいだよね・・・素直に可変長であるList
 を使うべき。返値もListで良い。

・for(int i...)ではなく拡張for構文を使ってくれた方が読みやすい

・・・この調子で書いていくとどうなるのだろう・・・






[ 続きを読む ] / [ 携帯版 ]

全部読む 前100 次100 最新50 [ このスレをブックマーク! 携帯に送る ] 2chのread.cgiへ
[+板 最近立ったスレ&熱いスレ一覧 : +板 最近立ったスレ/記者別一覧]( ´∀`)<160KB

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef