[SAMMELTHREAD] Kleine Scripting Fragen

Heya,
habe mir vor ca. einer Stunde einen kleinen /ammo CMD erstellt, der die momentan ausgewählte Waffe mit so oder so viel Patronen füllen soll, wobei der Spieler aber 500$ abdrücken muss.
Alles klappt, kein Error, jedoch ist es ingame nur Schrott - es zeigt mir bei JEDER Waffe den Text für die Melee-Waffen an.
if (strcmp("/ammo", cmdtext, true, 10) == 0)
{
new pMoney, pWeapon;
pMoney = GetPlayerMoney(playerid);
pWeapon = GetPlayerWeapon(playerid);

if(pMoney == 500 || pMoney > 500){
if (GetPlayerWeapon(playerid) == 0 || 1 || 2 || 3 || 4 || 5 || 6 || 7 || 8 || 9 || 10 || 11 || 12 || 13 || 14 || 15){
SendClientMessage(playerid,COLOR_RED,"You can't buy ammo for a melee weapon!");
}else if(GetPlayerWeapon(playerid) == 16 || 17 || 18){
GivePlayerWeapon(playerid,pWeapon,5);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 5 peaces of your current weapon!");
}else if(GetPlayerWeapon(playerid) == 22 || 23){
GivePlayerWeapon(playerid,pWeapon,75);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 75 cartridges for your pistol!");
}else if(GetPlayerWeapon(playerid) == 24){
GivePlayerWeapon(playerid,pWeapon,20);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 20 cartridges for your deagle!");
}else if(GetPlayerWeapon(playerid) == 25 || 26 || 27){
GivePlayerWeapon(playerid,pWeapon,20);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 20 cartridges for your shotgun!");
}else if(GetPlayerWeapon(playerid) == 28 || 29 || 32){
GivePlayerWeapon(playerid,pWeapon,85);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 285 cartridges for your SMG!");
}else if(GetPlayerWeapon(playerid) == 30 || 31){
GivePlayerWeapon(playerid,pWeapon,65);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 65 cartridges for your AK-47 / M4!");
}else{
SendClientMessage(playerid,COLOR_RED,"The Administrator disabled /ammo for this weapon!");
}
}
return 1;
}

