[PHP] Problem z odpowiednim wyświetleniem parzystości

Question

Witajcie mam takie zadanie aby napisać skrypt który wyświetli zawartość dwóch plików .php . Pierwszy z nich ma wyświetlić sumę dziesięciu losowo wygenerowanych liczb. Drugi ma wyświetlić czy ta suma jest parzysta czy nieparzysta. Cóż po dłuższym rozmyślaniu jakoś to napisałem, jednak to nie jest to czego oczekiwałem. Stworzyłem 3 pliki cwiczenie.php, suma.php i parzystosc.php. Co prawda wyświetla się parzystość jednak jest to odnośnie tej samej funkcji losuj() z pliku  suma.php. Aby to wszystko w ogóle działało musiałem skopiować częściowo zawartość funkcji losuj () do funkcji parzystość().  Nie mam pojęcia co zrobić aby warunki w funkcji parzystość() wyświetlały się w stosunku do TEJ SAMEJ SUMY. Oto linie kodu :

 

cwiczenie.php 

echo "Plik zawierający sumę wylosowanych liczb: <br><br>";
    include("suma.php");
     losuj();

  echo"<br><br>Plik sprawdzający parzystość sumy wylosowanych liczb <br><br>";
    include("parzystosc.php");
     parzystosc();

 

suma.php

 

function losuj()
     {
       $suma = 0;

       for($i=1; $i<=10; $i++)
         {
            $x = rand(1, 15);
            $suma = $suma + $x;
            echo $x." + ";
         }
        echo"0 = $suma";     
       
     }

 

parzystosc.php

 

function parzystosc()
     {
         
        $suma = 0;
        
       for($i=1; $i<=10; $i++)
         {
            $x = rand(1, 15);
            $suma = $suma + $x;
            
         }
      
        
        if($suma % 2 == 0)
          {
            echo " jest to liczba parzysta";
          }
        
        else if ($suma % 2 > 0)
          {
            echo " nie jest to liczba parzysta";
          }
     }

Answer 1

zwracaj sobie tą sumę z losuj() do jakiejś zmiennej, a w przystosc($liczba) podaj ją jako argument funkcji;

P.S.
po co w parzystosc() masz else if? skoro wiadomo, że jak nie jest parzysta o jest nieparzysta ;)

Comments

Właśnie na tym polega problem że jak wymażę funkcję losuj z parzystosc() to mi wywala błąd. Nie bardzo rozumiem co masz na myśli mówiąc abym dodał $liczba jako agument funkcji parzystosc().

---

cwiczenie.php 
 
include("suma.php");
include("parzystosc.php");

echo "Plik zawierający sumę wylosowanych liczb: <br><br>";
     $sum = losuj();

echo"<br><br>Plik sprawdzający parzystość sumy wylosowanych liczb <br><br>";
     parzystosc($sum);

suma.php
function losuj(){
       $suma = 0;

       for($i=1; $i<=10; $i++){
            $x = rand(1, 15);
            $suma += $x;
            echo $x." + ";
       }
       echo "0 = $suma";     
       return $suma;
}


parzystosc.php
function parzystosc($suma){
        if($suma % 2 == 0)
            echo " jest to liczba parzysta";
        else
            echo " nie jest to liczba parzysta";
}

 

---

Dzięki serdeczne działa ;). Według tego co napisałeś to zadecydowała zmienna $sum. Dziwne jest dla mnie to że trzeba było użyć return $suma; . Mam rozumieć że gdyby nie było return $suma; to funkcja nie zwróciłaby sumy tak jak powinna ? ;)

---

nie zwróciła by nic bo nawet nie wiadomo co miałaby zwrócić, dokumentacja PHP na temat return => http://php.net/manual/pl/functions.returning-values.php