Prostszy sposób na połączenie formularza z php

Question

Proszę o pomoc. Szukałam.

Chciałabym poznać drugi sposób połączenia formularza z php

<form action="tydzien.php" method="post">
       <label for="dzień">Dzień:</label>
     <select name="dzien">
        <?php 
    	      for ($i=1;$i<=31;++$i)
 
            echo '<option value="'.$i.'">'.$i.'</option>';
        ?>
    </select> </br></br>
<label for="miesiac">Miesiąc:</label>
    <select name="miesiac">
        <?php 
    			 for($l=1;$l<=12;++$l)
    	     echo '<option value="'.$l.'">'.$l.'</option>';
        ?>
    </select></br></br>
<label for="rok">Rok:</label>
    <select name="rok">
        <?php 
    			 for($t=1900;$t<=2020;++$t)
    	     echo '<option value="'.$t.'">'.$t.'</option>';
        ?>
    </select> </br>
	     </br></br><input type="submit" name="sprawdz"  value="Sprawdź dzień tygodnia" />
	     </br></br><input type="reset" name="reset"  value="Wyczyść" />
    </form>

Nie chcę tworzyć z if isset bo taki mam teraz . Chce mieć kod gdzie jest mało pisania. Dlatego utworzyłam poniższy kod, ale nie wiem gdzie jest błąd .

<?php
  $dzien = date('N', strtotime($_POST['dzien']. $_POST['miesiac']. $_POST['rok']));
    $dni_tygodnia = array('Poniedziałek', 'Wtorek', 'Środa', 'Czwartek', 'Piątek', 'Sobota','Niedziela');
    echo "Urodziłaś się w ";
    echo $dni_tygodnia[$dzien];

?>

 

 

 

Comments

A co się dzieje? Coś źle się pokazuje? Nie działa wcale? Wyskakuje jakiś komunikat błędu? Bo nikt się nie będzie raczej domyślał.

---

Przepraszam

Wybieram datę 19.06.1983

Klikam Sprawdż

Pokazuje Piątek a powinien pokazać NIEDZIELA

---

Funkcja date zwraca 7 dla podanych wartości. A co jest w tablicy $dni_tygodnia? Może tam tkwi błąd.

Answer 1

Po pierwsze myślę, że format daty jest niezrozumiały dla strtotime, bo zapis:

strtotime($_POST['dzien']. $_POST['miesiac']. $_POST['rok'])

dla podanej daty 19.06.1983 to będzie strtotime('1961983'). Nie ma tam zera przed miesiącem, bo go nie masz w select i nie ma też kropek, bo te służą do łączenia ciągów (konkatenacja), a nie są wstawiane w samą datę. Bez 0 przed miesiącem sobie pewnie poradzi, ale taki format jest raczej błędny, dodałbym kropki i spróbował.

Po drugie, jak właśnie sprawdziłem, date dla formatu N da numer dnia tygodnia z zakresu 1-7: https://www.php.net/manual/en/function.date.php a Twoja tablica ma 7 elementów, ale o indeksach od 0 do 6 (w programowaniu często liczymy od zera). W związku z tym będzie próbował dostać się do $dni_tygodnia[7], a to spowoduje błąd, bo takiego elementu nie ma. Najprościej chyba zapisać, aby tablica była numerowana od 1 i wtedy element o indeksie 7 faktycznie będzie niedzielą.

Tak przy okazji, w kodzie raczej niezbyt liczy się to, aby było mało pisania. Ważniejsze jest to, aby był generalnie dobry i czytelny, a nie najkrótszy. Tutaj brakuje Ci np. sprawdzenia czy $_POST['dzien'] i pozostałe wartości w ogóle zostały przesłane albo walidacji ich - co gdy wybiorę nieistniejącą datę lub w ogóle wyślę tam literki zamiast dni tygodnia.

Comments

Caps lock się zaciął? ;) Temat jest o to samo, jak sama zauważasz "podobny" i to nawet bardzo podobny, bo dotyczył bardzo podobnego kodu i przede wszystkim z tym samym celem - znaleźć błąd w skrypcie do sprawdzenia dnia urodzenia. Zwróciłem uwagę, aby nie powielać tematów, tylko tyle.

To super że się uczysz, ale wszyscy jesteśmy tu, aby się uczyć i rozwijać, więc to nic nadzwyczajnego i nie daje prawa zadania nie wiadomo ilu pytań na ten sam temat, bo wtedy jedna osoba utrudni życie wszystkim pozostałym.

No więc pokazałem 2 rzeczy, które wg mnie są źle zapisane i jak to poprawisz, to kod powinien zadziałać :) Jeśli nie to chętnie pomogę (na tyle ile wiem i mam czas), ale musi być też Twoja wola, aby zagłębić to samemu - poszukać, poświęcić czas, na spokojnie się nad tym wszystkim zastanowić i to zrozumieć. Nie wiem do czego dążysz konkretnie, jak mówiłem w programowaniu nie chodzi o to, aby usilnie zapisać kod najkrótszym sposobem czy zrobić wszystko, aby nie było warunku.

W temacie kursu PHP, to jeśli mogę zrobić małą autopromocję, napisałem własny kurs: https://www.phpdevs.pl/ Co prawda nie jest skończony i od dłuższego czasu niestety nie pojawiło się tam nic nowego oprócz aktualizacji kilku treści, ale generalnie całe podstawy są. Może coś Ci to pomoże, starałem się pisać tak, aby zrozumiał każdy, ale nie gwarantuję.

---

Poprawiłam to ale tym razem wyskakuje błąd: Parse error: syntax error, unexpected ';', expecting ')' in $dzien = date('N', strtotime($_POST['dzien'].  '.' . $_POST['miesiac'].  '.' . $_POST['rok']);  

Próbowałam 

<?php
  $dzien = date('N', strtotime($_POST['dzien'].  '.' . $_POST['miesiac'].  '.' . $_POST['rok']);  
  $dni_tygodnia = array(1 => 'Poniedziałek', 'Wtorek', 'Środa', 'Czwartek', 'Piątek', 'Sobota', 'Niedziela');
    echo "Urodziłaś się w ";
    echo $dni_tygodnia[$dzien-7];

?>

$dzien = date('N', strtotime($_POST['dzien'].  '.' . $_POST['miesiac'].  '.' . $_POST['rok']);  

I tak 

$dzien = date('N', strtotime($_POST['dzien']  '.' . $_POST['miesiac']  '.' . $_POST['rok']);  

i tak 

$dzien = date('N', strtotime($_POST['dzien']'.' $_POST['miesiac'] '.' $_POST['rok']);  

Nie mam już pomysłu jak to zrobić.

---

Zaraz się biorę za przerabianie tego kursu Dzięki

---

Właściwy jest pierwszy zapis, tylko że zabrakło Ci zamknięcia jednego nawiasu na końcu. Sprawdzaj uważnie. Otwierasz date(, otwierasz strtotime(, a zamykasz na końcu tylko jeden. Pozostałe nie mają prawa zadziałać, bo nie będzie wiedział co zrobić ze stringami zapisanymi obok siebie (ani nie mają kropek żeby je połączyć, ani przecinków jakoby to były kolejne argumenty, nic).

---

Wielkie dzięki teraz już rozumiem .