phpで画像ファイルを連番にリネームしたい

Question

前提

データベースに、ユニークなidが振り当てられた下記のようなテーブルがあります。

｜ id ｜name ｜
───────────
｜10001 ｜のび太 ｜
───────────
｜10002 ｜ドラえもん｜

そのユニークなidと同じ名前のフォルダがあり、そこに画像ファイルが数枚入っています。
画像のファイル名は1.jpg、2.jpg・・・と連番になっています。

現在上記情報の編集ページを作っていて
Ajaxを使用して編集ページ内で指定した1枚の画像の削除を行えるようにしました。

やりたい事

画像の削除を行った際に、画像フォルダ内の他の画像ファイルを
また1番からの連番にリネームしたいです。

例えば2.jpgを削除した場合はフォルダ内は
1.jpg、3.jpg、4.jpg　
となるのですが
この場合だと
1.jpg → 1.jpg
3.jpg → 2.jpg
4.jpg → 3.jpg
という風になる様にリネームしたいです。

どうやったらそんなことができるのか、ヒントだけでも知恵を貸していただけないでしょうか。

該当のソースコード

<button type="button" data-order="1" data-id="10001" class="delete">削除</button>

<script>
$('button.delete').on('click', function() {

	var order = $(this).data('order');
	var id = $(this).data('id');

	$.ajax({
		url: 'delete.php',
		type: 'GET',
		dataType: 'json',
		data : {
			order : order,
			id : id
		}
	}).done(function(data){
		alert('削除しました。');

	}).fail(function(data){
		alert('削除に失敗しました');

	});

});
</script>

delete.php

if(isset($_GET['order'] && $_GET['id'])){
	$order = $_GET['order'];
	$id = $_GET['id'];

	unlink(PATH_IMG . $id . "/" . $order . ".jpg");
}

補足情報（FW/ツールのバージョンなど）

xampp 8.0
php 8.0

Comments

[maisumakun]

なぜリネームしたいのでしょうか？

[kakedashi_php]

ご回答ありがとうございます
1,3,4となっていると　画像を表示するページで使いづらいなと思ったのと、
画像の削除と追加を繰り返していくと数字がめちゃめちゃになって何が何番目の画像か分かりづらいかなと思い
そろえた方がいいのかなと考えました。

表示する側の都合でデータを変えるというのは良くないのですね。
なぜリネームするのかというのを改めて考えるきっかけになりました
ありがとうございました。

Answer 1

ファイル名そのものをリネームするのではなく、抽出したいディレクトリ内のファイル数をカウントし、連番の request に対してreadfile()で適当に response してあげると良いかと。

なんとなくですが、非常に危険な実装をしている気がします。
アップローダも実装してますよね？アップローダは非常に危険な機能なので、もう一度リスクを精査してみてください。

Comments

[kakedashi_php]

ご回答ありがとうございます。
回答にあったファイル数をカウントというのを利用し
画像がそれほど多くないので、for文で1個ずつリネームすることにしました。

趣味で作っていて、世に出るようなものではないのですが
セキュリティについてももっと学ぶことにします。
ありがとうございます。

[退会済みユーザー]

> 画像がそれほど多くないので、for文で1個ずつリネームすることにしました。

それを否定した回答なんですけどね＾＾；

> 趣味で作っていて、世に出るようなものではないのですが

インターネット上に置いてあるなら世に出ていると認識してください。
たった一つ問題のあるファイルを置かれるだけで、サーバを乗っ取られてしまいます。
攻撃者の踏み台になるので、場合によっては訴訟に巻き込まれます。

[kakedashi_php]

＞request に対してreadfile()で適当に response
というのが理解できなかったので、すみません。

ファイルは全部ローカルにあって、今後もアップロードする事はありません。
気を付けます。

Answer 2

画像名をリネームしてしまうと、画像を使っているところ全部に影響しませんか。「データをどう使うか」という観点から「リネームするメリットは何か」考えてみては如何でしょうか。
「ユニークである」ことが保証されているなら、リネームすることによるデメリットの方が大きいかと思います。
「ユーザが削除されたからユーザIDを振りなおす」なんてこと普通はやりませんし。

Comments

[kakedashi_php]

ご回答ありがとうございます
1,3,4となっていると　画像を表示するページで使いづらいなと思ったのと、
画像の削除と追加を繰り返していくと数字がめちゃめちゃになって何が何番目の画像か分かりづらいかなと思い
そろえた方がいいのかなと考えました。

データに対しての考え方をもう一度考えてみます。
ありがとうございます。

[m.ts10806]

>画像を表示するページで使いづらいなと思ったのと
どこがですか？
既に「ユニークなidと同じ名前のフォルダ」があるならその中の画像名を全回しで取ってくるだけですよね。
フォルダ内全回しが難しいなら「画像名管理テーブル」もしくは「画像の実態をDBに保存」することで解決します。

[kakedashi_php]

画像ファイル名が連番で、それが順番も担っていたら表示するときの処理は簡単になると思います

ファイル名がバラバラだったらどうしたらいいかわかりません。
おっしゃるとおり画像名の管理テーブルをつくってそこに順番などの情報を入れて
フォルダ内のファイル名を取得
↓
配列に入れて回す　画像１枚ごとにデータベースにアクセスする

という風になると前者の方が簡単かなと、素人ながらに感じました。
馬鹿な事言ってたらすみません。

[m.ts10806]

どっちにしても結局全回しするので名前は関係なくなります。