[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

[erix]

I się ukaże... nic.

A czytałeś rozdział dotyczący obsługi plików? Bo widzę, że nie; np. do fwrite" title="Zobacz w manualu PHP" target="_manual podstawiasz jeden parametr zamiast dwóch...

Ogólnie, to wynajdujesz koło na nowo; męczysz się z zapisem/odczytem obsługując za każdym razem z osobna. A są metody file_get_contents" title="Zobacz w manualu PHP" target="_manual i file_put_contents" title="Zobacz w manualu PHP" target="_manual, które nie dość, że wygodniejsze, to nieraz wydajniejsze ze względu na wykorzystywanie mapowań pamięci. Ale do tego, to już przydałaby się lektura manuala, którego chyba nie znosisz. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Zamiast font to p color

Zapomnij o font; to relikt przeszłości. Ucz się CSS.

Unikaj do tego stosowania małpiszonów przed funkcjami; masz eliminować błędy, a nie je ukrywać.