Einfacher Scripten (Ressourcenschonender)

5Gum · 10. Oktober 2013

Guten Abend,

Ich wollte mir nun ein Pfandflaschensystem bauen, und ich mache es glaub ich ressourcenfressend.

new flasche1,flasche2;

//PFANDFLASCHEN LS
flasche1 = CreatePickup(1551,2,418.2114,-1545.0438,27.5781,-1);//1
flasche2 = CreatePickup(1551,2,509.3970,-1648.9506,18.6278,-1);//2

public OnPlayerPickUpPickup(playerid, pickupid)
{
if(pickupid == flasche1)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[128],string[128];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
format(string,sizeof(string),"Aktuell: %d",Spieler[playerid][pfandflaschen]);
SendClientMessage(playerid,weis,string);
return 1;
}
else if(pickupid == flasche2)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[128],string[128];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
format(string,sizeof(string),"Aktuell: %d",Spieler[playerid][pfandflaschen]);
SendClientMessage(playerid,weis,string);
return 1;
}
return 1;
}

Kann man soetwas ressourcenschonender schreiben?

Akino Kiritani · 10. Oktober 2013

Also eins fällt mir schonmal auf.

Deine strings sind mit 128 doch etwas groß, solltest mal nachzählen / schätzen wie viele du brauchst.
Tipp: http://www.woerter-zaehlen.de/

Dann vergiss nicht, das du strings mehrfach benutze kannst, in dem du sie neu formatierst,
somit sparst du dir einen.

Ansonsten sieht das ganz vernünftig aus.
Nur stellt sich die frage, ist "Speichern" auch so vernünftig, das gehört ja auch dazu

5Gum · 10. Oktober 2013

Ist es so besser?

public OnPlayerPickUpPickup(playerid, pickupid)
{
new string[14];
format(string,sizeof(string),"Aktuell: %d",Spieler[playerid][pfandflaschen]);
if(pickupid == flasche1)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[128];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
SendClientMessage(playerid,weis,string);
return 1;
}
else if(pickupid == flasche2)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[128];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
SendClientMessage(playerid,weis,string);
return 1;
}
return 1;
}

ohitsandrzej · 10. Oktober 2013

Kann man viel dynamischer machen, bzw in n enum oder speichern / laden lassen.
Gibt schon paar Möglichkeiten.

Akino Kiritani · 10. Oktober 2013

@Andrzejxy':
Da würde ich gar nicht drüber meckern, da du gar nicht weißt wie das insgesamt aussieht

@BaRkS:
Nicht wirklich.

Würde das eher so machen:
public OnPlayerPickUpPickup(playerid, pickupid)
{
if(pickupid == flasche1)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[90];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
format(query,sizeof(query),"Aktuell: %d",Spieler[playerid][pfandflaschen]);
SendClientMessage(playerid,weis,query);
return 1;
}
else if(pickupid == flasche2)
{
SendClientMessage(playerid,weis," Du hast eine Pfandflasche gefunden!");
Spieler[playerid][pfandflaschen] ++;
Speichern(playerid);
new query[90];
format(query,sizeof(query),"SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s'",Spieler[playerid][pfandflaschen],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
format(query,sizeof(query),"Aktuell: %d",Spieler[playerid][pfandflaschen]);
SendClientMessage(playerid,weis,string);
return 1;
}
return 1;
}
Weil sonst hast du zwar eine Pfandflasche gefunden, aber die Anzahl ist defakto falsch

5Gum · 10. Oktober 2013

Achso meinst du das
Falsch verstanden

Dankeschön!

Slash™ · 10. Oktober 2013

@BlackAce:
Warum den query nicht direkt unter den public?, weil so würde man ja auch 2 strings erstellen, oder hab ich nen denkfehler?

5Gum · 10. Oktober 2013

@BlackAce:

Ich brauche deine Hilfe noch einmal.

Der Speichert das nicht ab

