Crash Befehl Problem

Mike_Smith · 14. Januar 2012, 22:17

Hallo,

ich hab in meinem Selfmade ein Befehl womit man das Spiel von anderen Leuten crashen kann.
Da gibt es leider ein paar Errors.

[pwn]Zeile (2606) : Falscher Variablentyp als Parameter (Parameter #1)
Zeile (2610) : error 006: must be assigned to an array
Zeile (2611) : Falscher Variablentyp als Parameter (Parameter #1)
Zeile (2616) : error 012: invalid function call, not a valid address
Zeile (2616) : Befehl hat keinen Sinn
Zeile (2616) : Erwartete Zeichen ";", aber gefunden ")"
Zeile (2616) : Ungültiger Befehl
Zeile (2616) : Zuviele Fehler in einer Zeile

8 Errors, 1 Warnungen[/pwn]
[pwn]

new giveplayerid;
new idx;
new ReturnUser;
new cmd;[/pwn]

[pwn] if(strcmp(cmd, "/crash", true) == 0) //2606
{
if(IsPlayerConnected(playerid))
{
tmp = strtok(cmdtext, idx); //2610
if(!strlen(tmp)) //2611
{
SendClientMessage(playerid, Gelb, "Benutze: /crash [playerid/PartOfName]");
return 1;
}
giveplayerid = ReturnUser(tmp); //2616
GetPlayerName(giveplayerid, giveplayer, sizeof(giveplayer));
if (PlayerInfo[playerid][pAdmin] >= 5)
{
if(IsPlayerConnected(giveplayerid))
{
if(giveplayerid != INVALID_PLAYER_ID)
{
if(giveplayerid != playerid && Untouchable[giveplayerid]) return SendClientMessage(playerid, COLOR_GREY, "* Du kannst es an diesem Spieler nicht benutzen!");
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 1000, 0);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 2000, 1);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 3000, 2);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 4000, 3);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 5000, 4);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 6000, 5);
GameTextForPlayer(giveplayerid, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 7000, 6);
format(string, sizeof(string), "Du hast %s (%d)'s Spiel gecrasht", giveplayer,giveplayerid);
SendClientMessage(playerid,COLOR_DBLUE, string);
return 1;
}
else
{
format(string, sizeof(string), "** Unbekannter Spieler!", giveplayerid);
SendClientMessage(playerid, COLOR_GRAD1, string);
}
}
}
else
{
format(string, sizeof(string), "** Du hast nicht den nötigen Rang dazu, um diesen Befehl zu benutzen!", giveplayerid);
SendClientMessage(playerid, COLOR_GRAD1, string);
}
}
return 1;
}[/pwn]

Vincent Petritz · 14. Januar 2012, 22:31

Hallo Mike,
Sorry wenn ich dir so reinrassel aber ich hasse strcmp... ;)
Ich scripte dir das mal auf OCMD und damit ist es eigentlich richtig:
[pwn]ocmd:crash(playerid, params[])
{
new pID;
if(sscanf(params, "u", pID))
return SendClientMessage(playerid, Gelb, "/crash [playerid/PartOfName]");
if(IsPlayerConnected(pID) && pID != INVALID_PLAYER_ID)
{
if (PlayerInfo[playerid][pAdmin] >= 5)
{
if(pID != playerid && Untouchable[pID]) return SendClientMessage(playerid, COLOR_GREY, "* Du kannst es an diesem Spieler nicht benutzen!");
{
new Spieler[MAX_PLAYER_NAME], string[100];
GetPlayerName(pID, Spieler, sizeof(Spieler));
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 1000, 0);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 2000, 1);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 3000, 2);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 4000, 3);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 5000, 4);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 6000, 5);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 7000, 6);
format(string, sizeof(string), "Du hast %s (%d)'s Spiel gecrasht", Spieler, pID);
SendClientMessage(playerid, COLOR_DBLUE, string);
}
else {
SendClientMessage(playerid, COLOR_GRAD1, "Unbekannter Spieler!");
}
else {
SendClientMessage(playerid, COLOR_GRAD1, "Du hast nicht den nötigen Rang dazu, um diesen Befehl zu nutzen!");
}
}
return 1;
}[/pwn]

