[HTML][PHP]Robię sondaż, czyli zliczanie głosów na tak i nie Opcje

[ats2008]

Witajcie,

Z góry mówię, że jestem totalnie początkujący w php, ale się nie poddaję .

Założenie jest takie, żeby na pytanie były 2 możliwości odpowiedzi. Skrypt ma za zadanie zliczać głosy i po oddaniu głosu wyświetlić wynik głosy na tak i nie.

Po kilku dniach prób!!!, ocieraniu potu z czoła, korzystania z dziesiątków forów i postów, porad profesjonalistów i znachorów php - wymodziłem coś takiego - działa, ale jest totalnie siermiężne:

formularz (przyciski tak i nie):

[HTML] pobierz, plaintext 
<div><form id="formtak" name="formyes">
	<input type="hidden" name="buttonyes"  />
	<input type="submit" value="Tak" /> 
</form></div>
 
<div><form id="formnie" name="formnie">
	<input type="hidden" name="buttonno"  />
	<input type="submit" value="Nie" /> 
</form></div>
[HTML] pobierz, plaintext 

skrypt dodający do pliku tak.txt lub nie.txt liczbę 1 po naciśnięciu odpowiedniego przycisku:

[PHP] pobierz, plaintext 
<?php
//Wczytanie z pliku liczby oraz dodanie do niej 1.
$tak=file("tak.txt");
$liczbatak=$tak[0];
$takplus=$liczbatak + 1;
 
//Wczytanie z pliku liczby nie oraz dodanie do niej 1/
$nie=file("nie.txt");
$liczbanie=$nie[0];
$nieplus=$liczbanie + 1; 
 
//dodanie do tak.txt  +1. 
if (isset($_REQUEST['buttonyes'])) 
{$fp=fopen("tak.txt", "w+");
	//blokujemy plik tak.txt do zapisu: 
flock($fp, 2); 
	//zapisujemy dane do pliku: 
fwrite($fp, $takplus);
	//odblokowujemy plik:
flock($fp, 3); 
	//zamykamy plik:
fclose($fp);
}
//dodanie do nie.txt  +1. 
else if (isset($_REQUEST['buttonno']))
{
	$fnie=fopen("nie.txt", "w+");
	//opis jak powyżej
	flock($fnie, 2); 
	fwrite($fnie, $nieplus);
	flock($fnie, 3); 
	fclose($fnie);
}
 
//wyświetla wynik po naciśnięciu submit
if  (isset($_REQUEST['buttonyes']) ||  isset($_REQUEST['buttonno']))
 
{print ' <p>Na Tak głosowało: '.fgets(fopen("tak.txt","r")).' osób.</p>
<p>Na Nie głosowało: '.fgets(fopen("nie.txt", "r")).' osób.</p>';}
 
?>
[PHP] pobierz, plaintext 

Jest może jakiś zgrabniejszy sposób na napisanie tego skryptu?. Szczególnie próbowałem inkrustować zmienną $liczbatak++ (lub $liczbanie++), ale zapisywało w pliku txt i wyświetlało coś takiego: id #3. O co chodzi?

I sorry za zawracanie gitary:).

Z góry dzięki,

Pozdrawiam.

[DiH]

Poległeś na etapie HTML. Jeżeli nie masz możliwości użycia MySQL, to możesz zrobić co następuje. Po pierwsze stwórz jeden <form>, w nim dwa radio buttony do wyboru, których value to odpowiednio 0 i 1.

Twój plik powinien mieć formę np:
A|B
Gdzie:
A - to liczba wszystkich głosów (zwiększana po każdym wywołaniu skryptu)
B - liczba głosów na nie (A-B = ilość głosów na tak)

Jeżeli w pliku będzie: 10|3, to używasz na jego zawartości funkcji explode i operujesz na uzyskanych w ten sposób danych. Jeżeli masz możliwość postawienia skryptu na php5, to używaj file_put_contents i file_get_contents zamiast serenady fopen, fwrite etc.

Pozdrawiam

Ten post edytował DiH 22.02.2010, 20:19:40

[ats2008]

Dzięki stary,

teraz przechodzę w fazę testowań - napiszę wyniki. Pierwsze co mi jednak przychodzi do głowy to pytanie, a co się stanie jeśli będzie do wyboru więcej opcji? np. pięć? Nie uśmiecha mi się odejmować od A-(b+c+d+e+...+n), ale najpierw przetestuję

[bemol]

podałeś możliwości TAK i NIE, więc nie rozumiem, dlaczego ma działać dla 5.
Wtedy możesz sobie po prostu zrobić A|B|C|D|E
i zapisywać wszystkie opcje.
Suma wszystkich to będzie liczba 'głosowań'.

[ats2008]

Witajcie, DiH zarobiłeś u mnie browca za pokazanie mi funkcji explode i resztę.

Co do większej ilości opcji to po przetestowaniu metody nie mam pytań i wątpliwości:). Ogólnie podoba mi się bardzo i polecam innym.

W necie jednak podawali często i gęsto tą starszą metodę z fwrite etc.

Oto moje rozwiązanie (działa) :

