[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

[obelix94]

Zamieszczony kod na screenie IMHO nie jest poprawny.
Wybacz, ale zamieszczony przez Ciebie kod nie jest zbyt ładny,  (...)
Nie mieszaj kodu HTML z PHP! Ucz się na błędach ((..)). Tak zorganizuj sobie skrypt, żeby było jasno pokazane co to HTML, a co PHP.
Echo to nie jest funkcja! Kurs, który czytałeś mógł zbyt przestarzały. Zaglądnij do dokumentacji PHP.
Znacznik font jest przestarzały!
Nie przestrzegasz tabulacji!

Dlaczego nie jest ładny?
Jaki był twój błąd?
Zamiast echo to print_r; tak?
Zamiast font to p color
Uważam ze przestrzegam

Co do ciebie erix, dzieki za poradę, na pewno się do tego zastosuje. Pozdrawiam.

PS: strona dostępna jest pod, STRONA wtedy należy kliknąć wiersze (biblioteka -> wiersze tu. I się ukaże... nic. A powinien być tekst jak na stronie głównej. Po dodaniu tego skryptu, wcześniejsza zawartosć się nie wyswietla. Jak komuś pomoże: skrypt ładowny jest z index.php kodem:

[PHP] pobierz, plaintext 
<?php
$szare = @ $_GET['id'];
if($szare=='biblio1'){
@ include('specjalne/oceny.php');
}
else{
@ include('glowne/aktualnosci.html');
}
 ?>
[PHP] pobierz, plaintext 

Zaś główna strona (treść) za pomocą switch, w indexie.

Ten post edytował obelix94 12.12.2008, 09:56:39