Błąd z plikami cookie w prostej ankiecie

0

Witam, problem polega na tym że z tak napisanymi ciasteczkami ankieta się nie wysyła, jednak kiedy wszystkie części kodu dotyczące ciastek okomentuje, ankieta działa poprawnie. Pliki mają za zadanie dopuścić danego użytkownika do ankiety tylko raz. Proszę o pomoc, co może być nie tak... Z góry dziękuje :)

<?php
setcookie("oddano_glos", "0"); 

$ANKIETA = <<<PL
    <form method="POST" action="">

        <filedset>

            <legend><font size="5"><b>Ankieta oceny programów z baz¹ aktów prawnych Lex i LexisNexis</b></font></legend>
            <br /><br />
            <label>1. Jak oceniaj¹ Pañstwo aplikacje Lex ?</label><br />
            <input type="radio" name="1" value="pozytywnie">Pozytywnie<br />
            <input type="radio" name="1" value="negatywnie">Negatywnie<br />
            <br /><br />
            <label>2. Jak oceniaj¹ pañstwo aplikacje LexisNexis ?</label><br />
            <input type="radio" name="2" value="pozytywnie">Pozytywnie<br />
            <input type="radio" name="2" value="negatywnie">Negatywnie<br />
            <br /><br />
            <label>3. Która aplikacja oferuje szybsze i ³atwiejsze wyszukiwanie ?</label><br />
            <input type="radio" name="3" value="Lex">Lex<br />
            <input type="radio" name="3" value="LexisNexis">LexisNexis<br />
            <br /><br />
            <label>4.  Która aplikacja jest bardziej czytelna, intuicyjna (wizualnie)</label><br />
            <input type="radio" name="4" value="Lex">Lex<br />
            <input type="radio" name="4" value="LexisNexis">LexisNexis<br /> 
            <br /><br />
            <label>5. Uwagi dotycz¹ce uzytkowania programu Lex i LexisNexis</label><br />
            <textarea rows=5 cols=50 name="5"></textarea>
            <br /><br />

            <input type="submit" value="Wyœlij" />

        </fieldset>

        <input type="hidden" name="checker" value="1" />
    </form>

PL;

$head = "Ankieta"; 

$body = "Pyt1: " . $_POST['1'] . "\n"; 
$body .= "Pyt2: " . $_POST['2']. "\n";
$body .= "Pyt3: " . $_POST['3']. "\n"; 
$body .= "Pyt4: " . $_POST['4']. "\n"; 
$body .= "Uwagi: " . $_POST['5'];
$to = "[email protected]"; 

if(isset($_COOKIE['oddano_glos'])){
    echo "Dziêkujemy za opinnie.";
}else{
    if(isset($_POST['checker'])){
        if((isset($_POST['1'])) && (isset($_POST['2'])) && (isset($_POST['3'])) && (isset($_POST['4']))){
            echo "Wszystko ok !";
            setcookie("oddano_glos", "1");
            $res = mail($to, $head, $body );
            if($res){
                echo " Ankieta wys³ana poprawnie"; 
            } else {
                echo "Wystapil problem z wyslaniem ankiety, zapraszamy póŸniej"; 
            }    
        }else{
            echo $ANKIETA;
            echo "<font color=\"red\"><b>Prosimy przed wys³aniem obowi¹zkowo wype³niæ punkty od 1 do 4</b></font>"; 
        }
    } else {
        echo $ANKIETA;
    }
}

?>
0

A co te znaki takie dziwne? Piszesz w notatniku?

Po co Ci w ogóle wartość ciasteczka, skoro po samym isset rozpoznajesz czy jest ankieta uzupełniona?
To Ci w ogóle wyświetla ankietę tylko dlatego, że po setcookie ciasteczko jest dostępne dopiero w NASTĘPNYM żądaniu strony internetowej (nie jest widoczne od razu w $_COOKIE), ponieważ nie wiemy jeszcze czy przeglądarka zaakceptowała ciasteczko.

Doczytaj o ciastkach i protokole HTTP ;)

0

