Logisches Problem bei Datenbankänderung

Fipsi · 23. Dezember 2012, 13:46

Hallo zusammen,

diesmal hab ich ein Problem, an dem ich wohl eher an meiner fehlenden Logik scheitere. Das hält mich schon seit Wochen auf.

Also folgendes:

Mein Rechtesystem läuft über Datenbanken (Gruppen eine Tabelle, sowie auch die Rechte und deren Zuweisung und die User-Gruppen Zuweisung).
Ich hab ein Forum bereits halbwegs fertig und jetzt haperts an der Rechtezuweisung.
Wenn ich auf Rechte klicke, muss ich das Forum aufwählen und dann erscheint einer Liste aller Gruppen (wird aus der Datenbank aufgelesen, ebenso die Foren).
Ob jetzt bereits eine Gruppe schon ein Recht hat, wird dann über die Tabelle der gruppen/Rechtezuweisung ausgelesen.

Wenn ich jetzt auf Speichern kling, hängts.. (ab hier bin ich noch nicht weiter).
Ich hab bereits eine neue Schleife angelegt, dass alle Gruppen abgerufen werden. Aber wie vergleich ich jetzt, ob das schon vorhanden ist, und notfalls eintrag/rauslösch und wenns einen Fehler gibt abbreche?

Ich weiß nicht, ob ihr damit jetzt viel anfangen könnt, wenn nicht, liefer ich noch Codes dazu.

Vielen Dank schonmal

Fipsi

shureg · 23. Dezember 2012, 15:27

kannst ja erst selektieren über dein unique key und prüfen ob der datensatzt shcon da ist

Fipsi · 23. Dezember 2012, 15:38

Ja, soweit bin ich auch schon.. aber mir happerts jez, wie ich des umsetz.

Bring wohl besser Code-Teile mit rein:

Quellcode

$sql1 = "SELECT grup_id, grup_name FROM gruppen";
$ergebnis1 = $mysqli->query($sql1);
$sql2 = "SELECT rechte_id FROM forum_kate WHERE id = $kate_id";
$ergebnis2 = $mysqli->query($sql2);
$row2 = $ergebnis2->fetch_object();
if ($_SERVER['REQUEST_METHOD'] == "POST")
{
while ($row1 = $ergebnis1->fetch_object())
{
}
}
$seiten_inhalt .= "<form action=\"\" method=\"POST\">\r\n";
$seiten_inhalt .= "<table width=\"100%\">\r\n";
while ($row1 = $ergebnis1->fetch_object())
{
$sql3 = "SELECT re_id FROM f_rechte WHERE re_name = '".$row2->rechte_id."_see'";
$ergebnis3 = $mysqli->query($sql3);
while ($row3 = $ergebnis3->fetch_object())
{
$sql4 = "SELECT f_grupid FROM f_admin_rechte WHERE f_reid = $row3->re_id";
$ergebnis4 = $mysqli->query($sql4);
while ($row4 = $ergebnis4->fetch_object())
{
if ($row4->f_grupid == $row1->grup_id)
{
$get_right = TRUE;
}
}
}
$seiten_inhalt .= "<tr><td onclick=\"spoil('grup_".$row1->grup_id."')\"><u>".$row1->grup_name."</u></td>\r\n";
$seiten_inhalt .= "<td> </td></tr>\r\n";
$seiten_inhalt .= "<tr id=\"grup_".$row1->grup_id."\" style=\"display:none; background:#ffffff;\"><td>Darf die Kategorie sehen</td>\r\n";
$seiten_inhalt .= "<td>Ja <input type=\"radio\" name=\"".$row1->grup_id."_".$row2->rechte_id."_see\" value=\"1\" ";
if ($get_right == TRUE) {
$seiten_inhalt .= "checked=\"checked\" "; }
$seiten_inhalt .= "/> <input type=\"radio\" name=\"".$row1->grup_id."_".$row2->rechte_id."_see\" value=\"0\" ";
if ($get_right != TRUE) {
$seiten_inhalt .= "checked=\"checked\" "; }
$seiten_inhalt .= "/> Nein</td></tr>\r\n";
}
$seiten_inhalt .= "<tr><td> </td>\r\n";
$seiten_inhalt .= "<td><input type=\"submit\" value=\"Speichern\" /></td></tr>\r\n";
$seiten_inhalt .= "</table>\r\n";
$seiten_inhalt .= "</form>\r\n";

Alles anzeigen

Und da hängt's jetzt dran, an dem Teil (if ($_SERVER['REQUEST_METHOD'] == "POST")), wie ich das am besten mach, da ich die Rechte gegliedert hab in $zahl_$wort, $zahl steht hierfür für das Forum/die Kategorie und $wort dann, ob er lesen, schreiben, editieren u. w. machen darf.

Tolive · 23. Dezember 2012, 21:09