[HTML] pobierz, plaintext 
<form   method="">
	<input type="radio" name="tak" value="0"   />Tak, 
	<input type="radio" name="nie"  value="1" />Nie.<br />
	<input type="submit" name="submit" value="Głosuj" />
</form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$file='suma.txt';
//sprawdzamy, czy istnieje plik suma.txt
$jestfile=file_exists($file);
//jeśli plik nie został utworzony, tworzymy nowy plik i zapisujemy zawartość 0/0:
if (!$jestfile){fopen($file, 'r+');
$begin="0/0";
file_put_contents($file, $begin);}
 
//pobieranie zawartości suma.txt w postaci np. 100/50.
$suma = file_get_contents($file);
//rozbijamy treść na części (elementy tablicy $part[0],...
$part=explode("/", $suma); 
 
//sprawdzamy, jaki był wybór:
if (isset($_REQUEST["tak"])) 
{
	$part[0]++;
	$part[1]++;//dodajemy 1 do punktów na tak; 
	$nie=($part[0]-$part[1]);
	echo "Ogółem było: <b>".$part[0]."</b> z czego na TAK: <b>".$part[1]."</b>, a na NIE: <b>".$nie."</b> głosów.";
 
}
else if (isset($_REQUEST["nie"])) 
{
	$part[0]++;
	$nie=($part[0]-$part[1]); 
	echo "Ogółem było: <b>".$part[0]."</b> z czego na TAK: <b>".$part[1]."</b>, a na NIE: <b>".$nie."</b> głosów.";
}
 
else if (isset($_REQUEST["submit"])  && !isset($_REQUEST["nie"]) && !isset($_REQUEST["nie"])) echo "Nie dokonałeś wyboru, Koleś - spróbuj jeszcze raz!!!";
//zapisujemy teraz wynik głosowania do pliku suma.txt.
$newsuma=$part[0]."/".$part[1];
file_put_contents($file, $newsuma);
//well done, dzięki  DiH :)
?>
[PHP] pobierz, plaintext 

Poza tym jest jeszcze sprawa utworzenia pliku suma.txt, tzn. założyłem sytuację, że tego pliku nie ma... i chciałbym, żeby skrypt go utworzył .
Użyłem do tego funkcji fopen, a dokładniej: fopen($file, 'r+'), a potem dopisałem do niego wartość: 0/0.

Problem jest w tym, że kiedy usunę plik suma.txt do sprawdzenia, wywala mi coś taki komunikat: Warning: fopen(suma.txt) [function.fopen]: failed to open stream: No such file or directory in E:\wamp\www\temporary\explode\index.php on line 21 (21- to fopen).
ale jednocześnie zapisuje 0/0 do pliku, ale beznadziejnie to wygląda...

Znacie może jakieś inne sposoby utworzenia pliku txt bo w googlach faszerują fopen...

Z góry dzięki.



Ten post edytował ats2008 23.02.2010, 22:50:35

[mortus]

Przede wszystkim jeżeli jakiś plik otwierasz za pomocą fopen(...), to powinieneś go zamknąć za pomocą fclose(...). Może to jest przyczyna problemu, ale głowy za to nie dam?
EDIT:
Sprawdziłem to jeszcze raz i okazuje się, że funkcja file_get_contents($filename) zwraca false, jeżeli nie znajdzie pliku o nazwie $filename. Z kolei funkcja file_put_contents($filename, $content) sama otwiera plik, zapisuje zawartość i zamyka plik, w związku z czym nie ma potrzeby otwierania pliku za pomocą fopen(...). Poprawiony kod:

[PHP] pobierz, plaintext 
<?php
$filename = 'suma.txt';
$content = file_get_contents($filename);
if(!$content) {
	$part[0] = 0;
	$part[1] = 0;
} else {
	$part = explode('/', $content);
}
if(isset($_REQUEST["tak"])) {
	$part[0]++;
	$part[1]++;
	$nie = ($part[0] - $part[1]);
	echo 'Ogółem było: <b>' . $part[0] . '</b> z czego na TAK: <b>' . $part[1] . '</b>, a na NIE: <b>' . $nie . '</b> głosów.';
} else if(isset($_REQUEST["nie"])) {
	$part[0]++;
	$nie = ($part[0] - $part[1]);
	echo 'Ogółem było: <b>' . $part[0] . '</b> z czego na TAK: <b>' . $part[1] . '</b>, a na NIE: <b>' . $nie . '</b> głosów.';
} else if (isset($_REQUEST["submit"]) && !isset($_REQUEST["tak"]) && !isset($_REQUEST["nie"])) {
	echo "Nie dokonałeś wyboru, Koleś - spróbuj jeszcze raz!!!";
}
$newcontent = $part[0] . '/' . $part[1];
file_put_contents($filename, $newcontent);
?>
[PHP] pobierz, plaintext 

Ten post edytował mortus 24.02.2010, 09:16:05

[ats2008]

Dzięki mortus, chyba na browary nie wyrobię .

Najprostsze rozwiązania są najlepsze, Twoje rozwiązanie ze sprawdzeniem pliku txt bardzo mi się podoba. Po co kombinować z tworzeniem nowego, jak można to obejść .

Teraz kod wygląda cacy.

Ogólnie, nie mam więcej pytań, temat do zamknięcia.

Pozdrawiam!