W takim razie jak mam się odnieść konkretnie do ciasteczka ? Albo inaczej, jak to napisać żeby było ok ?

0

pseudokod

if (jest_ciasteczko) { // $_COOKIE
  // sorry, ankiete juz uzupelniles
}
else {
  if (dane_formularza) { // jakies pole POST
    // dzieki za uzupelnienie ciastka
    setcookie();
  }
  else {
    // wyswietl ankiete
  }
}

dodatkowo:

  • nie opóźniaj ładowania strony (oraz kodu, oraz nie irytuj użytkownika potrzebą wielokrotnego uzupełniania ankiety) funkcją mail(), tylko zapisuj wyniki do bazy/pliku.
  • jak potrzebujesz wysłać sobie to mailem, to lepiej napisz sobie zadanie cron, które podeśle Ci niewysłane dane ankiet.
  • unikaj funkcji mail - jest niewiarygodna, użyj klasy do wysyłki maili, np. phpMailer
0

ja do tego będę dopiero dopisywał kod który zamiast wysyłać dane na maile, zapisze je w bazie.
Mam takie pytanie, czym się różni struktura logiczna twojego kodu od mojego ? Może po prostu, ja nie widzę jakieś prostej rzeczy, ale nie potrafię dostrzec co tu jest nie tak (albo do końca nie rozumiem jak działają cookie).

Na chwilę obecną chcę utworzyć ankietę, która poprawnie wyślę wynik i będzie ją można wypełnić tylko raz.

Na samym początku tworze ciastko (ponieważ jeśli tego nie zrobie, wywala mi błąd że on nie może sprawdzić wartości ciastka którego nie ma, mogę sprawdzić czy ciastko w ogóle istnieje, ale nie wiem jak to zrobić inaczej niż 'isset()')

Później mam pierwszego if'a sprawdzajacego czaistko, nastepnie mam checker a po nim warunki sprawdzajace wypelnienie pola, nastepnie wysylam maila i ustawiam wartosc ciastka na 1.

Twój ciag logiczny jest taki sam jak mój, więc nie rozumiem co jest nie tak z tymi ciastkami.

0

wywala mi się taki błąd

Warning: Cannot modify header information - headers already sent by (output started at /home/storin/domains/malespointview.pl/public_html/ankietap.php:54) in /home/storin/domains/malespointview.pl/public_html/ankietap.php on line 61

1

A o tym błędzie to dopiero teraz sobie przypomniałeś? :/
Nie możesz wysłać ani skrawka kodu HTML, ani spacji, ani pustej linii przed wywołaniem setcookie (i paru innych funkcji). Możesz to ominąć poprzez zastosowanie ob_start().

Ciąg logiczny nie jest taki sam.
Twój kod:

  1. tworzy ciasteczko z WARTOŚCIĄ zero
  2. od razu sprawdza czy jest ustawione ciasteczko, niezależnie od wartości // to ZAWSZE będzie prawda przy drugiej wizycie na stronie, NIGDY nie będzie to prawda na pierwszej wizycie.
    2.1 jest ciastko = informacja
    2.2 nie ma ciastka - dalszy ciąg

2.2.1 sprawdza dane POST:

2.2.1.1 są dane - ustawia ciastko z wartością jeden
2.2.1.1.1 próbuje wysłać maila, ale niezależnie od tego - ciastko jest już uzupełnione (zresztą jak wspominałem - wystarczy wejść na stronę i odświeżyć i system już potraktuje to jako ankieta wypełniona)

2.2.1.2 nie ma danych - wyświetla ankietę.

Mój kod:

  1. sprawdza czy ciastko jest
    1.1 jak jest - informacja
    1.2 jak nie ma - sprawdzamy czy są dane POST
    1.2.1 są dane - zapisuje dane, tworzy ciastko, dziękuje za wypełnienie
    1.2.2 nie ma danych - pokazuje ankietę

Ja bym odpuścił robienie ankiet, jak nie rozumiesz różnicy między sprawdzeniem wartości zmiennej, a sprawdzeniem czy zmienna istnieje. Ludzie mają problem z załapaniem tego, że ciastko faktycznie istnieje dopiero przy drugim requeście po jego ustawieniu, Ty problemy masz już na podstawach języka, a bez tego - ciężko robić coś dalej.

