Forum PHP.pl > [PHP]Czy ten kod jest dobry ?

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]Czy ten kod jest dobry ?

marian2299

29.06.2009, 12:04:27

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

wizard144

29.06.2009, 12:18:02

[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

29.06.2009, 12:19:34

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

29.06.2009, 14:31:39

[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 '; ?>
?

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.