Damit geht es, ich hoffe ich konnte dir helfen.

Mike_Smith · 14. Januar 2012, 22:36

[pwn]Zeile (1544) : Unbekannte Variable : "PlayerInfo"
Zeile (1544) : Befehl hat keinen Sinn
Zeile (1544) : Erwartete Zeichen ";", aber gefunden "]"
Zeile (1544) : Ungültiger Befehl
Zeile (1544) : Zuviele Fehler in einer Zeile[/pwn]

Wen ich die Variable dazu mache: new PlayerInfo;

[pwn]Zeile (1545) : Ungültiges Array (kein Array?): "PlayerInfo"
Zeile (1545) : Befehl hat keinen Sinn
Zeile (1545) : Erwartete Zeichen ";", aber gefunden "]"
Zeile (1545) : Ungültiger Befehl
Zeile (1545) : Zuviele Fehler in einer Zeile[/pwn]

Vincent Petritz · 14. Januar 2012, 22:40

Hast du überhaupt ein Registrier- oder Loginsystem?
Beziehungsweise, zeig mir mal dein Enum.

Mike_Smith · 14. Januar 2012, 22:41

Mein Registrier/Loginsystem gehen nicht über Enum...
Ich mach so gut wie nichts über Enum.

Vincent Petritz · 14. Januar 2012, 22:44

Aber wie hast du dann "PlayerInfo" definiert? ;)

Mike_Smith · 14. Januar 2012, 22:46

[pwn]new PlayerInfo;[/pwn]

Vincent Petritz · 14. Januar 2012, 22:53

Also so, wie du das oben gemacht hast, ist ein Enum von Nöten ...
Sprich:
[pwn]enum Spieler {
pAdmin
};
new PlayerInfo[MAX_PLAYERS][Spieler];[/pwn]

Und dann geht's.

Mike_Smith · 14. Januar 2012, 22:54

Tut es immer noch nicht

[pwn]Zeile (1285) : Variable "Spieler" wurde mehrmals deklariert
Zeile (1288) : Falscher Variablentyp als Parameter (Parameter #1)
Zeile (1288) : Falscher Variablentyp als Parameter (Parameter #1)
Zeile (1289) : Falscher Variablentyp als Parameter (Parameter #1)
Zeile (1285) : Unbenutzte Variable: "Spieler"
Zeile (1549) : Unbekannte Variable : "Untouchable"
Zeile (1549) : Befehl hat keinen Sinn
Zeile (1549) : Erwartete Zeichen ";", aber gefunden "]"
Zeile (1549) : Ungültiger Befehl
Zeile (1549) : Zuviele Fehler in einer Zeile[/pwn]

Vincent Petritz · 14. Januar 2012, 23:02

Also ich erkenne bei dir kein System raus, was macht denn Untouchable?
Ist das ein Selfmade?

Mike_Smith · 14. Januar 2012, 23:07

Ja, ist ein Selfmade.

[pwn] if(pID != playerid && Untouchable[pID]) return SendClientMessage(playerid, COLOR_GREY, "* Du kannst es an diesem Spieler nicht benutzen!");[/pwn]

Vincent Petritz · 14. Januar 2012, 23:10

Nimm mal das:
"Untouchable[pID]" raus. ;)

Mike_Smith · 14. Januar 2012, 23:14

Jetzt sagt der mir 26 Errors...

Wo ist hier ein Klammerfehler?!?!

[pwn]ocmd:crash(playerid, params[])
{
new pID;
if(sscanf(params, "u", pID))
return SendClientMessage(playerid, Gelb, "/crash [playerid/PartOfName]");
if(IsPlayerConnected(pID) && pID != INVALID_PLAYER_ID)
{
if (PlayerInfo[playerid][pAdmin] >= 5)
{
if(pID != playerid &&) return SendClientMessage(playerid, Gelb, "* Du kannst es an diesem Spieler nicht benutzen!");
{
new Spieler[MAX_PLAYER_NAME], string[100];
GetPlayerName(pID, Spieler, sizeof(Spieler));
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 1000, 0);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 2000, 1);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 3000, 2);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 4000, 3);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 5000, 4);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 6000, 5);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 7000, 6);
format(string, sizeof(string), "Du hast %s (%d)'s Spiel gecrasht", Spieler, pID);
SendClientMessage(playerid, COLOR_DBLUE, string);
}
else {
SendClientMessage(playerid, COLOR_GRAD1, "Unbekannter Spieler!");
}
else {
SendClientMessage(playerid, COLOR_GRAD1, "Du hast nicht den nötigen Rang dazu, um diesen Befehl zu nutzen!");
}
}
}
return 1;
}[/pwn]

