[PHP]Funkcja if kilka warunków Opcje

[nospor]

Ja nie widze zadnych zmian. Nadal wszedzie tam gdzie mialo by == to jest =

[brzanek]

ach racja

[PHP] pobierz, plaintext 
        $dataa = date('Y-m-d');
 
        $result = $mysqli->query("SELECT date FROM liczniki ORDER BY date DESC LIMIT 1");
        while ( $row = mysqli_fetch_array($result) ) {	
        $dzisiaj = $row['date'];
        }
 
        $result = $mysqli->query("SELECT * FROM liczniki WHERE date = '".$dataa."'");
        while ( $row = mysqli_fetch_array($result) ) {	
        $miastoid = $row['miasto'];
        }
 
        if (( $dzisiaj == date('Y-m-d') ) && ( $miastoid == $rodzaj_danych )){
        $result = $mysqli->query("UPDATE liczniki SET licznik=licznik+1 WHERE date = '".$dataa."' AND miasto = '".$rodzaj_danych."'");
        }
        else if  (( $dzisiaj == date('Y-m-d') ) && ( $rodzaj_danych !== $miastoid )) {
        $result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
        }
        else {
        $result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
        }
[PHP] pobierz, plaintext 

[nospor]

Skoro jak twierdzisz, nie wykonuje ci sie to
$result = $mysqli->query("UPDATE liczniki SET licznik=licznik+1 WHERE date = '".$dataa."' AND miasto = '".$rodzaj_danych."'");

to sprawdz co zawieraja zmienne z twojego warunku
if (( $dzisiaj == date('Y-m-d') ) && ( $miastoid == $rodzaj_danych )){

var_dump($dzisiaj);
var_dump($miastoid);
var_dump($rodzaj_danych);
No przeciez to jest logiczne ze tak trzeba zrobic...

swoja droga moglbys jakos normalnie nazywac zmiennej, odnosnie tego co zaweiraja a nie losowo. W obecnej nazewniczej sytuacji nie trudno o mase glupich innych bledow

[brzanek]

[PHP] pobierz, plaintext 
        if (( $dzisiaj == date('Y-m-d') ) && ( $miastoid == $rodzaj_danych )){
        $result = $mysqli->query("UPDATE liczniki SET licznik=licznik+1 WHERE date = '".$dataa."' AND miasto = '".$rodzaj_danych."'");
        }
        else if  (( $dzisiaj == date('Y-m-d') ) && ( $rodzaj_danych !== $miastoid )) {
        $result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
        }
        else {
        $result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
        }
[PHP] pobierz, plaintext 

W tym całym kodzie poprawnie działa tylko przypadek kiedy w BD nie ma dzisiejszej daty - wtedy jest dopisywany nowy rekord. Jeśli jest już dzisiejsza data i przypisane do niej miasto Szczecin to po wywołaniu strony ze Szczecinem tworzy się nowy rekord w BD a powinien tylko dodać 1 do licznika przy tym wcześniejszym rekordzie.

[nospor]

Prosilem zebys sprawdzic zmienne z tego IFa. Zrobiles to?

[brzanek]

Zawierają
$dzisiaj - 2023-10-21
$miastoid - szczecin
$rodzaj_danych - szczecin

[nospor]

Ja cie proszse zebys podal mi dokladnie to co zwraca var_dump. Dokladnie

[brzanek]

string(10) "2023-10-21" string(9) "szczecin" string(9) "szczecin"

[nospor]

No to widac wyraznie ze wartosci wszystkie sie zgadzaja i ten warunek
if (( $dzisiaj == date('Y-m-d') ) && ( $miastoid == $rodzaj_danych )){
sie musi wykonac.

Moze wiec twoje zapytanie sie nie wykonuje

Zrob:

if (( $dzisiaj == date('Y-m-d') ) && ( $miastoid == $rodzaj_danych )){

echo 'WIDZISZ MNIE?!!';exit;
$result = $mysqli->query("UPDATE liczniki SET licznik=licznik+1 WHERE date = '".$dataa."' AND miasto = '".$rodzaj_danych."'");
}

[trueblue]

[PHP] pobierz, plaintext 
if (( $dzisiaj = date('Y-m-d') ) && ( $miastoid = $rodzaj_danych )){
$result = $mysqli->query("UPDATE liczniki SET licznik=licznik+1 WHERE date = '".$dataa."' AND miasto = '".$rodzaj_danych."'");
}
else if  (( $dzisiaj = date('Y-m-d') ) && ( $rodzaj_danych != $miastoid )) {
$result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
}
else {
$result = $mysqli->query("INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)");
}
[PHP] pobierz, plaintext 

Po co else if i else skoro i tak mają wykonać się takie same zapytania?

Inna sprawa, że zamiast tworzyć ify i kilka zapytań, wystarczy nadać na pole date i miasto klucz unique i stworzyć jedno zapytanie bez żadnych ifów:

[SQL] pobierz, plaintext 
INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)
ON DUPLICATE KEY UPDATE licznik=licznik+1;
[SQL] pobierz, plaintext 

Gdybyś użył powyższego, to nadając wartość domyślną 1 dla pola licznik, wystarczy wstawiać dane tylko do date i miasto.

Ten post edytował trueblue 21.10.2023, 09:42:22

[brzanek]

O co chodzi?
Mam listę miast po wybraniu odpowiedniego miasta jesli nikt wcześniej dzisiaj nie wybrał tego miasta to pojawia się nowy rekord w BD
id: 1
data: 2023-10-21
miasto: szczecin
licznik: 1
Jeśli ktoś wejdzie na stronę i wybierze inne miasto to do bazy trafia kolejny rekord:
id: 2
data: 2023-10-21
miasto: koszalin
licznik: 1
Teraz jeśli ktoś wejdzie na strone i wybierze ponownie szczecin to do bazy danych tam gdzie było id: 1 dodaje się tylko wartość do licznika.
Teraz mam tak, że jak ktoś ponownie wybierze szczecin to tworzy sie nowy rekord:
id: 3
data: 2023-10-21
miasto: szczecin
licznik: 2

Oczywiście przy zmianie daty ponownie pojawiają się nowe rekordy i tak dalej.

Dodałem to echo 'WIDZISZ MNIE?';exit; i pojawia się na stronie ale to chyba normalne?

Po co else if i else skoro i tak mają wykonać się takie same zapytania?

Inna sprawa, że zamiast tworzyć ify i kilka zapytań, wystarczy nadać na pole date i miasto klucz unique i stworzyć jedno zapytanie bez żadnych ifów:

[SQL] pobierz, plaintext 
INSERT INTO liczniki (date, miasto, licznik) VALUES ('".$dataa."','".$rodzaj_danych."',1)
ON DUPLICATE KEY UPDATE licznik=licznik+1;
[SQL] pobierz, plaintext 

Gdybyś użył powyższego, to nadając wartość domyślną 1 dla pola licznik, wystarczy wstawiać dane tylko do date i miasto.

Te wywołanie powoduje, że za każdym razem dodawany jest nowy rekord do BD

Ten post edytował brzanek 21.10.2023, 09:46:04

[trueblue]

Te wywołanie powoduje, że za każdym razem dodawany jest nowy rekord do BD

A założyłeś unikalny dwupolowy klucz na date i miasto?

[brzanek]

Niby założyłem unikalny klucz ale nie wiem czy dwupolowy

Ale nie działa to tak jak powinno

Dobra chyba sobie poradziłem - trueblue dzięki za pomoc