人力検索はてな
モバイル版を表示しています。PC版はこちら
i-mobile

Java初心者です。(Java歴半年くらい)
こんなツールを作ってオープンソースで公開したのですが、自分の設計がこれでよいのか自信がありません。
設計上変なところがあったら指摘していただけないでしょうか?

https://github.com/DQNEO/S3ParallelUploader
フォルダを丸ごとAmazonS3にアップするアップローダです。
CLIで動きます。
一応テスト済みで、ちゃんと動きます。

(質問というよりコードレビュー依頼です)
よろしくお願いいたします。

●質問者: DQNEO
●カテゴリ:ウェブ制作
○ 状態 :終了
└ 回答数 : 4/4件

▽最新の回答へ

質問者から

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


1 ● oil999
●0ポイント

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


DQNEOさんのコメント
そこはライブラリ側(AWS SDK)でやってくれているので大丈夫です。

2 ● techmedia-think
●100ポイント

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


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

3 ● Bright
●100ポイント

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


1-5件表示/6件
4.前の5件|次5件6.
関連質問

●質問をもっと探す●



0.人力検索はてなトップ
8.このページを友達に紹介
9.このページの先頭へ
対応機種一覧
お問い合わせ
ヘルプ/お知らせ
ログイン
無料ユーザー登録
はてなトップ