[skrypt][php]Skrypt Systemu Oceniania Opcje

[b_chmura]

Witam
Chciałbym abyście ocenili i porządnie skrytykowali poniższy skrypt systemu oceniania:
link

Przykład działania:
link

EDIT
Dane w pliku txt zapisywane są nastepująco:
pierwszy głos|drugi|trzeci|czwarty|...

Ten post edytował b_chmura 25.06.2007, 14:48:49

[kwiateusz]

1 blad: dodalem głos potem f5 juz jest 10 głosów...

[skowron-line]

Kod

  if(!empty($_POST['ocena']))

wewnatrz funkcji nie powinno sie stosowac takich rzeczy

[b_chmura]

fuck myślałem ze sesja wszytko załatwi wrr
macie może jakieś pomysły żeby temu zapobiec?

[skowron-line]

mozesz po dodaniu glosu przeniesc hederem spowrotem na strone poczatkowa i wtedy nie bedzie ptoblemu z odswiezaniem.

[b_chmura]

wolałbym unikać header... normalnie przy dodawaniu komentarzy itp stosowałem token który załatwiał sprawę...

EDIT
skowron-line: literówka... miało być

Kod

  if(!empty($glos))

Ten post edytował b_chmura 25.06.2007, 15:12:29

[Dandelion]

a czemu wolalbys unikac headerow ?

[b_chmura]

już poradziłem sobie z tym problemem (przy zapisie sprawdzam czy ciacho lub sesja istnieje jeśli tak to nie zapisuje)

jeśli chodzi o header to po prostu nie lubię zbędnie wykorzystywać nagłówków


EDIT
poprawiona wersja dostępna pod tymi samymi linkami

Ten post edytował b_chmura 25.06.2007, 17:36:56

[luki100011]

Nie wiem czy to błąd czy nie
Ale nawet jak nie wybiorę nic z listy to i tak oddam głos.

[b_chmura]

Głos automatycznie oddawany jest podczas pierwszego uruchomienia skryptu. Jest to ocena dostateczna (umiarkowana) od której dopiero teraz internauci mogą oddawać swoje głosy (z początku oceniana rzecz jest po prostu umiarkowana (ni jaka) a następnie albo coraz niżej albo wyżej oceniana)

[Fixer]

przydało by się by Twój skrypt był też kompatybilny z PHP4 bo działa tylko na PHP5

w lini 63 i 82 funkcja file_put_contents() wymaga poprawy na starszy zapis
Najlepiej zrobić to wykrywając czy funkcja istnieje i jeśli nie to użyć starego sposobu zapisu

[PHP] pobierz, plaintext 
<?php
if (function_exists('file_put_contents') == false) {
	function file_put_contents($file, $string) {
	  $f=fopen($file, 'w');
 
	  fwrite($f, $string);
	  fclose($f);
	}
  }
?>
[PHP] pobierz, plaintext 

PS dobra robota

Ten post edytował Fixer 7.08.2007, 13:54:21