Neuer zufälliger Eintrag, wenn nicht schon vorhanden

N3mezis

Moin !

Ich bin grad am PHP lernen und stehe nun vor folgendem Problem:
Ich will einen "Planeten" bei der Anmeldung eines Users bei zufälligen Koordinaten erzeugen, wenn eben dort nicht schon einer ist. Leider funktioniert es nicht so ganz.
Ich würde darauf wetten, dass es irgendwie an der Schleifen und If Geschichte liegt, kann aber keinen Fehler entdecken.
Bekomme keinen Fehler - er scheint also irgendwie durch die Ifs zu kommen, aber eintragen tut er nix

Vielen Dank schonmal für die Hilfe!

Code

<?
function create_colony($user){
include("db_config.php");
	$PID = getownerPID($user);
	$x = rand(1,1000);
	$y = rand(1,1000);
	$z = rand(1,1000);
	echo "$x $y $z $PID";
	$abfragex = mysql_query("SELECT xpos FROM colonies");
	$abfragey = mysql_query("SELECT ypos FROM colonies");
	$abfragez = mysql_query("SELECT zpos FROM colonies");
 	while ($row = mysql_fetch_object ($abfragex)) {
	if	($row->xpos==$x) {
		while ($row = mysql_fetch_object ($abfragey)) {
		if	($row->ypos==$y) {
			while ($row = mysql_fetch_object ($abfragez)) {
			if	($row->zpos==$z) {
				create_colony($user);
			}
			else {
				mysql_query("INSERT INTO colonies (xpos, ypos, zpos, ownerID) VALUES ('$x','$y', '$z', '$PID')");	
			}
			}
		}
		}
	}		
	}
}


?>

Alles anzeigen

Mfg N3mezis

Boman

Ich glaube nicht, dass die else richtig platziert ist.
Gehe deinen Code nochmal durch und mach dir Gedanken darüber, wann die else aufgerufen wird.
Ich glaube dann findst du den Fehler.

Was mich aber noch wundert ist, dass er nichts einträgt. Wie lange ist denn die Laufzeit von deinem Script? Mehrere Sekunden?

N3mezis

Ja, das werde ich nochmal machen.

Die Laufzeit liegt deutlich unter ner sec.

Mfg N3m

N3mezis

sry 4 DP

so, geht nu ^^. Muss wohl irgendwie an den "OFF"-Schalter an meinem Kopf gekommen sein

Code

<?
function create_colony($user){
include("db_config.php");
	$PID = getownerPID($user);
	$x = rand(1,1000);
	$y = rand(1,1000);
	$z = rand(1,1000);
	$abfragex = mysql_query("SELECT xpos FROM colonies");
	$abfragey = mysql_query("SELECT ypos FROM colonies");
	$abfragez = mysql_query("SELECT zpos FROM colonies");
 	while ($row = mysql_fetch_object ($abfragex)) {
	if	($row->xpos==$x) {
		while ($row = mysql_fetch_object ($abfragey)) {
		if	($row->ypos==$y) {
			while ($row = mysql_fetch_object ($abfragez)) {
			if	($row->zpos==$z) {
				create_colony($user);
			}
			}
		}
		}
	}	
	else {
				mysql_query("INSERT INTO colonies (xpos, ypos, zpos, ownerID) VALUES ('$x','$y', '$z', '$PID')");	
	}	
	}
}


?>

Alles anzeigen

K.Y.L.T.

bist auch so einen Nudel

ich glaube aber du musst überall diesen elsefall reinmachen , oder nicht?

N3mezis

Ne, darf nicht.
sonst würde ich ja "abkürzen". 1,2,3 ist was anderes als 1,2,4 z.b.

oder meinst ne leere else?

K.Y.L.T.

Bzw. worin liegt dann der Sinn in der weiterin überprüfung, wenn du sowieso nur die erste koordinatstelle brauchst

N3mezis

Ne, ich brauche doch alle. Darum hatte ich die else ja so "tief" drinne. Aber auch die neue lösung funktioniert nicht mehr, nix geändert und plötzlich gehts nimmer

Mfg N3m

K.Y.L.T.

ich glaube du musst das etwas umbauen, du darfst das nicht verschachteln , sondern musst es nacheinander machen und wenn der Fall auftritt, dass die Koordinaten schon existieren eine Variable setzen, wenn die Variable .
z.B.
Fall1 $i =1;

Fall2 $i +=1;

Fall3 $i +=1;

und dann if($i == 3) also koordinaten identisch, dann neu einfügen

N3mezis

Ayyyye

genau der Gedanke kam mir auch grad. So, wie es imo ist kanns garnicht gehen

dangö

