Dropdown z lokalizacją Opcje

[aerospeed]

Witam szanowne grono forumowiczów

Mam problem z działaniem potrójnego selecta z lokalizacją (klasyczne country, state, city).

index.php

[PHP] pobierz, plaintext 
	<select id="countrydd" onChange="changeCountry()">
		<option value="0">Select Country</option>
			<?php
			$countries_count = $connect->query("SELECT * FROM countries");
			while($cc = $countries_count->fetch_assoc())
			{
			echo '<option value="'.$cc['id'].'">'.$cc['name'].'</option>';
			}
			?>
	</select>
 
<div id="state">
	<select>
		 <option>Select State</option>
	</select>
</div>
 
<div id="city">
	<select>
		<option>Select City</option>
	</select>
</div>
 
<script type="text/javascript">
	function changeCountry()
		{
			var xmlhttp = new XMLHttpRequest();
			xmlhttp.open("GET", "ajax.php?country="+$("#countrydd").val(), false);
			xmlhttp.send(null);
			$("#state").html(xmlhttp.responseText);
		}
 
	function changeState()
		{
			var xmlhttp = new XMLHttpRequest();
			xmlhttp.open("GET", "ajax.php?state="+$("#statedd").val(), false);
			xmlhttp.send(null);
			$("#city").html(xmlhttp.responseText);
		}
 
</script>
[PHP] pobierz, plaintext 

ajax.php

[PHP] pobierz, plaintext 
<?php
 
	require_once "connection.php";
	mysqli_report(MYSQLI_REPORT_STRICT);
 
		$connect = new mysqli($host, $db_user, $db_password, $db_name);
		$country = $_GET["country"];
		$state = $_GET["state"];
 
		if(isset ($country))
		{
 
			$states_count = $connect->query("SELECT * FROM states WHERE country_id='$country'");
 
			if($states_count->num_rows > 0)
			{
				echo '<select id="statedd" onChange="changeState()">
								<option>Select State</option>';				
 
					while($sc = $states_count->fetch_assoc())
					{
						echo '<option value="'.$sc['id'].'">'.$sc['name'].'</option>';
					}				
 
				echo '</select>';
			}
			else
			{
				echo '<select><option>Select State</option></select>';
			}
 
		}
 
		if(isset($state))
		{
 
			$cities_count = $connect->query("SELECT * FROM cities WHERE state_id='$state'");
 
			if($cities_count->num_rows > 0)
			{
				echo '<select>
								<option>Select City</option>';				
 
					while($cc = $cities_count->fetch_assoc())
					{
						echo '<option value="'.$cc['id'].'">'.$cc['name'].'</option>';
					}				
 
				echo '</select>';
			}
			else
			{
				echo '<select><option>Select City</option></select>';
			}
 
		}
 
?>
[PHP] pobierz, plaintext 

Teraz, problem w tym, że pomimo iż skrypt działa, bo wyświetla prawidłowe wyniki z bazy danych, to z jakiegoś powodu php wyrzuca na ekran błędy:

[PHP] pobierz, plaintext 
Notice: Undefined index: state in C:\xampp\htdocs\test\ajax.php on line 8
Notice: Undefined index: country in C:\xampp\htdocs\test\ajax.php on line 7
[PHP] pobierz, plaintext 

Próbowałem już na wszystkie znane mi sposoby to naprawić i nadal nie wiem w czym problem.
Z góry dzięki za pomoc.
Pozdrawiam

[nospor]

Komunikat bledu jest dosc wyrazny...
W necie pelno rzeczy na ten temat...

Przenosze. Problem nie ma zadnego zwiazku z AJAX.

Co do problemu:

nie:
$country = $_GET["country"];

a:
$country = isset($_GET["country"]) ? $_GET["country"] : null;

Analogicznie reszta

[viking]

