mysql-abfrage in php

[jayjay]

entweder bin ich blind, blöd oder keine ahnung was. einfache abfrage, ein link

<a href="mannschaft.php?mannschaft_id=1">1</a>

und dann folgt 2, 3 etc.
in der datei mannschaft:

<?php
$sql = "SELECT mannschaft,mannschaft_id FROM $table_mitglieder WHERE mannschaft_id=$mannschaft_id GROUP BY mannschaft_id";
//
$res = mysql_query($sql) or die(mysql_error());
while($row = mysql_fetch_array($res, MYSQL_ASSOC))
	{
	echo $row['mannschaft'];
	}
?>

als antwort bekomme ich von browser:
You have an error in your SQL syntax near '' at line 1

[Boris]

Lass Dir doch die Query $sql mal ausgeben (echo $sql) ... vielleichst siehst Du dann, woran es liegt.

[steffenk]

ich wette, Du hast mannschaft_id nicht ausgelesen:

$mannschaft_id=(isset($_GET['mannschaft_id'])) ? $_GET['mannschaft_id'] : 0;

[jayjay]

hatte ich vergessen, in phpmyadmin funzt sauber, zb WHERE mannschaft_id=1 gibt nur die einser aus usw.
@boris: etwas genauer, bin noch anfänger in php&mysql

[Boris]

Wenn seine register_globals an sind und er diese URL benutzt, dürfte es nicht daran liegen rein theoretisch.

[Boris]

Zitat von jayjay

@boris: etwas genauer, bin noch anfänger in php&mysql

Wie genau soll ich das noch schreiben?

Lass Dir Deine Query, die Du in $sql hast, einfach auf dem Bildschirm mit echo ausgeben ... halt so:

...
$sql = "SELECT mannschaft,mannschaft_id FROM $table_mitglieder WHERE mannschaft_id=$mannschaft_id GROUP BY mannschaft_id";
echo $sql;
...

[Adromir]

Ich würde das Query spontan so gestalten:

PHP-Code:

$sql = "SELECT mannschaft,mannschaft_id FROM $table_mitglieder WHERE mannschaft_id='$_GET[mannschaft_id]' GROUP BY mannschaft_id" 


EDIT

Was für Performanceunterschiede würde es eigentlich machen, im Query die mannschafts_id nicht aus der DB zu holen.. Diese ist ja vorher "bekannt"

[Noch ein Edit]
Woher kommt eigentlich $table_mitglieder?[/Noch ein Edit]

[Boris]

@Adromir:
Nee, das gibt einen netten Parse error - man kann Superglobals nicht simpel wie eine Variable einsetzen, das würde wenn dann nur gehen, wenn man es mit ". ... ." einsetzt:

 $sql = "
 	SELECT
 		mannschaft,
 		mannschaft_id
 	FROM
 		$table_mitglieder
 	WHERE
 		mannschaft_id='".$_GET[mannschaft_id]."'
 	GROUP BY
 		mannschaft_id"

[Adromir]

@Boris: Ich mach das immer so, und es gibt keine Probleme..

richtig wäre natürlich $_GET['mannschaft_id'] anstelle von $_GET[mannschaft_id] . Dies macht dann den Parse- Error..
Ohne die Quotes funzt es und man muss nicht diese Verschachtelung mit ". [..] ." machen..

Ich weiß, daß es als "unsauber" gilt die Quotes wegzulassen, so bemühe ihc mich aber auch gleichzeitig keine Geschützen Wörter zu verwenden..

[steffenk]

@Adromir - das kannst Du ja so machen, wenn Du weisst, was Du machst -
Als Tipp für andere sollte das aber nicht so gepostet werden, besser ist hier immer, die "saubere" Methode anzubieten

[Adromir]

Stimmt.. Hast ja recht

[jayjay]

Zitat von St@eff.en

ich wette, Du hast mannschaft_id nicht ausgelesen:

$mannschaft_id=(isset($_GET['mannschaft_id'])) ? $_GET['mannschaft_id'] : 0;

wette gewonnen.
das kommt vor die abfrage, oder?

$mannschaft_id=(isset($_GET['mannschaft_id'])) ? $_GET['mannschaft_id'] : 0;
$sql = "SELECT mannschaft_id FROM $table_mitglieder WHERE mannschaft_id=$mannschaft_id GROUP BY mannschaft_id";
//
$res = mysql_query($sql) or die(mysql_error());
while($row = mysql_fetch_array($res, MYSQL_ASSOC))
	{
	echo $row['mannschaft'];
	}

gibt nix aus

[Boris]

Mach doch mal endlich ein echo $sql und schreib uns, was da dann steht

[jayjay]

hier:

<?php
$sql = "SELECT mannschaft_id FROM $table_mitglieder WHERE mannschaft_id=$mannschaft_id GROUP BY mannschaft_id";
echo $sql;
?>

ausgabe:
SELECT mannschaft_id FROM bw_mitglieder WHERE mannschaft_id= GROUP BY mannschaft_id

[Boris]

In $mannschaft_id steht nix drin, also wird es nicht übertragen. Bist Du sicher, dass Du die PHP-Seite mit einer kompletten URL (z.B. ... seite.php?mannschaft_id=2) aufgerufen hast?