[PHP] pobierz, plaintext 
<?php
$szukaj=mysql_real_escape_string($_POST['szukaj']);
$result = mysql_query('SELECT login FROM userdata WHERE szukaj = $szukaj');
$sql=&#092;"SELECT * FROM userdata WHERE login LIKE '%$szukaj%' \";
$result2 = mysql_query($sql);
 
while ($wiersz = mysql_fetch_row($result2)) 
{
  echo &#092;"
 
login: $wiersz[1] |email: $wiersz[3] />&#092;";
}
 
?>
[PHP] pobierz, plaintext 

jak można to w miare zoptymalizować nie tylko w formie poprawnośći kodu ale np. żeby poustawiać wyniki $wiersz[1] w divach bez echo? Odczyt danych z bazy mam taki:

[PHP] pobierz, plaintext 
<?php
$query = mysql_query(&#092;"SELECT * FROM userdata WHERE login = '$login'\");
$dane = mysql_fetch_array($query);
$login = $dane[login];
$email = $dane[email]
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$szukaj=mysql_real_escape_string($_POST['szukaj']);// bo cie h4k3rzy zjedza jak tak bedziesz wciagal niezabezpieczone dane
$result = mysql_query('SELECT login FROM userdata WHERE szukaj = "'.$szukaj.'"');
$sql="SELECT * FROM userdata WHERE login LIKE '%".$szukaj."%' ";
$result2 = mysql_query($sql);
 
while ($wiersz = mysql_fetch_row($result2)):?>
<div>login: <?=$wiersz[1];?> |email: <?=$wiersz[3];?> </div>";
<?php
endwile;
 
?>
[PHP] pobierz, plaintext 

nie wiem czy o to Ci chodzi

tak jakby, wcześniej ktoś mi napisał że lepiej dla bazy jest wczytwać dane tak:

[PHP] pobierz, plaintext 
<?php
$dane1 = mysql_fetch_array($query);
$login1 = $dane1[login];
$email1 = $dane1[email]
?>
[PHP] pobierz, plaintext 

tylko nie wiem jak teraz się do tego odwołać aby zastąpić

[PHP] pobierz, plaintext 
<?=$wiersz[1];?>
[PHP] pobierz, plaintext 

tak:

[PHP] pobierz, plaintext 
<? $dane1[email] ?>
 
albo 
 
<? $email1 ?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$szukaj=mysql_real_escape_string($_POST['szukaj']);
$result = mysql_query('SELECT login FROM userdata WHERE szukaj = $szukaj');
$sql="SELECT * FROM userdata WHERE login LIKE '%$szukaj%' ";
$result2 = mysql_query($sql);
$iles = mysql_num_rows($result2);
 
$dane1 = mysql_fetch_array($query);
$login1 = $dane1[login];
$email1 = $dane1[email]
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$szukaj=mysql_real_escape_string($_POST['szukaj']);
$result = mysql_query('SELECT login FROM userdata WHERE szukaj = $szukaj');
$sql= "SELECT * FROM userdata WHERE login LIKE '%$szukaj%'";
$result2 = mysql_query($sql);
while ($wiersz = mysql_fetch_array($result2)) 
{
echo 'Login:'.$wiersz['login'].' | email: '.$wiersz['email'].'';
}
?>
[PHP] pobierz, plaintext 

Bez echo, a jak chcesz wyświetlić dane?

umieszczanie w echo div ipt. jest dobrym rozwiązaniem?

A dlaczego nie. To od ciebie zalezy jak będzie wyświetlane.

[PHP] pobierz, plaintext 
<?php
echo '<div class="profil"><b>Login:'.$wiersz['login'].' | email: '.$wiersz[email].'</b></div>';
?>
[PHP] pobierz, plaintext 

Zastosowałem tylko jeden apostrof, bo w twoim wypadku nic by nie wyświetliło.

Możesz dać jeszcze naprzykład <li></li>, jako lista.