Albo w PHP7 $country = $_GET['country'] ?? null; (trzymam się wersji że v7 była tak dawno wydana że nie stanowi już problemu, zwlaszcza że niedługi 7.2).
Poza tym korzystasz z jquery zatem poc co ci ten obiekt XMLHttpRequest? Nigdzie nie sprawdzasz co zwraca zapytanie a że jest podatne na sql injection zwrócić może cokolwiek. I tak masz śmietnik ale możesz zacząć od wyrzucenia onChange="changeState()" -> zewnętrzny plik i .on() https://prophp.pl/article/15/jquery%2C_zdar...e_metoda_%24.on

[nospor]

@viking jeszcze troche minie zanim zaczne podawac na forum skladnie php7. Ludzie tutaj potrafia siedziec na starociach

[viking]

Rok i 10 miesięcy to staroć Nie spotkałem hostingu który by nie miał 7.

[nospor]

Mialem na mysli starsze starocie
I co z tego ze hostingi maja php7. Ja mowie o szarych uzytkownikach, ktorzy pomimo, ze maja php7 na hostingu to nadal siedza na php5.3

@aerospeed sorry za lekki offtopic ktory tu ci robimy ale takie nudy na forum....

[aerospeed]

Dzieki panowie

Oba sposoby działają. Zakładam, że to jest to samo wyrażenie tylko pod inne wersje php?

Nigdzie nie sprawdzasz co zwraca zapytanie(...)

Nie bardzo rozumiem

sorry za lekki offtopic ktory tu ci robimy ale takie nudy na forum....

Jęsli o to chodzi to dla mnie to żaden problem na forum się rozmawia o wszystkim

Poza tym korzystasz z jquery zatem poc co ci ten obiekt XMLHttpRequest?
I tak masz śmietnik ale możesz zacząć od wyrzucenia onChange="changeState()" -> zewnętrzny plik i .on()

Korzystałem z poradnika jak napisać ten skrypt i bardzo nie podobało mi się użycie document.getElementByID, co mogłem zamieniłem na jquery. Jednakże nie jestem jakiś biegły w javascipt (pewnie widać ) i nie mam za bardzo pomysłu czego użyć zamiast XMLHttpRequest i jak zamienić kod z użyciem .on()

[nospor]

Oba sposoby działają. Zakładam, że to jest to samo wyrażenie tylko pod inne wersje php?

Podane przez vikinga dziala tylko na php7. Podane przeze mnie dziala na kazdej wersji php. Co nie zmienia faktu, ze jednak masz php7 co sie chwali i lepiej bys uzywal skladni php7 (tej od vikinga) bo jest lepsza/krotsza/piekniejsza/laski szaleja jak jej uzywasz...

[aerospeed]

Słowem podsumowania: czy zawsze, kiedy robię porównanie zmiennej w ajax, powinienem dodać na końcu "?? null;"?
Bo nie wiem czy to jest kwestia tego konkretnego przykładu, czy też taki błąd będzie zawsze.

lepiej bys uzywal skladni php7 (tej od vikinga) bo jest lepsza/krotsza/piekniejsza/laski szaleja jak jej uzywasz...

hehe

[nospor]

to nie jest zadna zmienna AJAX. To jest zmienna ktora pochodzi z URL i niewazne czy to idzie przez normalny link czy przez AJAX.
Kolejna sprawa tu nie chodzi o zmienne URL tylko o generalnie dobieranie sie do indexu tablicy (tak, $_GET to tablica), ktory nie istnieje.


Generalnie lezysz na podstawach i moze wpierw wypadaloby ogarnac podstawy php, zmienne, tablice, teksty, petle, if a dopiero potem brac sie za budowanie czegos.

[viking]

Zamiast XHR http://api.jquery.com/jquery.ajax/
mysqli::query http://php.net/manual/en/mysqli.query.php ma definicję

[PHP] pobierz, plaintext 
mixed mysqli::query ( string $query [, int $resultmode = MYSQLI_STORE_RESULT ] )
[PHP] pobierz, plaintext 

gdzie to mixed to

Returns FALSE on failure. For successful SELECT, SHOW, DESCRIBE or EXPLAIN queries mysqli_query() will return a mysqli_result object. For other successful queries mysqli_query() will return TRUE.

Czyli u ciebie może się zdarzyć kod false->num_rows > 0
O sql injection już pisałem. Poczytaj o prepare.