回答編集履歴
1
異常系の処理について追記
answer
CHANGED
|
@@ -40,6 +40,10 @@
|
|
|
40
40
|
|
|
41
41
|
---
|
|
42
42
|
|
|
43
|
+
あとバグとは言い切れないものの考慮が甘い部分があります。各ディレクトリ内にフォルダや画像以外のファイル（OSが自動作成しているメタファイル）があった場合、意図しない動作をしてしまう恐れがあります。リネーム対象なのか（フォルダではなくファイルか、拡張子は画像(jpg, jpeg, gif, png...)なのか）をチェックする処理を入れた方がよいと思います。
|
|
44
|
+
|
|
45
|
+
---
|
|
46
|
+
|
|
43
47
|
最後に余談ですが、使っている変数名が i, j, y だと何に使っているカウンタなのかわかりませんし、image や photo も漠然としていて何を格納しているかわからないので、もう少し命名を工夫した方がよいと感じました。
|
|
44
48
|
|
|
45
49
|
また配列へキーを使ってアクセスしなくても for で変数に入れてしまえばすっきりします。
|