Ich habe mal stark die Vermutung, dass das ganze nicht funktioniert, da pWeapon nicht richtig geupdated wird.(Ja, ich weiß, dass man die new's direkt zuweißen kann, jedoch habe ich etwas ausprobiert. )

Wenn du mehrere sachen abfragen willst, müsstest du es so machen:
if (GetPlayerWeapon(playerid) == 0 || GetPlayerWeapon(playerid) == 1 || GetPlayerWeapon(playerid) == 2 || GetPlayerWeapon(playerid) == 3 //usw...
Bei deinem vorhaben würde sich die verwendung von switch mehr lohnen.
Ausserdem kannst du
if(pMoney == 500 || pMoney > 500){
direkt mit einer größer-gleich (>=) abfrage lösen.
if (strcmp("/ammo", cmdtext, true, 10) == 0)
{
new
pMoney = GetPlayerMoney(playerid),
pWeapon = GetPlayerWeapon(playerid)
;
if(pMoney >= 500)
{
switch(pWeapon)
{
case 0 .. 15:
{
SendClientMessage(playerid,COLOR_RED,"You can't buy ammo for a melee weapon!");
}
case 16,17,18:
{
GivePlayerWeapon(playerid,pWeapon,5);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 5 peaces of your current weapon!");
}
case 22,23:
{
GivePlayerWeapon(playerid,pWeapon,75);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 75 cartridges for your pistol!");
}
case 24:
{
GivePlayerWeapon(playerid,pWeapon,20);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 20 cartridges for your deagle!");
}
case 25,26,27:
{
GivePlayerWeapon(playerid,pWeapon,20);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 20 cartridges for your shotgun!");
}
case 28,29,32:
{
GivePlayerWeapon(playerid,pWeapon,85);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 285 cartridges for your SMG!");
}
case 30,31:
{
GivePlayerWeapon(playerid,pWeapon,65);
SendClientMessage(playerid,COLOR_GREEN,"You successfully bought 65 cartridges for your AK-47 / M4!");
}
default:
{
SendClientMessage(playerid,COLOR_RED,"The Administrator disabled /ammo for this weapon!");
}
}
}
return 1;
}

Das ist übrigens genug Problem für ein eigenes Topic

Ganz ganz schlimm ist die leserlichkeit hierbei - dafür gibt es eine Hilfe, die nennt sich "switch case"

Daraus wird aus folgender Zeile:

if (GetPlayerWeapon(playerid) == 0 || 1 || 2 || 3 || 4 || 5 || 6 || 7 || 8 || 9 || 10 || 11 || 12 || 13 || 14 || 15)
lediglich:

case 0..14:

Ich war jetzt zu faul das alles umzuschreiben, vielleichst machst du es ja selber

Das ganze müsste in etwa so aussehen, mehr zu switch/case findest du in der englischen Wiki oder hier in irgendwelchen Tutorials

switch(GetPlayerWeapon(playerid))
{
case 0..14:
{
SendClientMessage(playerid,COLOR_RED,"You can't buy ammo for a melee weapon!");
}

}

Das Hauptproblem allerdings war ja was anderes, das hattest du auch relativ gut erkannt.
Das Problem ist folgendes: GetPlayerWeapon gibt lediglich die aktuelle Waffe des Spielers zurück, die er auch in der Hand hat. Am leichtesten wäre daher (sofern du weißt, dass der Spieler aus jedem Slot eine Waffe besitzt) der Einsatz von SetPlayerAmmo, ansonsten musst du eine Spur komplizierter rangehen - GetPlayerWeaponData ist da die Funktion die du brauchst.

Probier erstmal selbst weiter, erst wenns wieder irgendwo hakt nachfragen.

EDIT: Uh, der Maddin war im Sprinttemp

Huch, Maddin hat ja richtig Gas gegeben
Argh, ich wusste das es "switch" auch in PAWN gibt!
Aber mir viel das ganze erst ein, als ich fertig mit dem Ganzen war ... aber das mit dem ".." war mir neu, kewl. :3
Vielen dank, funktioniert jetzt.

/edit ( Um doppelpost zu vermeiden, w00t )
Wie sieht es eigendlich nochmal mit boundary ... walls (?) aus?
Wie hieß das nochmal, mensch ... eh, denn ich brauche eine, um die Spieler in einer Zone zu halten.
Klar, positionen braucht man, mal angenommen ich hätte diese schon, was muss ich danach machen?

Was du meinst geht per SetPlayerWorldBounds.

Ah, genau, dass war es.
Humm, x-min und x-max wäre also zb. links oben, rechts oben, y-min und y-max also dann rechts unte, links unten, richtig?

edit: Erledigt^^

OT: Beitrag zu kurz

hiho, ich frage mich gerade wie man des ShowStats übersichtlich machen kann, dort habe ich mal folgendes gemalt:

nun meine frage die leerzeichen ist das alles abgezählt bzw muss man das x fach ausprobieren bis es passt pro zeile?
und wie könnte man die Stats im script übersichtlich machen formats in 3 zeilen dann noch beliebige leerzeichen dann ists schon nichmehr übersichtlch D:

versuchs ma mit tabs ist glaub ich \t
also
format(string,sizeod(string),"blaaaa \tbaaaa");
so in etwa

Hey

wisst ihr wo der Fehler ist?

public IsPlayerInGangZone(playerid)
{
for(new t = 1; t < MAX_GEBIETE; t++)
{
new string[75];
format(string, sizeof(string), "SAP/GangSystem/Zonen/%d.gebiet", t);
if(fexist(string))
{
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid, x, y, z);
if ((x > Gebiete[t][gminX] && x < Gebiete[t][gmaxX]) && (y > Gebiete[t][gminY] && y < Gebiete[t][gmaxY])){return 1;}
else{return 0;}
}
}}

Zeile (4362) : warning 209: function "IsPlayerInGangZone" should return a value

ich finde den wirklich nicht :o

MFG: Apoolecu

Einfach nen return 1; ans Ende des publics setzen.

Trotzdem hat die funktion glaube ich einen kleinen denkfehler. Wegen dem else return wirst du nie weiter als bis zum ersetn durchlauf der schleife kommen...
Und strings innerhalb von schleifen zu erstellen ist auch nicht empfehlenswert.

public IsPlayerInGangZone(playerid)
{
for(new t = 1,string[75]; t < MAX_GEBIETE; t++)
{
format(string, sizeof(string), "SAP/GangSystem/Zonen/%d.gebiet", t);
if(fexist(string))
{
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid, x, y, z);
if ((x > Gebiete[t][gminX] && x < Gebiete[t][gmaxX]) && (y > Gebiete[t][gminY] && y < Gebiete[t][gmaxY])) return 1;
}
}
return 0;
}

Hey habe den nächsten Fehler und zwar wird dies mal der Dialog irgendwie nicht erstellt, Definiert ist er oben im Script

if(dialogid == DIALOG_GGangC)
{
if(response == 0)
{}
if(response == 1)
{
if(listitem == 0) // Grün
{
Gang[Gangerstellt][gColor] = 0x339933FF;
ShowPlayerDialog(playerid,DIALOG_GGangW1,DIALOG_STYLE_LIST,"Gang System: Gang erstellen","Schlagring\nGolfschläger\nSchlagstock\nMesser\nBaseballschläger\nSchaufel\nBrechstange\nKantana","Weiter","");
}
if(listitem == 1) // Rot
{
Gang[Gangerstellt][gColor] = 0xCE000000;
ShowPlayerDialog(playerid,DIALOG_GGangW1,DIALOG_STYLE_LIST,"Gang System: Gang erstellen","Schlagring\nGolfschläger\nSchlagstock\nMesser\nBaseballschläger\nSchaufel\nBrechstange\nKantana","Weiter","");
}
if(listitem == 2) // Blau
{
Gang[Gangerstellt][gColor] = 0x0000F600;
ShowPlayerDialog(playerid,DIALOG_GGangW1,DIALOG_STYLE_LIST,"Gang System: Gang erstellen","Schlagring\nGolfschläger\nSchlagstock\nMesser\nBaseballschläger\nSchaufel\nBrechstange\nKantana","Weiter","");
}
if(listitem == 3) // Schwarz
{
Gang[Gangerstellt][gColor] = 0x000000;
ShowPlayerDialog(playerid,DIALOG_GGangW1,DIALOG_STYLE_LIST,"Gang System: Gang erstellen","Schlagring\nGolfschläger\nSchlagstock\nMesser\nBaseballschläger\nSchaufel\nBrechstange\nKantana","Weiter","");
}
if(listitem == 4) // Gelb
{
Gang[Gangerstellt][gColor] = 0xFFDE00FF;
ShowPlayerDialog(playerid,DIALOG_GGangW1,DIALOG_STYLE_LIST,"Gang System: Gang erstellen","Schlagring\nGolfschläger\nSchlagstock\nMesser\nBaseballschläger\nSchaufel\nBrechstange\nKantana","Weiter","");
}
}
}

maddin also die Funktion funktioniert jetzt schon

EDIT: Erledigt

Humm,
'tschuldige wenn ich dich einmal unterbrechen darf, apoolecu, aber ich komm einfach nicht weiter.
in BlackFoX' Tutorial , diesem hier, erwähnt erselbst, wie man Variablen wie Geld o.ä speichert.

Zitat von BlackFoX_UD_

Sollte eig. zu sehen sein wie man das Speichert & wie nicht

GivePlayerMoney(playerid,dini_Int(accFormat,"Geld"));
SetPlayerScore(playerid,dini_Int(accFormat,"Score"));

Und Speichern
dini_IntSet(accFormat,"Geld",GetPlayerMoney(playerid));
dini_IntSet(accFormat,"Score",GetPlayerScore(playerid));

Alles anzeigen

Ich wollte die Spieler positionen speichern und aufrufen, jedoch wurde mir etwas klar: Geht nicht so ich es mir vorgestellt hatte.
Im Grunde müssen die coord's einzeln gespeichert werden, x, y und z.
Mit GetPlayerPos würde das einfach gehen, wenn man nicht immer sowohl x wie auch y und z angeben müsste
Im Grunde wäre es dann so;
dini_FloatSet(accFormat,"lastx",GetPlayerPos(playerid, lastx));
dini_FloatSet(accFormat,"lasty",GetPlayerPos(playerid, lasty));
dini_FloatSet(accFormat,"lastz",GetPlayerPos(playerid, lastz));
Geht ja leider nicht.
Meine Frage nun; wie sonst? :o

new Float:lastx, new Float:lasty, new Float:lastz;
GetPlayerPos(playerid,lastx,lasty,lastz);
dini_FloatSet(accFormat,"lastx",lastx));
dini_FloatSet(accFormat,"lasty",lasty));
dini_FloatSet(accFormat,"lastz",lastz));

