Dotaz: Operace s AUTO_INCREMENT pomocí PHP a SQL

[Jan Richter]

Dobrý den, potřebuji pro aplikaci, která pracuje s databází vymazat záznam nezávisle na jeho ID klidně uprostřed tabulky, přečíslování ID mám vyřešeno v cyklu FOR pomocí SQL - UPDATE a podmínek, ale při pokusu o změnu ID mi to vyhodí chybovou hlášku přitom syntaxe je podle mě bezchybná.

if ($delete_index == 1) {

$autoinc = mysql_query("ALTER TABLE `Articles` AUTO_INCREMENT ='" . $delete_ID . "'");

if (!$autoinc) {

die('Operace zrušena: ' . mysql_error());

mysq_close($link);}

else {

echo "Increment snížen";

}

}

Odpovědi

[Kit]

Není mi jasné, proč při mazání záznamu měníš strukturu tabulky a proč chceš přečíslovávat ID. Ani jedno z toho se nedělá.

[Jan Richter]

Mám vypisování záznamů z tabulky v cyklu podle promenny X která se každou otočkou inkrementuje a zároveň je tatáž proměnná v klauzuli WHERE v SQL aby třídila záznamy a vypisovala do pole jen ten záznam který má ID = $X a kvůli tomuhle postupu musim měnit ID záznamů aby šli po sobě a měnit i inkrementaci, která se sama nezmění...nehledě na to že pokud chceš tabulku použít i jako data pro tisk a nechceš to upravovat EXCELEM je jednoduší zautomatizovat "úklid" toho bordelu...je to pro redakční systém, který ovlivňuje i uživatel, který nemá ani ponětí co všechno se stane když odešle formulář proto myslim i na to jak tabulku udržovat.

[l4m4]

Tohle celé zní jako nějaký nesmysl. Ty spouštíš na vypsání každého jednotlivého řádku nový SQL dotaz? Proč? Normálně dostaneš jako výsledek dotazu celou sadu, a tu si proiteruješ v PHP, jak je zapotřebí (jiná věc je případné stránkování, ale to se zase řeší v dotazu, abys dostal řádky od do).

[Kit]

Možná by bylo dobré nepoužívat MySQL jako NoSQL, ale jako SQL databázi.

Dávat SQL dotazy do cyklu po jednom řádku je hovadina, protože SQL databáze to umí udělat jedním dotazem.

[rADOn]

Jestli je to podle tebe bezchybne nikoho nezajima, dulezite je co si mysli databaze. Napis jaka chyba z toho leze.

BTW precislovani je silenost. Pokud skutecne potrebujes za kazdou cenu spojitou radu, tak bys nemel pouzivat auto_increment.

[Jan Richter]

if ($delete_index == 0) {
    $after_delete_ID = $delete_ID + 1;
    for ($x = $delete_ID; $x <= $maxrows_delete_artic; $x++) {
         mysql_query("UPDATE `Articles` SET ID='" . $x . "' WHERE ID='" . $after_delete_ID . "'");
    }
    $autoinc = mysql_query("ALTER TABLE `Articles` AUTO_INCREMENT ='" .  $maxrows_delete_artic . "'");
    if (!$autoinc) {
       die('Operace zrušena: ' . mysql_error());
       mysq_close($link);
} else {
  echo "Incrementy sníženy";
}
}
A KULI TOMUHLE UPRAVUJU TABULKU PRI MAZANI ZAZNAMU
for ($x = 1; $x <= $maxrows; $x++) {
        $query = mysql_query("SELECT * FROM Articles WHERE ID='" . $x . "'");
        $row = mysql_fetch_array($query, MYSQL_ASSOC);
        (HTML struktura pro data)
}

[Josef Kufner]

Tohle je divný, hodně divný. Popiš úlohu, kterou řešíš, protože zjevně na to jdeš zcela špatně.

[Jan Richter]

Jde o administrativu novinek se všim všudy (přidat upravit, mazat) po týhle diskuzi je jasné že AI v tabulce nepoužiju a ID si budu generovat sám tím bych měl dosáhnout kýženého výsledku...protože kombinovat to dohromady nemá smysl

[Josef Kufner]

