Dzien dobry,

Napisałem sobie prosty skrypt, który ma za zadanie przedstawić efekt taki jak na screenie:



Mimo iż skrypt działa, to nie jestem pewny czy jest on napisany w sposób dobry, i chciałbym prosić o pomoc czy takie rozwiązanie da się uzyskać w prostszy sposób.

Tutaj kod:

[PHP] pobierz, plaintext 
<?php
  $wb = fopen("plik.txt" , "rb");
  $dostep = file("plik.txt");
 
  $a = 1;
  $c = 1;
  $i = 0;
  $b = 0;
 
  while(!feof($wb)){
    $tak = fgets($wb);
    $b++;
  }
 
  while($i<$b-1){
    echo "Zamówienie nr. " . $a++ . "<br />";
    echo $dostep[$i] . "<br />";
    echo "Adres klienta: " . $dostep[$c] . "<br /><br />";
    $c += 2;
    $i += 2;
  }
?>
[PHP] pobierz, plaintext 

Powinny być obiekty klasy Order. Numer zamówienia powinien być cechą danego zamówienia, a nie wynikać z tego, na którym miejscu w pliku jest zamówienie. Przydałyby się nazwy angielskie zmiennych typu https://pl.wikipedia.org/wiki/CamelCase

Wyjątek zmienna $i, bo to iterator.

I te nazwy zmiennych trzeba z sensem ponazywać i nie powinny one mieć błędów ortograficznych ani być skrótami.

Owszem możesz przyjąć, że to będzie polski kod i nieobiektowy, ale wtenczas daleko nie zajedziesz

@trzczy ale ty tak na powaznie? Naprawde nie widzisz ze kolega dopiero zaczyna sie uczyc jezdzic maluszkiem a ty mu z promem kosmicznym wyjezdzasz?
Przejscie na obiektowke w zaden sposob nie poprawi w magiczny sposob bledow, ktora tu popelnil

@Mikijak popelniles calkiem sporo bledow jak na tak krotki kod
NIe potrzebnie czytasz dwa razy plik. Naprawde raz wystarczy
Tworzysz cala mase zbednych zmiennych

To powinno wygladac mniej wiecej tak

[PHP] pobierz, plaintext 
  $dostep = file("plik.txt");
 
$ilosc = count($dostep);
 for ($i = 0; $i < $ilosc; $i+= 2) 
 {
    echo "Zamówienie nr. " . ($i / 2 + 1). "<br />";
    echo $dostep[$i] . "<br />";
    echo "Adres klienta: " . $dostep[$i+1] . "<br /><br />";
  }
[PHP] pobierz, plaintext 

I juz.

@nospor dzięki wielkie