Polecenie SQL (proste pytanie)

Question

Witam. Co jest nie tak z tą instrukcją?

INSERT INTO logs(`banned`) VALUES('1') WHERE `steamid`=76561198093864167;

Wywala syntax error jak próbuję ją wykonać :/ Jestem mocno zielony w tym więc proszę o pomoc :)

Pozdrawiam.

Answer 1

W INSERT nie da się użyć WHERE. Jest po prostu wstawianie danych do bazy, nie da się wstawić rekordu z jakimś warunkiem.

Comments

W jaki sposób mogę to inaczej zapisać? Tak żeby spełnić ten WHERE

---

A w jakim celu Ci wstawienie do bazy nowego rekordu z jakimś warunkiem? Jeśli wykonujesz to z PHP to w nim możesz sobie zrobić jakieś warunki, ale dalej nie rozumiem jaki to ma tutaj cel.

---

Piszę taki "panel zarządzania" użytkownikami dla administratora. https://gyazo.com/8318e38db24f99c6ad194bcfececf5a9

Fragment kodu wyświetlającego rekordy z bazy danych: 

			 if($steamid == 76561198093864167){ //sprawdzenie czy uzytkownik jest administratorem
			 	echo " "."<a href=\"delete.php\">Usun ostatnia wiadomosc tego uzytkownika</a><br />";
			 	$userid = "Id tej wiadomosci: ".$extract['id']."<br/>";
			 	echo $userid;
			 	$_SESSION['useridi'] = $userid;
			 	echo " "."<a href=\"deleteall.php\">Usun wszystkie wiadomosci uzytkownika</a>"."<br/>";
			 	echo " "."<a href=\"ban.php\">Zbanuj uzytkownika"."<br/>";
			 }

Kod ban.php: 

<?php
$con = mysql_connect('localhost', 'root', 'password');
mysql_select_db('table', $con);

session_start();
$steamid = $_SESSION['steamrealid'];

mysql_query("INSERT INTO logs(`banned`) VALUES('1') WHERE `steamid`=".$steamid);
header("Location: index.php");

?>

Jak widać chcę żeby po naciśnięciu tekstu Zbanuj użytkownika wywołało rekord który ustawi użytkownikowi banned=1.

---

Przede wszystkim to zapomnij o funkcjach zaczynających się od mysql_ - są one przestarzałe a w PHP7 już całkowicie usunięte. Należy użyć mysqli_ lub PDO.

Nie wiem jak chcesz to zrobić i jak później działa sprawdzenie tego bana ale to co masz to raczej złe podejście. Jeśli ban wykrywany jest po stemaid to należałoby w tabeli przechowywać właśnie to steamid a nie cyferkę 1, która nic nie mówi. I przy sprawdzaniu po prostu robić prosty SELECT z WHERE steamid i na podstawie tego idzie łatwo określić czy ktoś ma bana czy nie. Mówię, nie znam całości, struktury bazy i założenia jak to ma działać więc ciężko więcej powiedzieć, ale wg mnie podejście do tego jest złe.

---

Dobrze, dzięki. Utworzyłem nową tabelę do której wkładałem steamid użytkowników. Dzięki też za przypomnienie mi o zmienieniu mysql na mysqli :)

---

Nie ma problemu, cieszę się, że pomogłem :)