[Fia]

Hej!
Jag har ett recept som har flera altenativ som tex GI, billigt, small mm Dessa ligger i en egen tabell och läggs in via en loop

while (list ($key,$val) = @each ($alternativ)) {
mysql_query("INSERT INTO tblReceptalt SET recId='$recId',altId='$val'") or die("Det har uppstått ett fel:" . mysql_error());
}

Nu skulle jag behöva att användaren kan uppdatera denna infomation så vid uppdatering hämtas infomationen till formuläret vilka som är valda sedan tidigare men sen när man ska skicka in det och uppdatera det till databasen kan jag inte riktigt få till det. Arryen från checkboxen inehåller väll bara infomation från de valda boxarna så när jag loopar igenom och en post ska tas bort hur gör man då? fins det någon som har haft något liknande någon gång?

kombinationen av recId och altId är unique

[Jonas]

Enklaste är att tömma all information som finns om receptet när man uppdaterar.

Sedan skulle jag rekommendera dig att göra följande:

PHP-kod:

while (list ($key,$val) = @each ($alternativ)) { 
mysql_query("INSERT INTO tblReceptalt SET recId='".mysql_real_escape_string($recId)."',altId='".mysql_real_escape_string($val)."'") or die("Det har uppstått ett fel:" . mysql_error()); 
} 


Eftersom din kod idag innehåller möjligheten till SQL-injection. Vilket mer eller mindre tillåter en att skicka oönskad information eller helt enkelt köra SQL-kod direkt mot din databas.

[allstars]

Har du funderat på att använda ajax?
Då ändrar du en checkbox åtgången och sparningen till databasen sker direkt.
Även ny post och radering av post fungerar fint med denna metod.
Dessutom är det rätt fräckt att ha på sin site

[Fia]

mysql_real_escape_string använder jag i en function som jag kör hela post variablen igenom där gör jag litte koller på det mesta. Jag använder Ajax vid ett par andra tillfällen som tex när man spara receptet in till sin användare men vid detta tillfället vill jag nog köra att spara allt på en knapptryckning.
Jag funderade på det också att slänga bort allt och lägga in alternativen på nytt men tänkte att det kanske fans ett annat alltenativ som att först lägga till nya och sen göra någon typ av delete på de som inte längre används. man kanske kunde lägga in en timestamp vid de nya och existerande och slänga bort de som inte har rätt timestamp sen. kanske lite overkill jag kastar nog bort och lägger in på nytt.
Tack för tipsen!