[PHP]Nie mogę sobie poradzić z zasięgiem zmiennej Opcje

[Zielonkawy18]

Witam

Na początku dokumentu deklaruję zmienną :

[PHP] pobierz, plaintext 
$sessionMonth=false
[PHP] pobierz, plaintext 

Poniżej mam przycisk ( Calcule ):

[PHP] pobierz, plaintext 
if(isset($_POST['calcule']))
{
if($_POST['month']=="styczeń") $month="01";
if($_POST['month']=="luty") $month="02";
if($_POST['month']=="marzec") $month="03";
if($_POST['month']=="kwiecień") $month="04";
if($_POST['month']=="maj") $month="05";
if($_POST['month']=="czerwiec") $month="06";
if($_POST['month']=="lipiec") $month="07";
if($_POST['month']=="sierpień") $month="08";
if($_POST['month']=="wrzesień") $month="09";
if($_POST['month']=='październik') $month="10";
if($_POST['month']=="listopad") $month="11";
if($_POST['month']=="grudzień") $month="12";
if($_POST['year']=="2012") $year="2012";
if($_SERVER['REQUEST_METHOD'] == 'POST')
{
global $session;
$session=true;
$_SESSION['month'] = $_POST['month'];
echo $_SESSION['month'];
}
}
[PHP] pobierz, plaintext 

Jeżeli zostanie on wciśnięty globalna zmienna $session, zostanie zmieniona na wartość true. W przeciwnym wypadku raczej tak się nie stanie.

Oto drugi przycisk ( donating )

[PHP] pobierz, plaintext 
if(isset($_POST['donating'])):
global $session;
if($session==false)
{
echo "false sesion";
if($_POST['month']=="styczeń") $month="01";
if($_POST['month']=="luty") $month="02";
if($_POST['month']=="marzec") $month="03";
if($_POST['month']=="kwiecień") $month="04";
if($_POST['month']=="maj") $month="05";
if($_POST['month']=="czerwiec") $month="06";
if($_POST['month']=="lipiec") $month="07";
if($_POST['month']=="sierpień") $month="08";
if($_POST['month']=="wrzesień") $month="09";
if($_POST['month']=='październik') $month="10";
if($_POST['month']=="listopad") $month="11";
if($_POST['month']=="grudzień") $month="12";
if($_POST['year']=="2012") $year="2012";
}
else
{
echo $_SESSION['month'];
if($month=="styczeń") $month="01";
if($month=="luty") $month="02";
if($month=="marzec") $mouth="03";
if($month=="kwiecień") $month="04";
if($month=="maj") $month="05";
if($month=="czerwiec") $month="06";
if($month=="lipiec") $month="07";
if($month=="sierpień") $month="08";
if($month=="wrzesień") $month="09";
if($month=='październik') $month="10";
if($mouth=="listopad") $month="11";
if($month=="grudzień") $month="12";
if($year=="2012") $year="2012";
}
}
[PHP] pobierz, plaintext 

Tutaj już wiem, że zmienna $session zawsze ( czy wcisne Calcule czy nie ) jest równa wartości false i nie mogę się z tym uporać. Wybaczcie za literówkę (mouth) winno być (month)

[abort]

1. Ja się ciągle nie umiem nadziwić, dlaczego ludzie sobie tak komplikują życie, i nie dobierają odpowiednich narzędzi do odpowiedniego celu. Jak rozumiem, miesiąc jest WPISYWANY przez usera, tak? No to nie lepiej zrobić

[HTML] pobierz, plaintext 
<select name="month">
<option value="1">styczeń</option>
<option value="2">luty</option>
  i tak dalej
<option value="12">grudzień</option>
</select>
[HTML] pobierz, plaintext 

i odczytywać po prostu $_POST['month'] dostając wartość numeryczną miesiąca?

2. Dziwi mnie fragment kodu:

[PHP] pobierz, plaintext 
if(isset($_POST['calcule']))
{
// kilka ifów
if($_SERVER['REQUEST_METHOD'] == 'POST')
[PHP] pobierz, plaintext 

Drugi jest niepotrzebny - jeśli tablica $_POST istnieje, to request method jest właśnie "POST".
Jeśli już, to warunki powinny być zamienione miejscami.

3. Jak już piszesz w jednym pliku obsługę wielu rzeczy, czyli wyświetlania (GET) i przewarzania (POST), to sugeruję zrobić to na zasadzie:

[PHP] pobierz, plaintext 
<?php
// Wspólny blok - jakieś include/require,
// ew. połączenie się do bazy przed odczytem/zapisem
// ---> a w Twoim przypadku operacje na sesjach - sprawdzenie czy istnieje i jest OK
// a jak sesja nie istnieje (lub jest "too old"), to np. redirect do strony logowania się userów.
if ($_SERVER['REQUEST_METHOD'] == 'GET') {
  // tu wyświetlamy formatkę (odczyt z bazy/sesji)
}
 
else  { // równoważne if ($_SERVER['REQUEST_METHOD'] == 'POST') {
  // a tu przetwarzamy (zapis do bazy/sesji)
}
 
// i znów, dla porządku część wspólna - np. zamykanie bazy
// i ewentualne czyszczenie skryptu po sobie (np. pliki tymczasowe)
?>
[PHP] pobierz, plaintext 

4. A, user b4rt3kk zwrócił uwagę na jedną rzecz: powinieneś użyć sesji. Bo to co do tej pory robisz, to wygląda tak, że:
a) idzie żądanie (GET) do wyświetlenia formatki - skrypt wyświetla formatkę i kończy działanie.
(IMG:style_emoticons/default/cool.gif) user wprowadza dane na formularzu
c) idzie żądanie (POST) do przetworzenia formatki (z danymi via $_POST).

Zakładasz, że zmienna będzie istniała taka, jak w poprzednim skrypcie. A to niestety błąd - tamtego skryptu (ze wszelkimi zmiennymi) już dawno nie ma. Zobacz sobie opis z punktu wyżej. Przeanalizuj go. Ratunkiem dla Ciebie będą sesje.

Ten post edytował abort 9.11.2012, 20:42:06