Dynamisches Radiosystem | MySQL R5

Alter, du mit deinem Dynamisch...

Naja egal.
Nice Work

Du nutzt zu viele querys beim Laden
So hast du nur einen
stock loadradio()
{
new data[64],i=0;
mysql_query("SELECT * FROM radio");
mysql_store_result();
for(;i<mysql_num_rows();i++)
{
if(mysql_retrieve_row())
{
mysql_fetch_field_row(data,"name");
format(RadioInfo[i][RName],32,"%s",data);
mysql_fetch_field_row(data,"url");
format(RadioInfo[i][Rurl],32,"%s",data);
RadioInfo[i][erstellt]=true;
}
}
mysql_free_result();
return 1;
}

Sieht Nett aus.

Probiere es gleich mal aus und schreibe meine Bewertung anschließend.

Zitat von Beavis

for(;i<mysql_num_rows();i++)

Wie oft muss man euch das noch mit den Funktionen in Schleifenbedingungen noch erklären?

Jedesmal wenn die Schleifedurchläuft wird die Funktion erneut aufgerufen, das ist doch dumm.
Wozu soll das insgesamt gut sein?

Richtig für rein gar nichts, außer man möchte dauerhaft prüfen wie viele Zeilen da drin sind,
das lässt sich aber auch durch eine Variable lösen, die nicht immer wieder an die MySQL Verbindung zurück muss.

Achja, deklarationen von Schleifenvariablen, sofern sie nicht weiter genutzt werden, ist außerhalb NICHT sinnvoll

BlackAce ja ich weiß ich hätte den Wert zwischenspeichern müssen...

So meinte er es
stock loadradio()
{
new data[64];
mysql_query("SELECT * FROM radio");
mysql_store_result();
new row = mysql_num_rows();
for(new i=0;i<row;i++)
{
if(mysql_retrieve_row())
{
mysql_fetch_field_row(data,"name");
format(RadioInfo[i][RName],32,"%s",data);
mysql_fetch_field_row(data,"url");
format(RadioInfo[i][Rurl],32,"%s",data);
RadioInfo[i][erstellt]=true;
}
}
mysql_free_result();
return 1;
}

Farbe ändern wäre schön

Schade das du sowas einfach zum Runterladen anbietes somit haben es wieder viele 0815 Server und sagen sie hätten es gescriptet

10/10

Ich finde das Dynamische Radiosystem gut, haste gut hinbekommen.

Mit freundlichen Grüßen,

BaRkS

@Music4You

man tipps den Namen ein und den URL
dan kommt noch ein dialog und da kommt die farbe rein.
Zb TechnoBase ; bei.spiel ; ROT
dan erscheint TechnoBase in rot

Zitat von Trevor

Schade das du sowas einfach zum Runterladen anbietes somit haben es wieder viele 0815 Server und sagen sie hätten es gescriptet

10/10

Naja hast nicht ganz unrecht aber die meisten 0815 Server kennen sich mit sowas garn nicht erst aus.

@Music4You: Deine Dynamischen Werke gefallen mir einfach mach weiter so aber wie auch schon ein Vorposter geschreiben hat schade das du sowas zum
Download anbietes.

mfg xRaiiin

Schaut gut aus, weiter so.

Richtig schön gemacht 9/10