Auto increment použij, ale vykašli se na všechny ty další blbosti, co děláš.

Spojitou posloupnost ID na něco takového nepotřebuješ. Vlastně tě ani nezajímá, jak takové ID vypadá. Prostě nech databázi, ať ke každé novince vymyslí unikatní identifikátor a nestarej se. Když pak chceš s nějakou novinkou něco dělat, tak vem identifikátor tak, jak ti ho databáze dala a vůbec nedumej nad nějakými jeho vlastnostmi – v rámci tabulky je unikátní, nic dalšího tě nezajímá.

Když chceš získat seznam novinek, tak uděláš jeden select pro celý výpis a pak iteruješ nad výsledkem získaným z databáze (každý výsledek sql dotazu je tabulka a ty dostaneš iterátor, kterým ji po řádcích projdeš). K tomu žádné ID není vůbec potřeba. To ID tam jen máš pro případ, že bys chtěl s daným řádkem něco dělat.

[Jan Richter]

Souhlasím, nicméně ten skript je navržený na tuto situaci kde cykly zvyšují $x a já ji používám jako podmínku v klauzuli WHERE snad si rozumíme co mám na mysli uspořádanou tabulkou minimálně dosáhnu menšího pracovního času cyklu i když se tu bavíme o milisekundách ;) Tvoje řešení je jednoduší ale musel bych kůli tomu změnit deset dalších věcí vydal sem se trošku jinou cestou u daných tabulek jsem zrušil vlastnost AI a předělal kód následovně:

$result_delete_artic = mysql_query("SELECT * FROM `Articles`");
$maxrows_delete_artic = mysql_num_rows($result_delete_artic);
if ($delete_ID == $maxrows_delete_artic) {
   $delete_index = 1;
} else {
   $delete_index = 0;
}

if ($delete_index == 0) {
   $after_delete_ID = $delete_ID + 1;
   for ($x = $delete_ID; $x <= $maxrows_delete_artic; $x++) {
        mysql_query("UPDATE `Articles` SET ID='" . $x . "' WHERE ID='" . $after_delete_ID . "'");
   }
} else {
  $query_delete = mysql_query("DELETE FROM `Articles` WHERE ID='" . $delete_ID . "'");
  if (!$query_delete) {
     die('Záznam se nesmazal: ' . mysql_error());
     mysq_close($link);
  } else {
    echo "Úspěšně smazáno";
  }
}

A funguje skvěle o ID se teď starám já přímo při INSERTU záznamu prostě je tam jen hodnota navíc co se musí zapsat také řízená cyklem :)

[Šangala]

Zkuste se zamyslet na možností „neměnit ID“, tedy zapomenout na UPDATE id - (rodné číslo si také neměníte).
On to nedává moc smysl. ID je jednou ID a používáte jej k identifikaci konkrétního záznamu a na procházení tabulkou jej nepotřebujete.

PS: Nedosažitelný a chybný kód je: mysq_close($link);

[Josef Kufner]

Žádná úspora odstraněním děr v posloupnosti ID neexistuje. Tedy alespoň na straně databáze, té je úplně jedno jaké ty ID jsou. I kdyby byly náhodně rozprostřené v celém rozsahu intu, bude to úplně stejně rychlé.

Změna ID je jedna z nejdražších operací nad daty v tabulce, neboť se musí aktualizovat primární index nad tabulkou. A vzhledem k tomu, že nepoužíváš transakce ani nevypínáš aktualizace indexů, tak se provádí tato aktualizace pro každý zavolaný update.

Shrnutí: Tvé řešení je komplikované, chybné a pomalé.

[Josef Kufner]

Navíc používáš chybně funci mysql_num_rows. Ta vrací počet položek ve výsledku, nikoliv ID. Je jen čirá náhoda, že hodnota těchto dvou věcí je v tvém případě stejná.

[Šangala]

Já tomu nerozumím, ale všiml jsem si, že při zklamání (pro mě nepochopitelného ) příkazu $autoinc = mysql_query("ALTER TABLE… PHP vyhodí chybu.

[Šangala]

A teď jsem si všiml, že ne - on je ten řádek nedosažitelný/zbytečný… :)