Herp, wie gesagt, so einfach geht das scheinbar nicht

Zitat

D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(97) : warning 213: tag mismatch
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(97) : error 001: expected token: ";", but found ")"
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(97) : error 029: invalid expression, assumed zero
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(97) : warning 215: expression has no effect
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(98) : warning 213: tag mismatch
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(98) : error 001: expected token: ";", but found ")"
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(98) : error 029: invalid expression, assumed zero
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(98) : warning 215: expression has no effect
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(99) : warning 213: tag mismatch
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(99) : error 001: expected token: ";", but found ")"
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(99) : error 029: invalid expression, assumed zero
D:\Programme\SA-MP Server\gamemodes\Gamemode.pwn(99) : warning 215: expression has no effect
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

6 Errors.

Alles anzeigen

public OnPlayerDisconnect(playerid, reason)
{
new iPlayer[MAX_PLAYER_NAME];
new String[100];
GetPlayerName(playerid, iPlayer, sizeof(iPlayer));
format(String, sizeof(String), "[SERVER] %s left the Server!", iPlayer);
SendClientMessageToAll(COLOR_RED, String);

new pName[SLOTS],accFormat[128];
GetPlayerName(playerid,pName,sizeof pName);
format(accFormat,sizeof accFormat,"%s.p",pName);
if(fexist(accFormat) && Player[playerid][logged_in])
{
dini_IntSet(accFormat,"rank",GetPlayerWantedLevel(playerid));
dini_IntSet(accFormat,"score",GetPlayerScore(playerid));
dini_IntSet(accFormat,"money",GetPlayerMoney(playerid));
dini_IntSet(accFormat,"skin",GetPlayerSkin(playerid));
dini_FloatSet(accFormat,"lastx",lastx)); // <---------------------- Zeile 97
dini_FloatSet(accFormat,"lasty",lasty));
dini_FloatSet(accFormat,"lastz",lastz));
}else{
Player[playerid][logged_in] = 0;
Player[playerid][rank] = 0;
Player[playerid][score] = 0;
Player[playerid][money] = 0;
Player[playerid][skin] = 0;
Player[playerid][lastx] = (0.0);
Player[playerid][lasty] = (0.0);
Player[playerid][lastz] = (0.0);
}
return 1;
}

/edit:
Natürlich noch enum;
enum pData
{
rank,
score,
money,
skin,
Float:lastx,Float:lasty,Float:lastz,
logged_in
}

könnte jemand per Teamviewer helfen weil ich pack mein script nicht Hochzuladen.

Bitte hilft mir.

Mfg

Kommt erstmal darauf an, über was du hochlädst, welches Programm du verwendest usw.
Hast du alle FTP daten die du brauchst?
Benutzt du überhaupt ein FTP Programm, oder hast du gar keins?
Diese ganzen Fragen spielen einen hohen Faktor. *trollface.jpg*

/edit:
hab jetzt auch meins herrausgefunden:
es waren 3 klammern zuviel bei BloodyEric's code.
Trotzdem danke Eric, wäre alleine nicht auf die Idee gekommen das Ganze auf diese Weiße zu speichern.

ich weis was du meinst xD ich hab des zum ersten mal könntest du mir damit helfen per teamviewer wäre au nett

Was musste man nochmal tun, damit skins die nicht CJ sind wie CJ laufen?