Wo liegt hier der Fehler? Radiobuttons wert speichern

[maydayz]

Ich weiß absolut nicht weiter.
es passiert einfach nix, es wird einfach kein wert übergeben, bitte findet den fehler

PHP-Code:

...
<td> <input type="radio" name="kategorie" value="spiele"> Spiele </td>
 </tr>
 <tr>
  <td> </td>
  <td>  <input type="radio" name="kategorie" value="skripts"> Skripts </td>
 </tr>
 <tr>
  <td> </td>
  <td> <input type="radio" name="kategorie" value="sounds">Sounds </td>
 </tr>
 <tr>
  <td> </td>
  <td><input type="radio" name="kategorie" value="sonstiges">Sonstiges </td>
 </tr>
  </table>

<?
 if (isset($_POST['submit231'])) {
 $kategorie=$_POST['kategorie']; 
   $query3= "INSERT INTO test (art)
   VALUES ('$kategorie')";
   mysql_query($query3, $conn);
   print '$kategorie';
} 
?><input class="norm" type="submit" name="submit231" value="los"/>
....

[webcreate]

Und wie schaut der form Tag aus?

[maydayz]

ich habs ma auf das wesentlichste beschränkt

PHP-Code:

 <form action="anzeigen_eing.php">
<table border="1" cellspacing="0" cellpadding="2">
<tr>Avatar
<td valign="top">
<table border=2>
 <tr>
  <td>Kleidung</td>
  <td> <input type="radio" name="kategorie" value="anzuge"> Anzüge </td>
 </tr>
<?
 if (isset($_POST['submit231'])) {
 $kategorie=$_POST['kategorie']; 
   $query3= "INSERT INTO test (art)
   VALUES ('$kategorie')";
   mysql_query($query3, $conn);
   print '$kategorie';
} 
?><input class="norm" type="submit" name="submit231" value="los"/>
</form>

[Adromir]

mach mal ein method="post" in den Form- Tag.

[maydayz]

ahh ich bin ein depp.
Großes dickes danke!!!
das hab ich voll übersehen.
und ich such schon seit 2 stunden nach dem fehler...lol

[steffenk]

trotzdem ist das gefährlich was Du machst, böse Leute löschen Dir damit die Datenbank.
Schau Dir mal SQL Injection an.

[maydayz]

Zitat von steffenk

trotzdem ist das gefährlich was Du machst, böse Leute löschen Dir damit die Datenbank.
Schau Dir mal SQL Injection an.

Wie meinst du das denn jetzt?
Geht doch garnicht? oder doch...

[Rizzo]

Zitat von maydayz

Geht doch garnicht? oder doch...

Was sagt Dir denn der logische Menchenverstand? Meinst Du ein kompetenter Programmierer, der seit einem Jahrzehnt (oder vielleicht auch länger) seine Brötchen in dieser Materie verdient, wirft das einfach so mal in den Raum, weil ihm gerade langweilig war?

Mal eine Gegenfrage: Wieso sollte es denn deiner Meinung nach unmögich sein hier etwas unterzuschieben? Weil Du dich darauf verläßt das von deinem Formular scheinbar nur ein kleiner Radiobutton versendet wird?

[maydayz]

mach mir jetzt keine angst.
Könntest du mir vielleicht lieber sagen wie ich das ganze dann verhindern kann?
bin dann auch ganz lieb und lernbereit

[webcreate]

Klar kann man Dir das sagen: Nutze die Suche!

Sorry aber das Thema wurde hier nun wirklich mehr als nur einmal behandelt.

[maydayz]

ist okay, dann mach ich mich jetzt mal auf die suche

[Rizzo]

Zitat von maydayz

ist okay, dann mach ich mich jetzt mal auf die suche

Kleiner Tipp dazu. Themen, die angepinnt und als Wichtig gekennzeichnet sind, könnten vielleicht, aber auch ganz vielleicht, schon wertvolle Links beinhalten, die einem eine lange Suche ersparen könnten.