[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]

No właśnie nie działa. Ten jeden błąd to...

Czytałem, że nalezy dbać o wygląd i funkcjonalnosć do przeróbki. Mimo że to pierwszy kod który tego wymagał, zastosowałem się do porady sprzed roku. http://www.fotosik.pl/pokaz_obrazek/8a62d48843c09082.html. Odpowiedź: forum.

[Kildyt]

Czytałem, że nalezy dbać o wygląd i funkcjonalnosć do przeróbki. Mimo że to pierwszy kod który tego wymagał, zastosowałem się do porady sprzed roku. http://www.fotosik.pl/pokaz_obrazek/8a62d48843c09082.html. Odpowiedź: forum.

Zamieszczony kod na screenie IMHO nie jest poprawny. Nie zagłębiałem się w tajne techniki tabulacji (jeżeli takie istnieją). Mam tylko prostą zasadę: coś jest w czymś to ma wcięcie w prawo - proste.

Wybacz, ale zamieszczony przez Ciebie kod nie jest zbyt ładny, a można to zauważyć już w pierwszej linijce kodu. Dbaj o kod i pamiętaj, że nie tworzysz go tylko "na teraz" a i "na jutro" i dla innych programistów.

Nie mieszaj kodu HTML z PHP! Ucz się na błędach (młodzieniec, który napisał tego posta nauczył się na błędzie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) ). Tak zorganizuj sobie skrypt, żeby było jasno pokazane co to HTML, a co PHP. Oba języki mają różne przeznaczenie.

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!

Życzę powodzenia! (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

PS Staraj się przedstawiać jaśniej swoje sprawy na forum. Nie możesz oczekiwać, że za każdym razem będziemy sprawdzać, czy kod jest poprawny. Przeglądaj forum i zobacz jak wyglądają porządne skrypty.