Einfacher Scripten (Ressourcenschonender)

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

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

@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

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

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

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

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

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

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 )

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...

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

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

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 ...

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:

SHOW CREATE TABLE `accounts`;

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

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':