Dotaz: php + sqlite + delete mi nic nesmaže

[otula]

Mám nejspíš úplně jednoduchý problém, ale už jsem z něj magor. Máme jistou skupinu, kam dáváme fotky. Převzal jsem její vlastnictví a chci pročistit galerii, takže jsem udělal hlasování, přičemž pro samotné naklikávání hlasů jsem si upravil jakýsi hotový skript, který používá sqlite. Hlasy jsou hlídány podle IP. No a jistý uživatel mi tam udělal trochu bordel z více adres, takže potřebuji záznamy promazat. Jenže když zadám

$dbh->exec("DELETE FROM $this->table WHERE id=$id")

tak to nefunguje (ani s query namísto exec). Přitom

$dbh->query("SELECT * FROM $this->table WHERE id=$id")

funguje úplně v pohodě.

Dokážete mi prozradit, co dělám blbě?

---

---

Odpovědi

[Šangala]

Nemělo by být:

$dbh->queryExec("DELETE FROM $this->table WHERE id=$id");

nebo

sqlite_exec("DELETE FROM $this->table WHERE id=$id",$dbh);

?

[otula]

Fatal error: Call to undefined method PDO::queryExec() in ...

Fatal error: Call to undefined method PDO::sqlite_exec() in ...

[Šangala]

Předpokládal jsem se používáte SQLite ne PDO.
Není tam někde před tím beginTransaction() a tedy nefunguje autocommit a je třeba commit explicitně zavolat?
Po tom mazání nšco jako print_r($dbh->errorInfo()); by vypsalo co?

[otula]

píše, že je databáze zamčená

Array ( [0] => HY000 [1] => 6 [2] => database table is locked ) 

[Šangala]

To by se dalo tušit, že se jedná o persistení spojení (při new PDO() ve poli jako čtvrtý parametr něco jako PDO::ATTR_PERSISTENT => true) a něco nedopadlo dobře a vysí tam nějaká transakce, nebo to mažete na blbém místě, nebo je tabulka opravu LOCK-nutá explicitně (jestli to SQLite lze)…
Pokud to nechcete řešit a jedná se o jednorázovku a je možné to odstavit, odpojte systém restartujte SQLite a pak to hned smažte.

[otula]

SQLite běží na webhostingu, explicitně locknutou tabulku (snad) nemám, INSERTem v pohodě vkládám záznamy. Kód jsem zkusil nacpat na pastebin. Moc tady to programování nezvládám, takže je možné, že dělám nějakou úplně základní chybu, to nevím. Jestli to nezvládnu, tak zkusím splihnit nějaký skriptík v BASHi, který by mi udělal offline to, co potřebuji (ale to se musím pro změnu zase podívat, jak se z příkazové řádky připojím na tuhletu srandu… Teď se tluču do hlavy, že jsem se nedonutil to ukládat do MySQL )

[otula]

Bohužel neznám dobře všechny souvislosti PDO a SQLite, ale jediné co tam vidím je, že by to mohlo viset na tom některém selektu, protože tam není uzavřen cursor. Takže bych si u selektů vrátil PDOStatement do promněné, na ní udělal fetchColumn() a pak ješte closeCursor(). Pokud to rovnou neudělat postupem $stmt = $dbh->prepare(); $stmt->execute(); $stmt->fetch(); $stmt->closeCursor();.
Možná je to ten $table_check, takže bych přidal jako první na řádek 14 $table_check->closeCursor();

[otula]

takže bych přidal jako první na řádek 14 $table_check->closeCursor();

Super, funguje! Mockrát děkuji za radu i věnovaný čas.

Ota

[Šangala]

Není zač, jen jsem tipoval .

[Kit]

Teď se tluču do hlavy, že jsem se nedonutil to ukládat do MySQL )

V MySQL by se to možná nedalo tak jednoduše vyřešit jedním složeným SQL dotazem jako např. v SQLite.

[otula]

Ale přece jen s ním mám alespoň nějaké zkušenosti, navíc tam například pro DELETE funguje LIMIT. No hlavně, že už to chodí :) Také díky za snahu pomoci.

[Šangala]

Včera jsem ho chtěl napsat (jeden SQL dotaz), ale hned na začátku jsem toho nechal, ze dvou důvodů, neznám SQLite a nevěděl jsem kde narazím, a druhý, když jsem si uvědomil, že se jedná o mazání duplicitních záznamu, přičemž libovolný jeden má zůstat.
Šlo by ten SQL dotaz sem hodit?

[Kit]

DELETE FROM table WHERE NOT id IN (SELECT max(id) FROM table GROUP BY ip);

Ponechá jen poslední záznam od každé IP.

[Šangala]

No vida to je elegantní
A je pravdou, že v MySQL je třeba udělat obstrukci (nevhodnou pro stovky milionů záznamů) něco jako:
DELETE FROM table WHERE NOT id IN (SELECT max(id) FROM (SELECT id,ip FROM table) AS t1 GROUP BY ip);

[Kit]

Možná je nutné v MySQL udělat neefektivní obstrukci, ale určitě to bude mnohem efektivnější, než to honit v cyklu přes PHP. Pokud bychom použili omezující podmínku na zmíněné 3 IP, efektivní to bude i pro velký objem dat.

[otula]

beginTransaction() před tím není

[otula]

Jej, u té druhé metody jsem dal blbě syntaxi, ale stejně nefunguje - vůbec tuto databázi nerozezná

[Kit]

A co vypíše

print_r($dbh->errorInfo());

?