[SAMMELTHREAD] Kleine Scripting Fragen

Zitat von Tomsen

Wie bekomm ich das weg?

Spoiler anzeigen

Spoiler anzeigen

Code

local variable "hunger" shadows a variable at a preceding level

Spoiler anzeigen

Spoiler anzeigen

ocmd:sethunger(playerid,params[])
{
new hunger,pID,string1[256],string2[256]; //betroffene Zeile
if(!IsPlayerAnAdmin(playerid,200)) return SendClientMessage(playerid,rot,"Du hast nicht das nötige Adminlevel!");
if(sscanf(params,"ud",pID,hunger)) return SendClientMessage(playerid,rot,"Benutze: /sethunger [Name/ID] [0-100]");
if(hunger<=0 || hunger>=100) return SendClientMessage(playerid,rot,"[Fehler]: Der Wert darf nicht weniger als 0 und nicht mehr als 100 betragen!");
format(string1,sizeof(string1),"Dein 'Hunger' wurde von Admin %s auf %d gesetzt.",SpielerName(playerid),hunger);
format(string2,sizeof(string2),"Du hast den 'Hunger' von %s auf %d gesetzt.",SpielerName(pID),hunger);
SendClientMessage(pID,grau,string1);
SendClientMessage(playerid,grau,string2);
SpielerInfo[pID][pHunger] = hunger;
ShowBedu(pID);
return 1;
}

- Tomsen

Alles anzeigen

Globale Variable mit der Bezeichnung "hunger" vorhanden? Könnte auch ein Public, Array o.ä. sein.

Der Compiler sagt dir ja, dass es diese Variable schon irgendwo gibt, da du in deiner Funktion die Variable "hunger" nicht 2 mal deklariert hast, müsste sie auf jeden Fall global irgendwo sein.

Ich bekomme hier Fehler weiß aber nicht wodran es liegt

Bread.pwn(2858) : error 029: invalid expression, assumed zero

1 Error.

dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA]== 1)
{
{
for(new i=0;i<5;i++)
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
}
else
{
for(new i=0;i<5;i++)
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
return 1;
}

}
return 1;
}

Die zeile ist beim "else"

Zitat von EliteBread

Ich bekomme hier Fehler weiß aber nicht wodran es liegt

Bread.pwn(2858) : error 029: invalid expression, assumed zero

1 Error.

dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA]== 1)
{
{
for(new i=0;i<5;i++)
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
}
else
{
for(new i=0;i<5;i++)
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
return 1;
}

}
return 1;
}

Die zeile ist beim "else"

Alles anzeigen

Wo ist die if-Abfrage geblieben?

Probier mal folgendes:
dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA]== 1)
{
for(new i=0;i<5;i++)
´ {
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
}
}
else
{
for(new i=0;i<5;i++)
´ {
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
}
return 1;
}
return 1;
}

hallo,

mein problem ist ich möchte dem gekillten in einem PVarInt +1 gegeben.
Hier Code:

SetPVarInt(playerid,"deads","+1");

Fehler:

C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(79) : error 035: argument type mismatch (argument 3)
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

UPPPS sehe ich jetzt erst
elyday
public OnPlayerDeath(playerid, killerid, reason)
{
if(IsPlayerConnected(killerid) && killerid != playerid)
{
SendClientMessage(killerid,FARBE, "Du hast ein Verbrechen begangen");
SetPVarInt(playerid,"kills","+1");
SendClientMessage(playerid,FARBE,"Du bist Gestorben");
SetPVarInt(playerid,"deads","+1");
}
return 1;
}

Zitat von elyday

hallo,

mein problem ist ich möchte dem gekillten in einem PVarInt +1 gegeben.
Hier Code:

SetPVarInt(playerid,"deads","+1");

Fehler:

C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(79) : error 035: argument type mismatch (argument 3)
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Alles anzeigen

SetPVarInt(playerid,"deads",GetPVarInt(playerid,"deads") + 1);

Du musst auch erstmal die aktuelle Anzahl der Tode auslesen, dann kannst du anschließend etwas dazu addieren.

danke an beide.

//Edit:
Macht das so sinn?

if(GetPVarInt(playerid,"World") == 1)
{
SetPVarInt(playerid,"deads",GetPVarInt(playerid,"deads") + 1);
SendClientMessage(playerid,ROT,"Du bist gestorben.");
}
else if(GetPVarInt(killerid,"World") == 1)
{
SetPVarInt(killerid,"kills",GetPVarInt(killerid,"kills") + 1);
SendClientMessage(playerid,BLAU,"Du hast jemanden getötet.");
}

