[php][html]Prośba o sprawdzenie, Proszę o sprawdzenie podanego skryptu Opcje

[obelix94]

Skrypt służy do zapisywania oceny, obliczania średniej, sprawdzania użytkowników, pokazywania średniej. Prosze o sprawdzenie i oceny. PS: wykluczona jest baza danych

[PHP] pobierz, plaintext 
<?php
        //sprawdzenie ciasteczka
        $wiersz = @ $_GET['wiersz'];
         if ($_COOKIE['obecny'] == 'byl'))
        {
              $status = 'glosowales';
          }
        else
        {
                        setcookie("obecny","byl",time() + 60 * 60 * 24 * 30); // 60s razy 60 minut razy 24h razy 30 dni
            //TRESC SKRYPTU
            //ustawianie zmiennych
            $plik_glosowan = 'zmienne/ilosc_glosowan'.$wiersz.'.txt';
                 $plik_sredniej = 'zmienne/srednia'.$wiersz.'.txt';
            $plik_wartosci = 'zmienne/suma_wartosci'.$wiersz.'.txt';
            //dodawanie ilosci, zapisywanie
            $ilosc_glosowan = file_get_contents($plik_glosowan);
            $ilosc_glosowan = $ilosc_glosowan + 1;
            file_put_contents($plik_glosowan, $ilosc_glosowan);
 
            //suma glosow
            $wartosc = $_POST('wartosc');
               $ilosc_wartosci = file_get_contents($plik_glosowan);
            $ilosc_wartosci = $ilosc_glosow + $wartosc;
            file_put_contents($plik_wartosci, $ilosc_wartosci);
 
            //liczenie sredniej
               $ilosc_glosowan = file_get_contents($plik_sredniej);
            $srednia = $ilos_wartosci / $ilosc_glosowan
            file_put_contents($plik_sredniej, $srednia);
        }
 
        if($status == 'glosowales'){
            echo('<div class="secondaryContentBox"><h2>Oceń wiersz</h2><p><font color="red">GŁOSOWAŁEŚ!</font></p> ');
            //ciag dalszy (drugi skrypt)
                        echo('<div id="linia"><br></div>');
            echo('<h2>Średnia ocen</h2><p>')
            $srednia = file_get_contents($plik_sredniej);
            echo($srednia.'</p>');
        }
        else{
            echo('
            <div class="secondaryContentBox">
            <h2>Oceń wiersz</h2>
            <p>');
            echo('<form action="index.php?id=biblio1&lang=pl&wiersz=
            echo($wiersz);
            echo('" method="post"><p>Twoja ocena<br>');
            echo('<input type="radio"  name="wartosc" value="6"  />6<br>
            <input type="radio" name="wartosc" value="5" checked="checked" />5<br>
            <input type="radio" name="wartosc" value="4"  />4<br>
            <input type="radio" name="wartosc" value="3"  />3<br>
            <input type="radio" name="wartosc" value="2"  />2<br>
            <input type="radio" name="wartosc" value="1"  />1<br>
            <input type="submit" style="width:10em; background-color:white; border: 2px solid green; " value="WY&#346;LIJ" name="ok" /></form>
            </p>');
 
 
            //ciag dalszy (drugi skrypt)
                        echo('<div id="linia"><br></div>');
            echo('<h2>&#346;rednia ocen</h2><p>')
            $srednia = file_get_contents($plik_sredniej);
            echo($srednia.'</p>');
            
        }
 
 
?>
[PHP] pobierz, plaintext 

Nie ponoszę odpowiedzialnosci za zniekształcenie w tabulacji oraz wygladzie skryptu.

Ten post edytował obelix94 12.12.2008, 19:54:38

[Pride]

1.Tak, jak powyżej - dlaczego zamykasz echo w echu? 0o
2. Dlaczego robisz tak:

[PHP] pobierz, plaintext 
<?php
echo('Linia 1');
echo('Linia 2');
 
//A nie po prostu tak:
echo '
    Linia1 
    Linia2
    Linia3
    Linia4';
?>
[PHP] pobierz, plaintext 

Bez tych ( ) oraz zamykania każdej linijki w osobnym echo

3. Jeżeli nie rozumiesz co jest napisane w manualu to... Żegnamy Cię serdecznie na minimum najbliższy rok/dwa lata. (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif) Nie ma sensu, abyś bawił się w programowanie, jeżeli nie masz "możliwości umysłowych" do tego. Najczęściej, gdy ktoś nie wie jak wykonać "to coś", to podawany mu jest link do manual" title="Zobacz w manualu PHP" target="_manual z daną funkcją, która jest w stanie wykonać daną operację. Tak samo jak było w twoim przypadku.

Pozdrawiam.