Vincent Petritz · 14. Januar 2012, 23:17

Da ist keiner ... eigenartig, woanders evtl.?

Mike_Smith · 14. Januar 2012, 23:19

Nö, hab ja nur da was verändert...

Simple · 15. Januar 2012, 00:25

Meines Erachtens ist da wohl ein Klammerfehler.

[pwn]ocmd:crash(playerid, params[])
{
new pID;
if(sscanf(params, "u", pID)) return SendClientMessage(playerid, Gelb, "/crash [playerid/PartOfName]");
if(IsPlayerConnected(pID) && pID != INVALID_PLAYER_ID)
{
if (PlayerInfo[playerid][pAdmin] >= 5)
{
if(pID != playerid &&) return SendClientMessage(playerid, Gelb, "* Du kannst es an diesem Spieler nicht benutzen!");
{
new Spieler[MAX_PLAYER_NAME], string[100];
GetPlayerName(pID, Spieler, sizeof(Spieler));
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 1000, 0);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 2000, 1);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 3000, 2);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 4000, 3);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 5000, 4);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 6000, 5);
GameTextForPlayer(pID, "•¤¶§!$$%&'()*+,-./01~!@#$^&*()_-+={[}]:;'<,>.?/", 7000, 6);
format(string, sizeof(string), "Du hast %s (%d)'s Spiel gecrasht", Spieler, pID);
SendClientMessage(playerid, COLOR_DBLUE, string);
}
else
{
SendClientMessage(playerid, COLOR_GRAD1, "Unbekannter Spieler!");
}
}
else
{
SendClientMessage(playerid, COLOR_GRAD1, "Du hast nicht den nötigen Rang dazu, um diesen Befehl zu nutzen!");
}
}
return 1;
}[/pwn]

Maddin · 15. Januar 2012, 14:11

[pwn]if(pID != playerid &&)[/pwn]Macht keinen sinn, wegen den && am ende^^
Ich würds so machen:
[pwn]ocmd:crash(playerid, params[])
{
if(PlayerInfo[playerid][pAdmin] < 5) return SendClientMessage(playerid, COLOR_GRAD1, "Du hast nicht den nötigen Rang dazu, um diesen Befehl zu nutzen!");
new pID;
if(sscanf(params, "u",pID)) return SendClientMessage(playerid, Gelb, "/crash [playerid/PartOfName]");
if(!IsPlayerConnected(pID)

pID == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_GRAD1, "Unbekannter Spieler!");
ApplyAnimation(pID, "GANG", "DRUGS_BUY", 10, 0, 0, 0, 0, 5*1000);
new Spieler[MAX_PLAYER_NAME], string[100];
GetPlayerName(pID, Spieler, sizeof(Spieler));
format(string, sizeof(string), "Du hast %s (%d)'s Spiel gecrasht", Spieler, pID);
SendClientMessage(playerid, COLOR_DBLUE, string);
return 1;
}[/pwn]

Simple · 15. Januar 2012, 14:14

Jup, hatte nur den Klammerfehler berichtigt, auf anderes habe ich nicht geachtet, Sorry.

iPsy · 16. Januar 2012, 18:06

An deiner Stelle würde ich dennoch enum benutzen, da es Resourcen sparender ist. Und übersichtlich und angenehme Verwendung.

Maddin · 16. Januar 2012, 19:56

iPsy schrieb:

An deiner Stelle würde ich dennoch enum benutzen, da es Resourcen sparender ist. Und übersichtlich und angenehme Verwendung.

Das ist so nicht richtig. Im gegensatz zu "normalen" Variablen werden PVars gelöscht wenn der Spieler den Server verlässt und geben somit den speicher wieder frei.

Teilen