Ob das Sinn macht? Keine Ahnung, was stellt denn "World" überhaupt da? GetPlayerVirtualWorld?

Ansonsten müsste das funktionieren. Sofern natürlich die if-Abfragen eintreffen, was ich nicht beurteilen kann.

Zitat von Mostey

Wo ist die if-Abfrage geblieben?

Probier mal folgendes:
dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA]== 1)
{
for(new i=0;i<5;i++)
´ {
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
}
}
else
{
for(new i=0;i<5;i++)
´ {
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
}
return 1;
}
return 1;
}

Alles anzeigen

__________________________________________________________________________________________________
So muss das
bei deinem Kamen Errors
dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA]== 1) return SendClientMessage(playerid,Rot, "Du hast kein PDA");
{
for(new i=0;i<5;i++)
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
}
else//Zeile2857
{
for(new i=0;i<5;i++)
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
return 1;
}
return 1; //Zeile 2863
}

jedoch bekomme ich wieder Errors
diese Errors:
Bread.pwn(2857) : error 029: invalid expression, assumed zero
Bread.pwn(2863) : warning 225: unreachable code
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

@Monstey:
Die PVar's werden in einem Dialog gesetzt. 1 = DM 2 = Stunt 3 = Gungame deswegen die Abfrage.

Zitat von elyday

@Monstey:
Die PVar's werden in einem Dialog gesetzt. 1 = DM 2 = Stunt 3 = Gungame deswegen die Abfrage.

Dann sollte es funktionieren, probier's doch mal aus?

bin grad dabei das ganze am verfoständigen.
Nun errors:

C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(339) : error 037: invalid string (possibly non-terminated string)
C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(339) : warning 215: expression has no effect
C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(339) : error 001: expected token: ";", but found ")"
C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(339) : error 029: invalid expression, assumed zero
C:\Users\User\Desktop\MCAnims\gamemodes\selfmade.pwn(339) : fatal error 107: too many error messages on one line

Compilation aborted.Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

4 Errors.

//Edit: Fehler gesehen

Code:

ShowPlayerDialog(playerid,DIALOG_AUSWAHL,DIALOG_STYLE_LIST,"Auswahl","TDM Welt\nStunt Welt (In Bau!)\nGun Game (In Bau!)","Okay,"");

Du hast hinten 2 "" mach das mal so
ShowPlayerDialog(playerid,DIALOG_AUSWAHL,DIALOG_STYLE_LIST,"Auswahl","TDM Welt\nStunt Welt (In Bau!)\nGun Game (In Bau!)","Okay,");

PS: ^Mein Problem weiter oben ist auch noch nicht gelöst^

Zitat von EliteBread

PS: ^Mein Problem weiter oben ist auch noch nicht gelöst^

Teste mal das hier
dcmd_showpda(playerid,params[])
{
#pragma unused params
if(PlayerInfo[playerid][PDA] != 1)
{
for(new i=0;i<5;i++)
TextDrawHideForPlayer(playerid, StatsDraw[playerid][i]);
return SendClientMessage(playerid,Rot, "Du hast kein PDA");
}
else
{
for(new i=0;i<5;i++)
TextDrawShowForPlayer(playerid, StatsDraw[playerid][i]);
}
return 1;
}
Ich vermute mal wenn PDA 0 ist das er keins hat und dein Code ergab sowiso kein sinn
Die einrückung nach der for schleife so lassen, sonst kommen die Fehler wieder

Ist jetzt keine Kritik, aber ich frag mich manchmal ob manche hier gegen Klammern allergisch sind

Ne ma im Ernst: bringt das was die wegzulassen (Abgesehen von dem einen Byte den man da spart?)

Zitat von Hagi

bringt das was die wegzulassen

Das spart enorm an Zeilen
Bein Mir im Script habe ich allein knappe 3k Zeilen gespart daher mach ich das so

Wie erstellt man einen Zufalls spawn von 4 Punkten?

Kann mir jemand diese beiden Farben in HTML code umwandeln ?!

Rot 0xC20000FF
Grün 0x33AA33FF

Zitat von elyday

Wie erstellt man einen Zufalls spawn von 4 Punkten?

enum ispawns
{
Float:x,
Float:y,
Float:z,
Float:rot,
}
new spawns[4][ispawns] =
{
{x,y,z,rot},
{x,y,z,rot},
{x,y,z,rot},
{x,y,z,rot}
};

new rand = random(sizeof(spawns));

Dann nimmste rand als Index für die erste Zelle des Arrays spawns

Ok, habs nun anders gelöst aber trozdem danke.