N3mezis

hm, da ich mich nicht anmelden kann muss ich wohl oder übel andauernd DPs machen

So, wie du es geschrieben hast gehts leider auch nicht.
Stell dir vor es gibt 3 planeten:
1,2,3
1,3,2
2,1,3

Wäre der wert für die erste variable schon 2

N3mezis

Problem ist, dass ich die nicht nacheinander vergleichen kann, da die x koodinate schon z.B. 10 mal vorkommt und nicht garantiert ist, dass die erste $x=xpos auch die gleichen $y=ypos usw. hat.

müsste die db durchgehen und alle tripel mit dem zufälligen vergleichen

Boman

Mir ist gerade eine entzückende Idee gekommen:
Du kannst doch einfach alle 3 Werte aus der Datenbank mit einer Abfrage rausholen.
Dann vergleichst du mit in der while-Schleife immer wieder ob die 3 Variablen übereinstimmen.
Falls ja, dann setzt du eine Variable auf false (dieser musst du vor der while true zuweisen).
Wenn dann am Ende der while die Variable true ist, dann dürfte die Position noch frei sein.

[EDIT]
Jetzt fällt mir auch der Fehler auf den du gemacht hast. du hast die Varieblen nicht paarweise verglichen, sondern den X-Wert des ersten "Planeten" danach die Y-Werte aller "Planeten" usw.

N3mezis

ui, super idee, mal schauen, ob ichs umsetzen kann. danke!

N3mezis

sieht nun folgendermaßen aus:

Code

<?
function create_colony($user){
include("db_config.php");
   $PID = getownerPID($user);
   $x = rand(1,1000);
   $y = rand(1,1000);
   $z = rand(1,1000);


$abfrage = mysql_query("SELECT xpos, ypos, zpos FROM colonies");
 while ($row = mysql_fetch_array($abfrage)) {
	if	($row["xpos"]->xpos==$x && $row["ypos"]->ypos==$y && $row["zpos"]->zpos==$z) {
	create_colony($user);
	exit;
	}
	else {
		mysql_query("INSERT INTO colonies (xpos, ypos, zpos, ownerID) VALUES ('$x','$y', '$z', '$PID')");   
  		exit;		
	}  
 }
}




?>

Alles anzeigen

Allerdings gehts nicht weiß nicht, wie ich das mit dem array machen soll liegt garantiert daran

Mfg N3m

Boman

Ich würde es so schreiben:

Code

if ($row["xpos"] == $x && ...

Denn ich kenne "->" nur von Objekten her. Kenne es aber nicht von Arrays.

N3mezis

Jetzt gehts garnicht mehr allerdings gibt er mir in der function keinen fehler, sondern in einer ganz anderen, die damit überhaupt nix zu tun hat (ist auch in ner anderen datei). Kann ich, weil sie in ner anderen Datei ist n Klammerfehler ausschließen ?

Code

<?
function create_colony($user){
include("db_config.php");
   $PID = getownerPID($user);
   $x = rand(1,1000);
   $y = rand(1,1000);
   $z = rand(1,1000);


$abfrage = mysql_query("SELECT xpos, ypos, zpos FROM colonies");
 while ($row = mysql_fetch_array($abfrage)) {
	if	($row["xpos"] == $x && $row["ypos"] == $y && $row["zpos"] == $z) {
	create_colony($user);
	exit;
	}
	else {
		mysql_query("INSERT INTO colonies (xpos, ypos, zpos, ownerID) VALUES ('$x','$y', '$z', '$PID')");   
  		exit;		
	}  
 }
}




?>

Alles anzeigen

N3mezis

oops. das erste exit; hab ich schon rausgenommen

Seht ihr denn irgend einen Fehler im code?

Code

<?
function create_colony($user){
include("db_config.php");
   $PID = getownerPID($user);
   $x = rand(1,1000);
   $y = rand(1,1000);
   $z = rand(1,1000);


$abfrage = mysql_query("SELECT xpos, ypos, zpos FROM colonies");
 while ($row = mysql_fetch_array($abfrage)) {
   if   ($row["xpos"] == $x && $row["ypos"] == $y && $row["zpos"] == $z) {
   create_colony($user);
   }
   else {
      mysql_query("INSERT INTO colonies (xpos, ypos, zpos, ownerID) VALUES ('$x','$y', '$z', '$PID')");   
        exit;      
   } 
 }
}




?>

Alles anzeigen

No0ob

Sind alle Datentypen in der Tabelle varchar? Oder warum setzt du alle in Anführungszeichen?

Neuer zufälliger Eintrag, wenn nicht schon vorhanden

Tags

Benutzer online in diesem Thema