🚀 go-pugleaf

 1. function cos() {
 2.     $i = 0;
 3.     while ($i <= 10)
 4.         $tmp[$i][1] = $i;
 5.         $tmp[$i][2] = $i;
 6.         $tmp[$i][3] = $i;
 7.         $tmp[$i][4] = $i;
 8.         $tmp[$i][5] = $i;
 9.         $i += 1;
10.     }
11.     return $tmp;
12. }

ad. 1: funkcja o nazwie 'cos' już istnieje.
ad. 2, 3, 9: zamiast ręcznie inicjować, sprawdzać
    warunek oraz inkrementować zmienną iterującą
    lepiej byłoby użyć specjalnie do tego przeznaczonej
    konstrukcji 'for'.
ad. 3-8: dlaczego nie ma tutaj zastosowanych parametrów,
    tylko funkcja zwraca _zafiksowane_ wartości?!
    Tak nie powinno się programować, ponieważ funkcje
    nie będą przydatne - nie przewidują każdej dostępnej
    kombinacji. To jak w matematyce -- funkcje powinny
    być jak wzory, przez które jedynie coś się przetwarza.
    Czyli nie:
        function zwróćPoleTrójkąta() { return 5;}
    ...lecz:
        function poleTrojkata($a, $h) {
            return (($a / 2) * $h);
        }
    Rozumiesz? To są podstawy programowania!
ad. 9: jeśli stosować skróty, to na całego! Dlaczego
    używasz operatora skróconego '+=' skoro mógłbyś
    zastosować '++'?

Na koniec wskazówka, że ja bym tą funkcję napisał tak:

    function foo($n, $m) {
        for ($i = 0; $i <= $n; $i++) {
            for ($j = 1; $j < $m + 1; $j++) {
                $tmp[$i][$j] = $i;
            }
        }
        return $tmp;
    }
    $result = foo(10, 5);

...co prawda nie widzę zastosowania :)

--
l9