ID`s ophalen en deze vergelijken met de ID`s uit ander database

Door Arno van Zanten op 07-04-2023 15:38

2.041 views

Ik heb per gebruiker bepaalde opties, deze opties staan in de database in een andere tabel.
Nu per gebruiker worden deze opgeslagen in een andere tabel, enkel met de Optie_ID.

Dus even een korte samen vating, ik ben weer met iets te moeilijks bezig.
Kan wel weer op YT gaan zoeken etc... maar daar zal ik weer hele oude codes tegen komen.
Volgens mij moet ik nu een functie gaan aanmaken, maar dat weet ik dus ook even niet.


<?php
                $sql = 'SELECT * FROM UserOptions WHERE User_ID = '.$UserID.'';
		$result = mysqli_query($connect, $sql);
		$rows = mysqli_num_rows($result);
			if($rows == 0) {
				echo "<div class='view-grid'>".$rows." = No Options found by user!<br /></div>";
			exit;
			}else{
				$optionid = mysqli_fetch_all($result, MYSQLI_ASSOC);
				
				echo "<div class='view-grid'>Options that the user provide</div>";
				$Uoptions = mysqli_fetch_all($result, MYSQLI_ASSOC);
				foreach($Uoptions as $options) {
					$services[] = $options['Options_ID'];
				}
			}
?>

Dit is wat ik nu heb, maar hij moet de ID`s vergelijken met de ID`s uit een andere tabel.
Ik denk dat ik met deze code verkeerd bezig ben en dit naar een functie moet omzetten.
Ik heb een tabel : Options waarin alle opties instaan waar de user uit kan kiezen, deze worden dan opgeslagen in UserOptions, maar dan enkel de Option_ID en User_ID, dus nu moet ik denk ik er eerst een functie van maken of kan ik het op een betere manier doen?

- Ariën -

07-04-2023 15:50 gewijzigd op 07-04-2023 15:54

Heb je delen (geen data) uit je code die je herhaalt? Dan is een functie interessant. En anders is het prima.

Ik zou wel je exit; weghalen. Ik schop ook nooit iemand de deur uit als ze geen cadeautjes bij zich hebben ;-).
Lijn 9 is ook zinloos.

Arno van Zanten

07-04-2023 15:55

lol, Ja in principe herhaalt de code zich grotendeels wel ja

- Ariën -

07-04-2023 16:00 gewijzigd op 07-04-2023 16:06

Met welke reden in gedachte? Dit lijkt mij een script die zich een keer uitvoert per aanvraag. Want waar komt $UserID vandaan?

Arno van Zanten

07-04-2023 16:10

uit de $_GET

- Ariën -

07-04-2023 16:15

Dus herhaalt het zich niet, omdat het om één specifieke user gaat.

PS: Denk aan SQL-injection bij $_GET!!

Arno van Zanten

07-04-2023 16:38

Ja ik zal er vanavond of anders morgen naar kijken, want ik moet nu ineens weg voor werk.
Alvast bedankt voor een paar goede tips.

Arno van Zanten

09-04-2023 17:18

Ik heb eens zitten kijken op internet en ik kwam de optie inner join tegen, zou dit de oplossing voor mijn vraag zijn?

[size=xsmall]Toevoeging op 09/04/2023 17:57:44:[/size]

Ik heb een database naam : Vakantie
Daarin heb ik dus een tabel options, met deze velden
|ID|Option|Term|
De andere tabel is UserOptions, met deze velden
|ID|Options_ID|User_ID

Nu moet dus gekeken worden per gebruiker, welke Options zij hebben, dus ik zoeken hoe ik dit kon doen, kwam ik INNER JOIN tegen op internet. Is dit de oplossing en zo ja, hoe pass ik dit goed toe?

[size=xsmall]Toevoeging op 09/04/2023 18:41:34:[/size]

ik heb nu dit in de database toegepast, maar als ik de LIMIT weg laat, haat hij alles op, ook waar geen user_id bij staat.


<?php
Select UserOptions.User_ID FROM UserOptions INNER JOIN Options ON Options.ID WHERE UserOptions.User_ID = 3;
?>

[size=xsmall]Toevoeging op 09/04/2023 19:04:41:[/size]

Ik heb de code en werkt.


<?php
$sql = 'SELECT * FROM UserOptions WHERE User_ID = '.$UserID.'';
		$result = mysqli_query($connect, $sql);
		$rows = mysqli_num_rows($result);
			if($rows == 0) {

				echo "<div class='view-grid'>".$rows." = No Options found by user!<br /></div>";

			}else{

				echo "<div class='view-grid'>Options that the user provide</div>";
				$MySql = 'Select UserOptions.Options_ID, Options.Option FROM UserOptions INNER JOIN Options ON Options.ID WHERE Options.ID = UserOptions.Options_ID;';
				$myresult = mysqli_query($connect, $MySql);
				$uoptions = mysqli_fetch_all($myresult, MYSQLI_ASSOC);
				foreach($uoptions as $options) {

					echo "<div>".$options['Option']."</div>";

				}
			}
?>

Ivo P

09-04-2023 21:58

inner join kan wel, maar dan moet je hem goed schrijven



Select UserOptions.User_ID
 FROM UserOptions 
INNER JOIN Options ON Options.ID = UserOptions.Option_ID
WHERE UserOptions.User_ID = 3;

Reageren

Inloggen om te reageren