PHP - ocenianie Opcje

[MP1]

[php:1:1c149813d5]<?php
if($_GET['tocena']=="OK") {
if(file_exists($f."/".$plikl[$_GET['ii']]."o.txt")) { pobierzlcz(); } else { $licznik=$_POST['ocena']; }
zapiszlcz($licznik); echo 'Twoja ocena została dodana. Aktualna ocena: '; if(file_exists($f."/".$plikl[$_GET['ii']]."o.txt")) {
include($f."/".$plikl[$_GET['ii']]."o.txt"); }
else { echo "(BŁĄD)"; }
echo '<br /><span style="font-size: 15px">Okno zamknie się automatycznie za 3 sekundy.</span>'; }
else { echo '<form method="post" action="./ocen.php?tocena=OK&amp;idstr='.$_GET['idstr'].'&amp;ndk='.$_GET['ndk'].'&amp;pdk='.$_GET['pdk'].'&amp;ii='.$_GET['ii'].'">Twoja ocena: <input type="radio" name="focena" value="1" /> 1 &nbsp;<input type="radio" name="focena" value="2" /> 2 &nbsp;<input type="radio" name="focena" value="3" /> 3 &nbsp; <input type="radio" name="focena" value="4" /> 4 &nbsp;<input type="radio" name="focena" value="5" checked="checked" /> 5 &nbsp; <input type="submit" value="Oceń" /></form>'; }
function pobierzlcz() {
global $f; global $licznik; global $plikl;
$pliczek=fopen($f."/".$plikl[$_GET['ii']]."o.txt","r");
flock($pliczek,1);
$licznik=fgets($pliczek);
flock($pliczek,3);
fclose($pliczek);
$$licznik=round($licznik + $_POST['ocena'] / 2 , 1); }
function zapiszlcz($ocena1) {
global $f; global $plikl;
$pliczek=fopen($f."/".$plikl[$_GET['ii']]."o.txt","w");
flock($pliczek,2);
fwrite($pliczek,$ocena1);
flock($pliczek,3);
fclose($pliczek); } echo $licznik;
?>[/php:1:1c149813d5]
To jest skrypt do oceny. Nie wiem, co jest źle, ale oceniać nie chce. CO JEST ŹLE?
-- poprawiono czytelność --

[Majdan]

Prosiłbym o dodanie na początku '<?php' i '?>' na końcu, bo składnia nie jest pokolorowana, a trudno jest się doczytać.

[borec]

rotfl! moglbys to jakos uporzadkowac bo tego sie czytac nie da!

[DeyV]

czasami pare dodatkowych enterów i wcięć czyni cuda....
Polecam.

[MarcinJarek]

lub mieć dobrego edytora (http://forum.php.pl/viewtopic.php?t=10324)

[MP1]

Ja w edytorze robię mało enterów ze względu na wielkość pliku. Poprawiłem czytelność. Czekam na odpowiedź na pytanie.

[tara]

prawdopodbnie brakuje Ci jednego } na końcu. sprawdziłem, jeden jest otwarty.

[Majdan]

Ja też sprawdziłem i wszystkie są pozamykane.
A oceniać mi się nie chce , bo kod powinien być dobry.

[MP1]

Ja nie rozumiem - czyli kod jest dobry?
Kiedyś miałem problem z licznikiem i musiałem przerobić kod.
Teraz robię ocenianie - zrobiłem podobnie i powinno być dobrze. Co jest w takim razie źle?

[Majdan]

Już nie rozumiem. Chcesz, żeby ocenić; czy sprawdzić w czym jest błąd, bo nie działa :?:

[tara]

chodzi o to, że ten skrypt wpisuje ocene jaką dał użytkownik ale nie działa chyba

[php:1:8ac95a9b80]<?php
function zapiszlcz($ocena1) { //ten koleś jest otwarty
global $f; global $plikl;
$pliczek=fopen($f."/".$plikl[$_GET['ii']]."o.txt","w");
flock($pliczek,2);
fwrite($pliczek,$ocena1);
flock($pliczek,3);
fclose($pliczek); } echo $licznik;
//a tu nie ma zamknięcia
?>
?>[/php:1:8ac95a9b80]

[borec]

Ja w edytorze robię mało enterów ze względu na wielkość pliku.

to jaki ty masz dysk ze tak oszczedzasz? 2mb? bo dla serwera pare znakow nowej lini i wciec nie robi zadnej roznicy

[Chewolf]

Dokładnie obejrz swój kod np :
[php:1:4a87879d4d]<?php
$$licznik=round($licznik + $_POST['ocena'] / 2 , 1);
?>[/php:1:4a87879d4d]
$ o jeden dolar za dużo :-)
Linia 16
Ogólnie to jest strasznie nagmatwane w tym kodzie ja bym go napisał od nowa :wink:

[MP1]

Chodzi o to, że skrypt nie działa. Co jest źle?
Piszcie, co trzeba poprawić, jeśli widzicie błąd.

[rzseattle]

Przeniesione z php poczatkujacy.

[MP1]

Problem jest już rozwiązany. Wystarczył prosty skrypt.