Diskuze: Problém s filtrováním obsahu databáze

shakki

Člen

shakki:29.12.2013 15:01

Zdravím, dělám jednoduchý databázový filtr pro sbírku hokejových karet, který funguje na principu vyhledávání společného textu. Na netu jsem našel množství filtrů, ale vzhledem k tomu, že s ním teprv začínám, zkusil jsem něco jednodušího. Krok po kroku jsem nastavil vlastnosti filtru v souboru, uložil a chtěl vyzkoušet. Zadal jsem tedy hledaný výraz, ale tabulka, kterou jsem součástí filtru nechal vypsat se nevyfiltrovala. Připojuji kod.

<form name="search_form" method="POST" action="hledani.php">
Hledej: <input type="text" name="search_box" value=" "/>
<input type="submit" name="search" value="Hledat"/>
</form>

<?php
$link = mysql_connect("mysql.wz.cz", "---", "---")
or die("Nelze se připojit: " . mysql_error());
mysql_select_db("shakaar") or die("Nelze vybrat databázi");
$query = "SELECT * FROM HokejKarty";
$result = mysql_query($query) or die("Dotaz nelze provést: " . mysql_error());

if(isset($_POST['search'])) {
$search_term = mysql_real_escape_string($_POST['search_box']);
$query .= "WHERE Vyrobce = '{$search_box}'";
$query .= "OR Zeme = '{$search_box}'";
}
?>
<table width="70%" cellpadding="5" cellspacing="5">
<tr>
<td><b>id</b</td>
<td><b>Znacka</b></td>
<td><b>Cislo</b></td>
<td><b>Zeme</b></td>
<td><b>Rok</b></td>
<td><b>Nazev</b></td>
</tr>
<?php
while ($row= mysql_fetch_array($result)) { ?>
<tr>
<td><?php echo $row ['id']; ?></td>
<td><?php echo $row ['Vyrobce']; ?></td>
<td><?php echo $row ['Cislo']; ?></td>
<td><?php echo $row ['Zeme']; ?></td>
<td><?php echo $row ['Rok']; ?></td>
<td><?php echo $row ['Nazev']; ?></td>

<?php } ?>

</table>

Editováno 29.12.2013 16:34

Odpovědět

29.12.2013 15:01

Kit

Tvůrce

Kit:29.12.2013 15:08

Skript v PHP musí být na začátku souboru "hledani.php". Pro vyhledávání se zásadně používá metoda "GET".

Nahoru Odpovědět

29.12.2013 15:08

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

shakki

Člen

shakki:29.12.2013 15:36

dík za radu, chtěl jsem se ještě zeptat, který z těch scriptů. Ten s výpisem tabulce asi ne, takže předpokládám že máš na mysli připojující skript s podmínkou. Změnil jsem to tedy podle toho a nestalo se zase nic.

Nahoru Odpovědět

29.12.2013 15:36

David Hartinger

Vlastník

David Hartinger:29.12.2013 15:38

V query používáš nějakou proměnnou search_box, ale máš tam jen search_term. Kdybys používal prepared statements s PDO, tak bys tu chybu asi neudělal.

Nahoru Odpovědět

29.12.2013 15:38

New kid back on the block with a R.I.P

FastNode

Tvůrce

FastNode:29.12.2013 16:26

Příště až budeš někde publikovat kód alespoň vyhvězdičkuj nebo úplně smaž přístupové údaje do databáze, speciálně v případě kdy je administrace databáze veřejně přístupná. Takhle se ti kdokoliv může hrabat v databázi!

Nahoru Odpovědět

29.12.2013 16:26

Michal Žůrek - misaz

Tvůrce

Michal Žůrek - misaz:29.12.2013 16:29

prosímtě změň si heslo k db.

Nahoru Odpovědět

29.12.2013 16:29

Vojta Pšenák

Tvůrce

Vojta Pšenák:29.12.2013 16:33

Mimochodem, měl bys ošetřit hesla, jsou volně přístupná v databázi. Taky by nemělo být možné zaregistrovat se s prázdným jménem/heslem/emailem. Nevím jestli se ti v tom někdo hrabal, ale máš v databázi hroznej bordel

Nahoru Odpovědět

29.12.2013 16:33

David Hartinger

Vlastník

David Hartinger:29.12.2013 16:35

Toho jsem si ani nevšiml, už jsem mu to z toho příspěvku vymazal, stejně to ale určitě vidělo plno lidí. Alespoň si to bude pamatovat a neudělá podobnou hloupost až opravdu půjde o důležitý projekt.

Nahoru Odpovědět

29.12.2013 16:35

New kid back on the block with a R.I.P

Kit

Tvůrce

Kit:29.12.2013 16:37

V důležitých projektech se přihlašovací údaje do skriptů nedávají.

Nahoru Odpovědět

29.12.2013 16:37

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

shakki

Člen

shakki:29.12.2013 16:51

Díky za upozornění, zjistil jsem to hned jak ten kod odeslal a bohužel jsem to tady už nemohl vymazat. Heslo jsem změnil, udělal jsem pěknou hovadinu :/.

Nahoru Odpovědět

29.12.2013 16:51

Naučíme tě pracovat na home-office.

Více info »

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 10 zpráv z 10.