Chciałbym zrobić tak, że zdjęcie które było następne po usuniętym, wskakuje na miejsce tego usuniętego, i tak wszystkie zdjęcia zmieniają swój numerek.

Załóżmy, że mamy 4 zdjęcia.
- 1,2,3,4
Usuwam 3
- 1,2,4
Przeładowanie zdjęć
- 1,2,3

Zacząłem coś pisać, ale nie mogę sobie z tym poradzić, mój smieszny kod.

[PHP] pobierz, plaintext 
<?php
$kod_klienta = $_SESSION['kod_klienta'];
$id = $_POST['id'];
$photo = $_POST['photo'];
 
unlink($cfg['oitia']['photos_path'].&#092;"/\".$id.\"/\".\"$photo.jpg\");
sql(&#092;"UPDATE oitia SET zdjecie_$photo = '0' WHERE id = '$id' AND kod_klienta = '$kod_klienta'\");
 
$query = sql(&#092;"SELECT * FROM oitia WHERE kod_klienta = '$kod_klienta' AND id = '$id'\");
$wpis = mysql_fetch_array($query);
 
for ($a = 1;$a <= 20;$a++) {
	$zdjecia[&#092;"zdjecie_\".$a] = $wpis[\"zdjecie_\".$a];
}
$zdjecia[&#092;"zdjecie_\".$photo] = '0';
 
for ($a = 1;$a <= 20;$a++) {
	$previous = ($a-1);
	if (($zdjecia[&#092;"zdjecie_\".$a] == 1) && ($zdjecia[\"zdjecie_\".$previous] == 0)) {
		$new = ($a-1);
		rename($cfg['oitia']['photos_path'].&#092;"/\".$id.\"/\".\"$a.jpg\", $cfg['oitia']['photos_path'].\"/\".$id.\"/\".\"$new.jpg\");
		$zdjecia[&#092;"zdjecie_\".$a] = 0;
	}
}
?>
[PHP] pobierz, plaintext 

Mam jedno pytanie.. PO CO?

To w takim razie jak ty byś to widział ?
Bo to wygląda tak, że klient ma możliwość wprowadzenia 20 zdjęć, dodaje, usuwa.

zapisuję na dysku zdjęcie o nazwie

[PHP] pobierz, plaintext 
<?php
$sFileName = md5_file( 'nazwapliku' ).'rozszerzenie_pliku';
?>
[PHP] pobierz, plaintext 

Dzięki czemu moge w ułamek sekundy sprawdzić, czy identyczne zdjęcie nie jest już zapisane.

Zapisuję do bazy ID zdjęcia, ID usera, nazwę oryginalną, $sFilename.
Usuwając zdjęcie usuwam odpowiednie ID z bazy, usuwam odpowiedni plik z dysku.

Przypominam, ze ID się nie przenumerowywuje, bo.. po prostu NIE. Ma byc unikalne zawsze. (jak uważasz, ze INT to za mało, zawsze masz BIGINT).

To lubię ( bez złośliwosci)
Oczywiście argument scanner-a ma bardziej racjonalne przesłanki niż NIE, BO NIE. Wyobraź sobie sytuację w której Twoja baza zawiera relację pomiędzy ID Twojego obrazka a ID w 125 tabelach, które informacje nt. tej fotki zawierają. Musisz zatroszczyć się o "przenumerowanie" wszystkich relacji, bo inaczej zapytanie (nieważny kod) zwróci fałszywe wyniki. Poza tym dodanie czegokolwiek co ma być unikalne najlepiej wykonać przy użyciu Sprawdzanie, czy w 17849 elementowej bazie miejsce 345 nie jest akurat wolne, żeby tam coś dodać to komplikacja, której nie tłumaczą ewentualne zyski.

A jak to rozwiązać gdy mam 250 użytkowników z tego każdy może mieć 20 zdjęć dodanych. A co się dzieje w przypadku gdy skonczą się wolne miejsca w polu id przy auto_increment ?

Zmien typ pola na BIGINT UNSIGNED. Masz wtedy możliwość zapisania liczb od 0 do 18446744073709551615. Styknie to Twoim userom na zapisanie 73786976294838206 zdjęć przez każdego z nich.

Nie no aż tyle rekordów to miał nie będe.
Ale chyba zamiast auto_increment wykorzystam moją funkcje, do wyszukiwania pierwszego wolnego rekordu. To chyba będzie lepsze rozwiązanie. Jak uważacie ?

Funkcja wygląda w ten sposób :

[PHP] pobierz, plaintext 
<?php
 
function get_first_free_record($table,$field) {
 
		$last = &#092;"65534\";
 
		$query = sql(&#092;"SELECT $field FROM $table ORDER BY $field ASC\");
 
		$tab[] = array(&#092;"0\" => \"\");
 
		while ($a = mysql_fetch_array($query)) {
				$tab[] = $a;
		}
 
		for ($b = 1;$b < $last;$b++) {
				$id = $tab[$b];
				if ($id[$field] != $b) {
						$first = $b;
						break;
				}
		}
		return $first;
}
 
?>
[PHP] pobierz, plaintext 

Co sądzicie ?

Głupota do kwadratu.

Od tego jest auto_increment w mysql zeby sobie radzic z takimi rzeczami. Jezeli pole jest 'auto-increment' i 'unique', masz pewnosc ze beda unikalne wpisy, i nie musisz sie bawic skryptem. 20 zdjec dla usera? Zadnen problem. Suma wszystkich rekordow dla danego usera musi byc mniejsza/rowna 20 i tyle. Klopoty z listowaniem pokolei? Zadnego. Polecam dla odmiany teraz manuala mysql poczytac [LIMIT itp].

Jeżeli robisz to dla treningu, to rób, ale jeżeli to ma być użyteczny skrypt, to , jak już wielkorotnie padło w tym wątku: PO CO?
Poza tym uczysz się złych schematów i będziesz później generował skrypty 100-linijkowe zamiast 5-linijkowych robiących dokładnie to samo, tylko dużo szybciej.