stock Speichern(playerid)
{
if(Spieler[playerid][eingeloggt] == 1)
{
new query[250];
format(query,sizeof(query),"UPDATE accounts SET level='%i',admin='%i',pfandflaschen='%i',rucksack='%i' WHERE Username='%s'",GetPlayerScore(playerid),Spieler[playerid][admin],Spieler[playerid][pfandflaschen],Spieler[playerid][rucksack],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
}
return 1;
}

Akino Kiritani · 10. Oktober 2013

Zitat von Music4You

Warum den query nicht direkt unter den public?

Das wäre nur dann sinnig, wenn er keine anderen Pickups habt, die er verwenden möchte oder das CallBack nur dafür verwendet,
ich gaube nicht das er genau das vorhat

@BaRkS:
Nicht eingeloggt oder deine Tabelle stimmt ncht mit deinem Query.
Du solltest dir mal die dazugehörige Log angucken

Tion · 10. Oktober 2013

SQL

SELECT pfandflaschen='%i' FROM accounts WHERE Username='%s

Bin ich der einzige, dem diese Query suspekt vorkommt ?
Vielleicht liegt der Fehler hier dran ?

Akino Kiritani · 10. Oktober 2013

Zitat von Tion

Bin ich der einzige, dem diese Query suspekt vorkommt ?

Stimmt ist mir gar nicht aufgefallen

Das würde aber nicht erklären, warum er nicht speichert

5Gum · 10. Oktober 2013

Ich bin eingeloggt, und Tabelle passt auch

Tion · 10. Oktober 2013

Zitat von BlackAce

Das würde aber nicht erklären, warum er nicht speichert

Ich gebe mal 3 Theorien ab:

1. Die Query in "Speichern" hat einen Syntax-Fehler
Zwar sollte es rein theoretisch egal sein, aber man kann ja mal testweise eine komplett wohlgeformte MySQL-Query basteln ( wie es sich gehört )

SQL

UPDATE `accounts` SET `level` = %i, `admin` = %i, `pfandflaschen` = %i, `rucksack` = %i WHERE `Username` = "%s"

Vielleicht geht es dann ja, weil einfache Striche um Integer... ewww....

2. Die total falsch geformte Query raus
Siehe 2 Beiträge vorher. Vielleicht gibt es ja echt so etwas, wenn ich mal referenzieren darf...

PHP

if(($res = 5*2) >= 10) {  }

Wenn es das auch mit Select gibt -> überschrieben ?

3. Verschwörung der Illuminaten
Sie wurden dadurch verärgert, dass ihr Strohmann aus dem Adminteam getreten ist und sie sich einen anderen bestechlichen Mod suchen müssen

Trevor · 10. Oktober 2013

new query[60]; reicht auch wens nix großes ist

5Gum · 10. Oktober 2013

Hab das mal eingefügt, geht aber nicht

Akino Kiritani · 10. Oktober 2013

Zitat von Tion

Vielleicht geht es dann ja, weil einfache Striche um Integer... ewww....

Beim Integer ist das defakto egal, eigentlich macht man das mit den ' immer wenn man einen Wert festlegt, egal ob beim WHERE SET oder sonst wo.

Wenn du Werte festlegen willst nuztz man die ' nunmal, das hat nichts mit Integer Float oder nem Text zu tun.

Ich würde viel eher darauf tippen, das seine Tabelle die Großschreibung drin hat.
Also
Accounts Level ...
und nicht
accounts level ...

5Gum · 10. Oktober 2013

Nein, Großschreibung war es ja auch nicht, es muss an den Pfandflaschen liegen, vorher ging das ja

Tion · 10. Oktober 2013

Zitat von BlackAce

[...]
Wenn du Werte festlegen willst nuztz man die ' nunmal, das hat nichts mit Integer Float oder nem Text zu tun.
[...]

Bei Integers / Floats / Doubles / Longs braucht man die nicht, habe sie da nie gebraucht. Aber man lernt nie dazu

Für mal bitte in phpMyAdmin ( oder ähnliches ) folgenden Befehl aus:

SQL

SHOW CREATE TABLE `accounts`;

Bekomme ich noch einen Like von @Andrzejxy': ? D:

5Gum · 10. Oktober 2013

Mir ist gerade aufgefallen, der speichert auch das Level usw. nicht mehr.

/edit

Das kommt, ist das richtig?
Tion:

/edit 2

Ich habe bei dem stock das einmal geprintet, und habe gesehen, dass es gar nicht geprintet wird, beim ausloggen

stock Speichern(playerid)
{
if(Spieler[playerid][eingeloggt] == 1)
{
new query[250];
format(query,sizeof(query),"UPDATE accounts SET level='%i',admin='%i',pfandflaschen='%i',rucksack='%i' WHERE Username='%s'",GetPlayerScore(playerid),Spieler[playerid][admin],Spieler[playerid][pfandflaschen],Spieler[playerid][rucksack],Name(playerid));
mysql_function_query(dbhandle,query,false,"","");
print(query);
}
return 1;
}

public OnPlayerDisconnect(playerid, reason)
{
Speichern(playerid);
Reset(playerid);
return 1;
}

Tion · 10. Oktober 2013

Das ist exakt das, was wir wollen.

Nimm mal den Stock bitte so: stock Speichern(playerid){
print("1");
if(Spieler[playerid][eingeloggt] == 1)
{
print("2");
new query[250];
format(query,sizeof(query),"UPDATE accounts SET level='%i',admin='%i',pfandflaschen='%i',rucksack='%i' WHERE Username='%s'",GetPlayerScore(playerid),Spieler[playerid][admin],Spieler[playerid][pfandflaschen],Spieler[playerid][rucksack],Name(playerid));
print("3");
mysql_function_query(dbhandle,query,false,"","");
print(query);
} else print("Nicht eingeloggt");
return 1;
} Und guck dann mal, wie weit er kommt

Edit: @Andrzejxy':