Podstrony
- telcocafe
- [PHP+MySQL] Problem z połączeniem mysqli('localhost', 'xxx', 'xxx', 'xxx'
- [PHP] Wyrażenia regularne. Zamienić name="info_imie" na name="{info_imie}"
- [php] Dodanie textu do pliku. Problem... :) POMOCY!!!!!!!!!!!!!
- [PHP] Skrypt cenzury Chcę cenzury!!!
- Dobre programy do edycji php Dobre programy do edycji php
- [PHP] podział na strony php: podział na strony
- [PHP i MySQL] jak pobrac liczbe rezultatow Jak pobrac liczbe rezultatow???
- jak zrobic aktualizacje php? pomocy!!!!!
- Błąd w PHP-Fusion pomocy !!!!!
- Daczego nie dziala?? skrypt php nie chce działać!!!
- zanotowane.pl
- doc.pisz.pl
- pdf.pisz.pl
- granada.xlx.pl
telcocafe
Macie jakieś pomysły co do optymalizacji poniższego kodu? Chdzi mi o przyspieszenie działania i poprawienie zabezpieczeń. Poza tym skrypt działa i każdy może z nigo skorzystać. Oto ten kod:<?php include('../data/config.php'); $connection = mysql_connect($host, $username, $password); mysql_select_db($database, $connection); $acc = addslashes(strip_tags($_GET['acc'])); if(empty($acc)) { $query = 'SELECT id, tytul, priorytet FROM notes ORDER BY priorytet'; $result = mysql_query($query, $connection); echo '<tr><td width="500" class="nag">Notatnik</td><td align="center" width="124" class="nag"><a href="index.php?id=main&acc=new">Dodaj notatkę</a></td></tr></table>'; echo '<table border="0"><tr><td width="24" class="nag">Del</td><td width="576" class="nag">Tytuł</td><td width="24" class="nag"><a title="priorytet">P</a></td></tr>'; while($row = mysql_fetch_array($result)) { echo '<tr><td align="center" width="24" class="zaw"><form action="index.php?id=main&acc=del" method="post"><input type="hidden" name="id" value="'.$row['id'].'"><input type="submit" value="X" class="btt" style="width: 20px;"></form></td><td width="576" class="zaw"><a href="index.php?id=main&n='; echo($row['id']); echo '">'; echo($row['tytul']); echo '</a></td><td width="24" class="zaw">'; echo($row['priorytet']); echo '</td></tr>'; } echo '</table><br /><table border="0"><tr><td width="624" class="zaw">'; $n = addslashes(strip_tags($_GET['n'])); if (empty($n)) { echo 'Wybierz notatkę do wyświetlenia'; } else { $query = 'SELECT id, tytul, tresc FROM notes WHERE id='.$n.' LIMIT 1'; $result = mysql_query($query, $connection); while($row = mysql_fetch_array($result)) { echo 'Tytuł: <b>'; echo($row['tytul']); echo '</b><br /><i>'; echo($row['tresc']); echo '</i>'; } echo '</td></tr>'; } } elseif($acc == 'new') { echo '<tr><td width="500" class="nag">Notatnik</td><td align="center" width="124" class="nag"><a href="index.php?id=main">Przeglądaj</a></td></tr></table>'; echo '<table border="0"><tr><td width="624" colspan="2" class="nag">Dodaj notatkę</td></tr>'; echo '<tr><td width="624" colspan="2" class="zaw"> <form action="index.php?id=main&acc=add" method="post"> <input type="text" name="tytul" size="50" maxlength="255" class="txt"> <select name="priorytet" size="1" class="sel" style="width: 74px;"> <option value="1">wysoki</option> <option value="2">średni</option> <option value="3">niski</option> </select><br><br> <textarea name="tresc" cols="60" rows="5" wrap="off" class="txt"></textarea><br> <input class="btt" type="submit" value="Zapisz"> </form> </td></tr>'; } elseif($acc == 'add') { $tytul = addslashes(strip_tags($_POST['tytul'])); $tresc = addslashes(strip_tags($_POST['tresc'])); $priorytet = addslashes(strip_tags($_POST['priorytet'])); $query = "INSERT INTO notes ( id , tytul , tresc , priorytet ) VALUES ('', '$tytul', '$tresc', '$priorytet'); "; $result = mysql_query($query, $connection); echo '<tr><td width="500" class="nag">Notatnik</td><td align="center" width="124" class="nag"><a href="index.php?id=main">Przeglądaj</a></td></tr></table>'; echo '<table border="0"><tr><td width="624" colspan="2" class="nag">Dodaj notatkę</td></tr>'; echo '<tr><td width="624" colspan="2" class="zaw">Notatka została zapisana</td></tr>'; } elseif($acc == 'del') { $id = addslashes(strip_tags($_POST['id'])); $query = "DELETE FROM notes WHERE id=$id"; $result = mysql_query($query, $connection); echo '<tr><td width="500" class="nag">Notatnik</td><td align="center" width="124" class="nag"><a href="index.php?id=main">Przeglądaj</a></td></tr></table>'; echo '<table border="0"><tr><td width="624" colspan="2" class="nag">Kasowanie notatki</td></tr>'; echo '<tr><td width="624" colspan="2" class="zaw">Notatka o id = '.$id.' została usunięta</td></tr>'; } mysql_close($connection); ?>
Pozdrawiam i czekam na odpowiedź.
Po co dawać kilka razy echo:
echo 'Tytuł: <b>'; echo($row['tytul']); echo '</b><br /><i>'; echo($row['tresc']); echo '</i>';
jak można:
echo 'Tytuł: <b>' .$row['tytul']. '</b><br /><i>' .$row['tresc']. '</i>';
;)
Użytkownik Coldpeer edytował ten post 09 marzec 2006, 15:45
bo mam czytelniejszy kod. Myślisz, że to co napisałeś w jakiś sposób wpłynie na szybkość działania skryptu ?? Co sądzicie o konstrukcji zapytań do bazy ??