Entweder zerteilst du $zahl_$wort mit z.B. str_split() oder du vergleichst $zahl_$wort mit $zahl.$wort.

Fipsi · 23. Dezember 2012, 21:15

mh.. ich glaub, ich muss da nochmal einiges in der praxis ausprobieren.. aber den tipp merk ich mir, danke

Fipsi · 23. Dezember 2012, 23:18

soo..
also der Teil im if ($_SERVER['REQUEST_METHOD'] == "POST") sieht jetzt folgendermaßen aus:

Quellcode

while ($row1 = $ergebnis1->fetch_object())
{
$recht_name = $row2->rechte_id."_see";
$formular_data = $_POST[$row1->grup_id."_".$recht_name];
$sql3 = "SELECT re_id FROM f_rechte WHERE re_name = '".$recht_name."'";
$ergebnis3 = $mysqli->query($sql3);
$row3 = $ergebnis3->fetch_object();
$sql4 = "SELECT f_grupid FROM f_admin_rechte WHERE f_reid = $row3->re_id";
$ergebnis4 = $mysqli->query($sql4);
$row4 = $ergebnis4->fetch_object();
if ($row4->f_grupid == $row1->grup_id)
{
if ($formular_data == "0")
{
$sql5 = "DELETE FROM ".$db_data['datenbank'].".`f_admin_rechte` WHERE `f_admin_rechte`.`f_reid` = ".$row3->re_id." AND `f_admin_rechte`.`f_grupid` = ".$row4->f_grupid." LIMIT 1";
$delete = $mysqli->query($sql5);
if ($delete != TRUE)
{
$error = TRUE;
break;
}
}
}
else if ($ergebnis4->num_rows == NULL)
{
if ($formular_data == "1")
{
$sql5 = "INSERT INTO `f_admin_rechte`
(`f_reid` , `f_grupid`)
VALUES
('$row3->re_id' , '$row1->grup_id')";
$eintragen = $mysqli->query($sql5)or die($mysqli->error);
//echo $mysqli->error;
if ($eintragen != TRUE)
{
$error = TRUE;
break;
}
}
}
}
if ($error == TRUE)
{
header("location:forum.php?do=kategorie&take=rechte&id=".$kate_id."&done=false");
}
else
{
header("location:forum.php?do=kategorie&take=rechte&id=".$kate_id."&done=true");
}

Alles anzeigen

Das funktioniert (überraschenderweise^^) sogar fast so ganz, wie ichs will. Aber eben leider nur fast:
Nach aktuellem System durchläuft er immer nur solange bis eine Gruppe ein Recht eingeschrieben bekommt und dann bricht die Schleife hab.. kann mir jemand sagen, wieso?

Fipsi · 23. Dezember 2012, 23:28

Ok, hab den Fehler gefunden.. hat sich erledigt, aber danke für die denkanstöße.. haben doch etwas auf die Sprünge geholfen

Exi · 24. Dezember 2012, 12:06

Wie hast du es nun gelöst bzw. was war der Fehler?

Fipsi · 24. Dezember 2012, 12:42

wegen dem "else if ($ergebnis4->num_rows == NULL)" (hier in Zeile 28).

Einfach noch ein "||($row4->f_grupid != $row1->grup_id)" dran setzen und dann läuft die Schleife weiter durch.

Tolive · 24. Dezember 2012, 22:43

PS: Wenn du Lust hast kanst du dir ja mal nach "left join" googlen. Damit kannst du deine Datenbankabfrage optimieren und übersichtlicher gestalten.

Fipsi · 24. Dezember 2012, 22:48

Außer dass in dem SQL-Befehl dann des (left) join und on steht, hab ich sowas ähnliches im Code schon an anderer stelle verarbeitet.. da ich aber bis zu dem Teil noch nicht wusste, warums nicht funktioniert, hab ich den hier geposteten Teil noch nicht überarbeitet. Steht aber auf der to-do-Liste. Danke noch für den Tipp

Tolive · 24. Dezember 2012, 22:57

Fipsi schrieb:

Außer dass in dem SQL-Befehl dann des (left) join und on steht, hab ich sowas ähnliches im Code schon an anderer stelle verarbeitet..

Es geht ja nicht nur ums Aussehen. Es geht ja auch darum wie die Abfrage funktioniert. Im aktuellen Stand holst du dir Datensätze um einen Teil der Datensätze wieder an die DB zurückzuschicken (für die where Bedingung). Dieses kostet nicht nur Zeit sondern auch Last. Mit einem join würdest du gleich das Gesamtergebnis bekommen.

Bei einem kleinen Script wie deinem Sicherlich kein Problem, doch bei Anwendungen mit vielen Benutzern sind solche Fehler in Summe fatal.

Fipsi · 24. Dezember 2012, 23:03

Ok, dann werd ich mich doch bei Zeiten mal in die Join's einlesen.
Vielen Dank

Quellcode

Quellcode

Teilen