[PHP]Czy ten kod jest dobry ?

[PHP]Czy ten kod jest dobry ?

marian2299 Zobacz profil	29.06.2009, 12:04:27 Post #1
Grupa: Zarejestrowani Postów: 272 Pomógł: 9 Dołączył: 6.06.2009 Ostrzeżenie: (0%)	Cześć, Mam pytanie, czy kod poniżej jest dobry i czy można go jakoś... zoptymalizować ? [PHP] pobierz, plaintext <html> <?php echo "<link rel='stylesheet' type='text/css' href='".$imgs.$fimgs."default.css'>"; ?> <head> <title><?php echo $title; ?> - Przerwanie budowania</title> </head> <? if ($_SESSION["user"][4]) echo ' <body class="q_body"> <div align="center"> <table style="border-collapse: collapse" border="0" width="570"> <tr> <td width="570" height="366" align="center"> <form name="form1" method="post" action="check_d_all_.php?value=1"> <label> <br> Twoje hasło <input type="password" name="pass"> <br><br> <input type="submit" name="check" value="Sprawdź"> </label> </form> </td> </tr> </table> </div> </body>'; else echo 'Nie masz uprawnień.'; ?> </html> [PHP] pobierz, plaintext -------------------- film edit student

Odpowiedzi (1 - 3)

wizard144 Zobacz profil	29.06.2009, 12:18:02 Post #2
Grupa: Zarejestrowani Postów: 138 Pomógł: 3 Dołączył: 4.05.2009 Ostrzeżenie: (0%)	[PHP] pobierz, plaintext <html> <?php echo "<link rel='stylesheet' type='text/css' href='".$imgs.$fimgs."default.css'>"; ?> <head> <title><?php echo $title; ?> - Przerwanie budowania</title> </head> <? if ($_SESSION["user"][4]) echo ' <body class="q_body"> <div align="center"> <table style="border-collapse: collapse" border="0" width="570"> <tr> <td width="570" height="366" align="center"> <form name="form1" method="post" action="check_d_all_.php?value=1"> <label> <br> Twoje hasło <input type="password" name="pass"> <br><br> <input type="submit" name="check" value="Sprawdź"> </label> </form> </td> </tr> </table> </div> </body>'; else echo 'Nie masz uprawnień.'; ?> </html> [PHP] pobierz, plaintext Ja zawsze używam długich tagów <?php ?> to po 1... Po dwa do echo dałbym " " a nie ' ' Po 3 przy if/else proponował bym klamry ({ } ) dla zachowania przejżystości kodu... To chyba tyle...

osl Zobacz profil	29.06.2009, 12:19:34 Post #3
Grupa: Zarejestrowani Postów: 260 Pomógł: 41 Dołączył: 6.04.2009 Skąd: Gdańsk/okolice Ostrzeżenie: (0%)	specjalnie dużo tego php tu nie ma... ale znajdzie się coś co można zoptymalizować ;P [PHP] pobierz, plaintext <?php echo "<link rel='stylesheet' type='text/css' href='".$imgs.$fimgs."default.css'>"; ?> -> <?php echo '<link rel="stylesheet" type="text/css" href="'.$imgs.$fimgs.default.css">"; ?> [PHP] pobierz, plaintext konstrukcje typu "tekst" są odrobine wolniej parsowane, a to z powodu wyszukiwania zmiennych, czyli np. "tekst $zmienna" - $zmienna zostanie zastąpiona zawartością. A że i tak z tego nie korzystasz.... i druga rzecz: zamiast robić echo 'dłuższy tekst ...' lepiej wstawić go statycznie, czyli [PHP] pobierz, plaintext <?php if(1==2): ?> długi tekst w html <?php else:?> cos tam innego <?php endif; ?> [PHP] pobierz, plaintext w teorii będzie szybciej. w praktyce, i tak nie zauważysz różnicy w przypadku tego skryptu...

marian2299 Zobacz profil	29.06.2009, 14:31:39 Post #4
Grupa: Zarejestrowani Postów: 272 Pomógł: 9 Dołączył: 6.06.2009 Ostrzeżenie: (0%)	[PHP] pobierz, plaintext <?php if(1==2): ?> długi tekst w html <?php else:?> cos tam innego <?php endif; ?> [PHP] pobierz, plaintext O, podoba mi się to . Dzięki Aha, dlaczego proponujecie echo dawać w "" np tak:<? echo "TUTAJ NP: <p align='left'>sda</p>"; ?> Skoro łatwiej jest zrobić <? echo ' I tutaj `normalny` kod '; ?> ? -------------------- film edit student

1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)

0 Zarejestrowanych:

Tryb wyświetlania: Przełącz na: Standardowy · Linearny+ · Przełącz na: Drzewo

Aktualny czas: 20.08.2025 - 14:18

Hosting zapewnia

Forum PHP.pl