0

Po prostu się nie zrozumieliśmy, nie wiedziałem o istnieniu tej funkcji o której powiedziałeś dla tego nie potrafiłem zrozumiem zastosowania ciastek, teraz już jest wszystko jasne :)

dla potwierdzenia wysyłam działający kod :)

<?php
ob_start();

$ANKIETA = <<<PL
    <form method="POST" action="">

        <filedset>

            <legend><font size="5"><b>Ankieta oceny programów z bazą aktów prawnych Lex i LexisNexis</b></font></legend>
            <br /><br />
            <label>1. Jak oceniają Państwo aplikacje Lex ?</label><br />
            <input type="radio" name="1" value="pozytywnie">Pozytywnie<br />
            <input type="radio" name="1" value="negatywnie">Negatywnie<br />
            <br /><br />
            <label>2. Jak oceniają państwo aplikacje LexisNexis ?</label><br />
            <input type="radio" name="2" value="pozytywnie">Pozytywnie<br />
            <input type="radio" name="2" value="negatywnie">Negatywnie<br />
            <br /><br />
            <label>3. Która aplikacja oferuje szybsze i łatwiejsze wyszukiwanie ?</label><br />
            <input type="radio" name="3" value="Lex">Lex<br />
            <input type="radio" name="3" value="LexisNexis">LexisNexis<br />
            <br /><br />
            <label>4.  Która aplikacja jest bardziej czytelna, intuicyjna (wizualnie)</label><br />
            <input type="radio" name="4" value="Lex">Lex<br />
            <input type="radio" name="4" value="LexisNexis">LexisNexis<br /> 
            <br /><br />
            <label>5. Uwagi dotyczące uzytkowania programu Lex i LexisNexis</label><br />
            <textarea rows=5 cols=50 name="5"></textarea>
            <br /><br />

            <input type="submit" value="Wyślij" />

        </fieldset>

        <input type="hidden" name="checker" value="1" />
    </form>

PL;

$head = "Ankieta"; 

$body = "Pyt1: " . $_POST['1'] . "\n"; 
$body .= "Pyt2: " . $_POST['2']. "\n";
$body .= "Pyt3: " . $_POST['3']. "\n"; 
$body .= "Pyt4: " . $_POST['4']. "\n"; 
$body .= "Uwagi: " . $_POST['5'];
$to = "[email protected]"; 

if((isset($_COOKIE['oddano_glos']))){
    echo "Dziękujemy za opinnie. ";
}else{
    if(isset($_POST['checker'])){
        if((isset($_POST['1'])) && (isset($_POST['2'])) && (isset($_POST['3'])) && (isset($_POST['4']))){
            echo "Wszystko ok !";
            $res = mail($to, $head, $body );
            if($res){
                echo " Ankieta wysłana poprawnie"; 
            } else {
                echo "Wystapil problem z wyslaniem ankiety, zapraszamy później"; 
            }  
            setcookie("oddano_glos");   
        }else{
            echo $ANKIETA;
            echo "<font color=\"red\"><b>Prosimy przed wysłaniem obowiązkowo wypełnić punkty od 1 do 4</b></font>"; 
        }
    } else {
        echo $ANKIETA;
    }
}

?>

Dziękuje za pomoc ;)

0

@storin zajrzyj do dokumentacji setcookie i poczytaj o tym, co się stanie jak pominiesz trzeci parametr

0

rozumiem że chodzi o to

expire
The time the cookie expires. This is a Unix timestamp so is in number of seconds since the epoch. In other words, you'll most likely set this with the time() function plus the number of seconds before you want it to expire. Or you might use mktime(). time()+606024*30 will set the cookie to expire in 30 days. If set to 0, or omitted, the cookie will expire at the end of the session (when the browser closes).

dodałem w takim razie ważność pliku na miesiąc, tyle mniej-więcej ankieta będzie opublikowana

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0