[PHP]Lista rozwijana zapamiętanie wyboru Opcje

[pietroff]

Witajcie,

raczkuję w php i próbuję napisać kalendarz z możliwością wyboru miesięcy. Wszystko działa nawet nieźle tylko chciałbym aby lista rozwijana zapamiętywała wybór i aby ustawiała się automatycznie na miesiąc który jest obecnie. Poniżej kod samej listy:

[PHP] pobierz, plaintext 
$MonthNames = array(1=>'Styczeń','Luty','Marzec','Kwiecień','Maj','Czerwiec','Lipiec','Sierpien','Wrzesień','Październik','Listopad','Grudzień');
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<form action="" method="post" class="center">
    <h3> Wybierz interesujący Cię miesiąc:&nbsp;&nbsp;&nbsp; 
        <select name="wybMiesiecy" class="lista-rozwijana" >
		<?php
                 for ($m=1; $m<=(12); $m++){
                          echo '<option>'.$MonthNames[$m].'</option>';                       
                ?>
	</select> 
        <input type="submit" name="wyslij" class="lista-rozwijana-button" value="Wyświetl wskazany miesiąc" />
    </h3>
</form>
[HTML] pobierz, plaintext 

[gino]

to raczej podstawy HTML-a niż php, prawidłowo powinno być:

Kod

<option value="watrosc" selected>nazwa_meisiąca</option>

gino

edit: jak chcesz bez parametru value określić które option ma być selected (wybrane)?

Ten post edytował gino 14.04.2020, 13:30:29

[kreatiff]

Dodaj w pętli warunek na sprawdzanie liczby porządkowej miesiąca i jeśli się zgadza z bieżącym $m, to go zaznacz:

[PHP] pobierz, plaintext 
for ($m = 1; $m <= 12; $m++) {
	$selected = $m == date('n') ? ' selected' : '';
	echo '<option value="' . $m . '"' . $selected . '>' . $MonthNames[$m] . '</option>';
}
[PHP] pobierz, plaintext 

[gino]

@kreatiff ja rozumiem, że osoba początkująca ale jeśli operuje tablicami, pętlami, to powinna chociaż w podstawowym stopniu opanować podstawy języka, czy instrukcje warunkowe i tag html-a, którym się posługuje, chyba lepiej pobudzić kreatiffność () naprowadzając na problem, niż dać gotowca zwłaszcza w rzeczach tak prostych.

gino

[pietroff]

Po zmodyfikowaniu kodu niestety ustawia mi się aktualny miesiąc czyli kwiecień ale nie można już zmienić na inny ciągle wraca do kwietnia

[HTML] pobierz, plaintext 
<form action="" method="post" class="center">
    <h3> Wybierz interesujący Cię miesiąc:&nbsp;&nbsp;&nbsp; 
        <select name="wybMiesiecy" class="lista-rozwijana" >
		<?php
                 for ($m=1; $m<=(12); $m++){
                          $selected = $m == date('n') ? ' selected' : '';
                          echo '<option value="' . $m . '"' . $selected . '>' . $MonthNames[$m] . '</option>';   
 
        }                 
                ?>
	</select> 
        <input type="submit" name="wyslij" class="lista-rozwijana-button" value="Wyświetl wskazany miesiąc" />
    </h3>
</form>
[HTML] pobierz, plaintext 

[gino]

robi dokładnie to, co chcesz, przed wysłaniem formularza ustawia aktualny miesiąc. Wybierz innych miesiąc, zrób obsługę w php po wysłaniu formularza i zobaczysz, że wybiera ten miesiąc który chcesz.
Przed tablicą miesięcy wklej ten kod i zobaczysz, że i kod i formularz działają ok. Poczytaj o $_POST:

Kod

if(isset($_POST['wybMiesiecy']))
echo $_POST['wybMiesiecy'];

gino

Ten post edytował gino 14.04.2020, 16:52:06

[pietroff]

W takiej formie działa bez zarzutu. Czy można tu coś poprawić zmienić? Czy jest dobrze?

[PHP] pobierz, plaintext 
<form action="" method="post" class="center">
    <h3> Wybierz interesujący Cię miesiąc:&nbsp;&nbsp;&nbsp; 
        <select name="wybMiesiecy" class="lista-rozwijana" >
		<?php
                 for ($m=1; $m<=(12); $m++){
                            if(isset($_POST['wybMiesiecy'])){
                                $selected = $m == $_POST['wybMiesiecy'] ? 'selected' : '';
                            }else{
                            $selected = $m == date('n') ? ' selected' : '';}
	echo '<option value="' . $m . '"' . $selected . '>' . $MonthNames[$m] . '</option>'; 
 
        }
 
                ?>
	</select> 
        <input type="submit" name="wyslij" class="lista-rozwijana-button" value="Wyświetl wskazany miesiąc" />
    </h3>
</form>
[PHP] pobierz, plaintext 

[nospor]

Mozna to poprawic. Teraz masz duplikacje w ustawianiu selected i o wiele za duzo logiki w tej banalnej petli FOR

[PHP] pobierz, plaintext 
$toSelect = isset($_POST['wybMiesiecy']) ? $_POST['wybMiesiecy'] : date('n');
                 for ($m=1; $m<=12; $m++){
                                $selected = $m == $toSelect ? 'selected' : '';
	echo '<option value="' . $m . '"' . $selected . '>' . $MonthNames[$m] . '</option>'; 
 
        }
[PHP] pobierz, plaintext 

Prawda ze czytelniej?

[pietroff]

Bardzo dziękuję wszystkim za pomoc