Kod php - uproszczenie Opcje

[kill15]

Napisałem taki kod:

[PHP] pobierz, plaintext 
 while($nwyb1 = mysql_fetch_assoc($wszystkie))
      {
 
      if($nwyb1[nazwa] == $infok)
        {
        echo " <option  value=\"$nwyb1[nazwa]\" selected=\"selected\" />$nwyb1[nazwa]</option>";
        }
        elseif(!isset($infok))
        {
          if(!isset($spr))
          {
          $spr = 'istnieje';
          echo " <option  value=\"\" selected=\"selected\" /></option>";
          echo " <option  value=\"$nwyb1[nazwa]\" />$nwyb1[nazwa]</option>";
          }
        }
        else
        {
          if(!isset($spr))
          {
          $spr = 'istnieje';
          echo " <option  value=\"\" /></option>";
          }
        echo " <option  value=\"$nwyb1[nazwa]\" />$nwyb1[nazwa]</option>";
        }
      }
[PHP] pobierz, plaintext 

On działa ale czy dało by się to jakoś uprościć
Wg mnie ten kod nie jest zbyt dobry.
Jak można zauważyć chodzi mi tu o puste pole (option) w pętli no i oczywiście select do tego ;>

Ten post edytował kill15 1.05.2011, 10:38:54

[darko]

Nie

[PHP] pobierz, plaintext 
echo " <option  value=\"$nwyb1[nazwa]\" selected=\"selected\" />$nwyb1[nazwa]</option>";
[PHP] pobierz, plaintext 

a

[PHP] pobierz, plaintext 
echo "<option  value=\"" . $nwyb1['nazwa'] . "\" selected=\"selected\" />" . $nwyb1['nazwa'] . "</option>";
[PHP] pobierz, plaintext 

i analogicznie dalej.

[kill15]

Czy to zwiększy jakoś bezpieczeństwo kodu??

[Spawnm]

@Darko -
nie

[PHP] pobierz, plaintext 
 echo "<option value=\"" . $nwyb1['nazwa'] . "\" selected=\"selected\" />" . $nwyb1['nazwa'] . "</option>";
[PHP] pobierz, plaintext 

tylko

[PHP] pobierz, plaintext 
echo '<option value="' . $nwyb1['nazwa'] . '" selected="selected" />'. $nwyb1['nazwa'] . '</option>';
[PHP] pobierz, plaintext 

[darko]

@Spawnm: Można i tak i tak, nie ma większej różnicy, chociaż Twoja wersja ładniej wygląda i powinno się ją stosować.
@kill15: nie chodzi o bezpieczeństwo, ale o poprawne wyświetlenie danych; zakładając że nie masz nigdzie zdefiniowanej stałej nazwa, sprawdź różnicę (var_dump) pomiędzy zapisem:
$nwyb1[nazwa]
a
$nwyb1['nazwa']

Ten post edytował darko 1.05.2011, 13:07:18

[Spawnm]

Darko - jest różnica, dając " " informujesz php że mogą w stringu występować zmienne, czyli kod jest wolniejszy od ''.

Co do kodu który pokazał ~kill15 warto dodać że tablice w "" obejmujemy {}.

[Bags_Bunny]

@darko: jeśli:

[PHP] pobierz, plaintext 
echo "<option  value=\"" . $nwyb1['nazwa'] . "\" selected=\"selected\" />" . $nwyb1['nazwa'] . "</option>";
[PHP] pobierz, plaintext 

to czemu nie po prostu:

[PHP] pobierz, plaintext 
echo "<option  value=\"{$nwyb1['nazwa']}\" selected=\"selected\" />{$nwyb1['nazwa']}</option>";
[PHP] pobierz, plaintext 

Jaki jest tutaj sens konkatenacji, skoro zmienne są parsowane wewnątrz cudzysłowów?

Kod Spawnma jest najczystszy, najprostszy i nie wprowadza zbędnego zamieszania. To jest różnica. Uczmy początkujących dobrych nawyków.

[darko]

Ja nie neguję tego, co napisał @Spawnm, wręcz się z tym zgadzam. Kod w poście sformatowałem tak, a nie inaczej (czyt. lepiej) z czystego lenistwa, przerabiając fragment podany przez autora tematu. Nie prowadźmy sporu o pietruszkę.