[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

[wookieb]

Dziecko.. Czy ty naprawde jesteś tak niedowartościowany, że musisz wrzucać do oceny pare linijek kodu?
Co mamy ci powiedzieć. "Wow ładną czcionkę dobrałeś? Choć tutaj to nie ma żadnego znaczenia...".

Jeżeli działa to działa. Ciesz się.
Poza tym jeżeli masz php5 to uzywaj file_put_contents. Po co się męczyć.
Kod i tak jest brzydki bo gdybyś wlaczyl raportowanie

Kod

error_reporting(E_ALL);

to miałbyś chociaż 1 błąd który widze odrazu. Po drugie nie wiem czy to forum czy ty ale nie ma odpowiednich wcięć.

Ten post edytował wookieb 11.12.2008, 16:41:04

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

[obelix94]

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 rozumiem. Mozecie mi onjaśnić wasze.... skróty myślowe. (IMG:http://forum.php.pl/style_emoticons/default/blinksmiley.gif)

[erix]

Czytaj: inni się uczący mogą kiedyś natrafić na Twój listing z pierwszego posta i znowu będą zadawać pytania, na które była już odpowiedź...

~obelix94, taka dobra rada - więcej samodzielności, ściągnij jakieś warte uwagi skrypty PHP (np. źródła frameworków, popatrz, jak tam kod jest sformatowany, podzielony, jak zaplanowano logikę) i ucz się wg tego. Jak już kiedyś powiedziałem - nie wal z byle pierdołą na forum. Fakt, każdy kiedyś zaczynał, ale zważ na to, że niektórzy w ogóle nie mieli takiego luksusu jak Ty, aby zadawać takie pytania-błahostki. Więcej konsekwencji.

Ten post edytował erix 11.12.2008, 21:15:26

[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

[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ć.

[obelix94]

Nieznosze bo jest po angielsku. Niby 5 z angola, a jak przychodzi co do czego...
Nie gadajcie ze jest po polsku, bo to co potrzebuje to wpisać pl.php.net czy php.net.pl to i tak niekiedy jest angielski.
Z tego co zrozumiałem z manuala (na dodatek pl) po angielsku, to to ze file_put_contents otwiera, "upuszcza" "zapisuje" zamyka. Z file_get_contents nic nie zrozumiałem, ale na logikę - pobiera dane.

int file_put_contents ( string $filename , mixed $data [, int $flags [, resource $context ]] )
filename: rozumie, data: ?, flags: rodzaj, tryb, context: ciąg do zapisania (chyba)
dobrze napisałem?

Ten post edytował obelix94 12.12.2008, 17:13:43

[erix]

Nieznosze bo jest po angielsku

To daruj sobie programowanie i nie siadaj do komputera, bo 95% dokumentacji jest po angielsku. Tak nawiasem, jeśli przyszłoby Ci kiedyś zdawać ZCE, to po polsku jest tylko "dzień dobry" od pana, co Cię pilnuje, a reszta tylko i wyłącznie po angielsku/francusku/hiszpańsku. Nie znasz angielskiego, będziesz miał duży problem.

file_put_contents otwiera, "upuszcza" "zapisuje" zamyka.

Po prostu zapis danych do pliku.

file_get_contents nic nie zrozumiałem, ale na logikę - pobiera dane.

Tak

filename: rozumiem, data: ?, flags: rodzaj, tryb, context: ciąg do zapisania (chyba)

data = dane.

Bierz się za angielski, bo będziesz zadawał masę niepotrzebnych pytań.

[obelix94]

Nieznosze bo jest po angielsku. Niby 5 z angola, a jak przychodzi co do czego...

THX (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Jeszce context, skoro data to dane.

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

[Mize]

Człowieku, brak mi słów... wróć za rok, dwa i pochwal się czymś czego się nauczyłeś.
Nie wiem jak można być w drugiej klasie gimnazjum tak nierozgarniętym ludkiem...
Znajdź jeden swój topic z problemem, na który nie ma rozwiązania w pierwszych trzech wynikach z google'a albo w manualu. ;s

Myślę, że rok sam na sam z manualem, dobrze Ci zrobi.

[obelix94]

Takie osobiste pytanko: skąd wiesz, że jestem w drugiej gimnazjum?

[Mize]

OT: Tak wnioskuje po Twoim nicku, a jeśli masz więcej to się lepiej nie przyznawaj. ;o

Zeby nie bylo, ze sam offtopic to dodam, że lepiej spędzić nad kodem tydzień, niż pół godziny.
Bo czytelność i elastyczność kodu powinny być na równi rozwijane z funkcjonalnością.

[obelix94]

Poprawiłem swój kod. Myślę że bardziej się wam podoba. Niestety nie działa. Jak ustawiam error reporting e-all, też nic nie pokazuje. Błąd polega na tym, że nie zwraca wyników i zatrzymuje reszte skryptów znajdujacych się pod nim.

Ten post edytował obelix94 12.12.2008, 19:53:34

[erix]

[PHP] pobierz, plaintext 
<?php
if ($_COOKIE['obecny'] == 'byl'))
       {
             $status = 'glosowales';
         }
       else
       {
?>
[PHP] pobierz, plaintext 

W klamrach też panują pewne reguły. Widziałem na niektórych screenach, że używasz czegoś w stylu PHPEditor. Ściągnij sobie Komodo Edit, będziesz widział, gdzie masz błąd parsera.

Myśl więcej, mniej gadaj.

[obelix94]

Dziekuje. Ten twój "styl na phpeditor" to najzwyklejszy notatnik + kolorowanie składni i odwracanie ś,ź,ę itp na "dziwne znaczki"


CZY MOŻE KTOŚ POMÓC? POWIEDZIEĆ CZEMU TO NIE DZIAŁA?

Doszedłem do wniosku, ze to w tej części kodu coś jest nie tak. Ale za chiny nie mogę się dopatrzeć co jest źle. e reporting nic nie daje.

[PHP] pobierz, plaintext 
<?php
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 

Ten post edytował obelix94 13.12.2008, 16:43:32

[Lion_87]

echo('<form action="index.php?id=biblio1&lang=pl&wiersz=

Linia 16 to taki jakis specjalny efekt ?

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

[Ociu]

Crossing post. Zamykam.