• はてなアカウントでログイン
• 無料ユーザー登録

みんなの回答

• DQNEO

  2012/11/08 01:47:02

  カテゴリが「ウェブ制作」になってしもた orz

oil9992012/11/08 07:25:08

0pt

アップロードするファイル長の検査が無いような気がするのですが、間違っていたらごめんなさい。

スター

• 

  1件のコメントを見る

• DQNEO 2012/11/09 12:48:04

  そこはライブラリ側(AWS SDK)でやってくれているので大丈夫です。

  スター

techmedia-think2012/11/08 12:02:18

満足100pt

FileFinderの実装でメソッドやフィールドが全てstaticで定義されてますが、filesの内容の初期化が最初にされてるだけどなので、FileFinder#findを複数回呼び出すと前のfind結果も保持したまま要素が追加されることになりませんか？

スター

• 

  1件のコメントを見る

• DQNEO 2012/11/09 12:49:16

  おお、たしかに！
  盲点でした。
  FileFinderは外部から呼ばれることを想定してないので、パッケージの可視性をパッケージプライベート（でしたっけ？）に変更しようと思います。
  
  指摘ありがとうございます。

  スター

DQNEO

2012/11/09 12:50:49

質問文を編集しました。詳細はこちら。

Bright2012/11/14 18:54:46

満足100pt

上の回答以外のことは問題なさそうです。

スター

a-kuma32012/11/14 23:37:16

満足100pt

クラス設計的な視点で。
ちっちゃいツールなんだから、細けーこと気にせんわ、ということなら、読み飛ばしてください。


com.amazonaws.services.s3 なパッケージの import があちこちに散らばってるのが気になります。
ぼくなら、Task か Uploader に、Amazon の都合を押し込めます。
どっち（もしくは、両方）に押し込めるかは、ちょっと悩みます。
Task は、Callable の実装が欲しいだけなので、Uploader の inner class にして、call メソッドでは Uploader のメソッドを呼ぶ、という形にするかも。

同じ理屈で、CLI は、あくまでもパラメータの解釈とチェックだけにとどめる。

UploadableFile は、java.io.File の気持ち悪さを引きずってます（個人的感想です）。
java.io.File が、OS が提供する「ファイル」の現身だとしたら、remove() とか renameTo() とかが、ものすごく気持ち悪く感じます。インスタンスとしての実体があるのに「消えろ」とかいうメッセージが。

	File f = ...;
	FileSystem fs = ...;
	fs.remove(f);

みたいな感じだったら、分かるんですけど。
なので、UploadableFile の upload や getS3Key メソッドが気持ち悪くって。
ぼくなら、Uploader で実装します。


「Amazon S3 の仕様が変わりました」とか「他のストレージサイトにも対応したい」というときに、何をどこまで変更しなきゃいけないのか、というところで効いてくるはずです。

スター

• 

  3件のコメントを見る

• DQNEO 2012/11/15 10:19:05

  おお～！
  このような指摘を待っていました。
  大変参考になります。
  ちょっと考えてみます。
  
  ありがとうございます！

  スター

• a-kuma3 2012/11/15 22:35:07

  ぼくなら、こうする、って雰囲気だけクラス図にしてみました。
  FileFinder などのユーティリティなクラスは除外。
  
  
  CLI から、AmazonS3Uploader って具体的なクラス名が見えてるじゃん、というのは、小物ツールなので。
  Uploader の実装が増えたら、Factory を導入して、CLI は Uploader を知ってるだけ、というふうにする、という感じ。

  スター

• DQNEO 2012/11/16 18:18:44

  おお～～！！
  めちゃ勉強になります！
  確かに祖結合になってますね。
  
  あとでリファクタリングしてみます。
  ありがとうございます！

  スター
• その他の回答を読む

この質問へのコメント

質問の情報

登録日時

2012-11-08 01:45:19

終了日時

2012-11-15 01:50:03

回答条件

1人5回まで

この質問のカテゴリ

• ウェブ制作

この質問に含まれるキーワード

人気の質問

• この英文どうなってますか？ The boy seemed to be waiting for his mother to come.（その少年は母親が来…匿名質問者312日前
• 不定詞の形容詞的用法らしいんですけど、副詞的用法と書いたら✖️ですか？ I want to buy a book to read …匿名質問者112日前
• 日向陽葵という○V女優が好きなのですが似ている○V女優はいますか匿名質問者1日前
• 共通テストで使われるプログラミング言語は共通テスト特有の言語と聞いたのですが、受験勉強で身につけたプ…匿名質問者211日前
• 内定してほしいポケモンはいますか？匿名質問者1日前

メニュー

• 質問一覧
• カテゴリ一覧
• 無料ユーザー登録
• 人力検索はてなトップ

PC版

• お問い合わせ
• ヘルプ
• お知らせ
• はてなトップ