Makeadmin funktioniert nicht, warum?

H4lo · 17. Mai 2012

Tag liebe Breadfish Community,
ich habe ein Scriptfehler und ich weiß nicht wo der Fehler liegt.
Ich habe keine Errors oder Warnings:
Ich möchte auch (bitte) keine OCMD Command als Vorschläge

So sieht mein Befehl aus:

Zitat

if(strcmp(cmdtext, "/makeadmin", true) == 0)
{
if(IsPlayerAnAdmin(playerid,6))
{
if(!strlen(tmp))
{
SendClientMessage(playerid, GRAU, "Benutze /makeadmin [SpielerID] [Level]");
return 1;
}
tmp = strtok(cmdtext, idx);
new level = strval(tmp);
new Spieler = ReturnUser(tmp);
new Spielerdatei[64];
new name[MAX_PLAYER_NAME];
new giveplayer[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
GetPlayerName(Spieler, giveplayer, sizeof(giveplayer));
format(Spielerdatei,sizeof(Spielerdatei),"/Account's/%s.txt",giveplayer);
dini_IntSet(Spielerdatei,"Adminlevel",level);
new string[124];
format(string,sizeof(string),"Du hast %d zu einem Admin des Levels %s befördert.",giveplayer, level);
SendClientMessage(playerid,ROT,string);
new string2[124];
format(string2,sizeof(string2),"Du wurdest von %d zu einem Admin des Levels %s befördert.",name, level);
SendClientMessage(Spieler,ROT,string2);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Diesen Befehl darfst du nicht benutzen!");
return 1;
}
}

Alles anzeigen

Ich hoffe ihr könnt mir helfen.

Mit freundlichen Grüßen
iMan

Kaliber · 17. Mai 2012

Meines Wissens muss das so aussehen:

if(strcmp(cmdtext, "/makeadmin", true) == 0)
{
if(IsPlayerAnAdmin(playerid,6))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, GRAU, "Benutze /makeadmin [SpielerID] [Level]");
return 1;
}
new level = strval(tmp);
tmp = strtok(cmdtext, idx);
new Spieler = ReturnUser(tmp);

mfg.

H4lo · 17. Mai 2012

Jetzt habe ich 2 Warnings

Zitat

C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(252) : warning 204: symbol is assigned a value that is never used: "Spieler"
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(250) : warning 204: symbol is assigned a value that is never used: "level"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

2 Warnings.

Kevin. · 17. Mai 2012

if(strcmp(cmdtext, "/makeadmin", true) == 0){
if(IsPlayerAnAdmin(playerid,6))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, GRAU, "Benutze /makeadmin [SpielerID] [Level]");
return 1;
}
tmp = strtok(cmdtext, idx);

H4lo · 17. Mai 2012

Das funktioniert auch nicht.
Ich habe schon wieder warnings, die aber nicht im Script stehen.

Zitat

C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(550) : warning 203: symbol is never used: "IsNumeric"
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(550) : warning 203: symbol is never used: "ReturnUser"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

2 Warnings.

Hat jemand noch mehr Ideen??

Kaliber · 17. Mai 2012

Du solltest mit meinem Command nur deinen oberen Teil ersetzen

mfg.

H4lo · 17. Mai 2012

Wenn ich nur makeadmin eingebe kommt da :
Du hast ID 0 zum Admin befördert.

Aber wenn ich /makeadmin 0 6 eingebe, kommt da, der Befehl existiert nicht.
Kannst du mir da weiterhelfen??

Buschmensch · 17. Mai 2012

Nutze einfach zcmd + Sscanf, so klappt es auf jeden Fall.

H4lo · 17. Mai 2012

Ich bin Anfänger Scripter, deshalb kann ich noch nicht mit Sscanf gut scripten bzw. garnicht.

Buschmensch · 17. Mai 2012

COMMAND:makeadmin(playerid,params[])
{
new otherid,string[128],rank;
new aname[MAX_PLAYER_NAME],otheridname[MAX_PLAYER_NAME];
GetPlayerName(playerid,aname,sizeof(aname));
GetPlayerName(playerid,otheridname,sizeof(otheridname));
if(IsPlayerAnAdmin(playerid,6))
{
if(sscanf(params,"ud",otherid,rank))return SendClientMessage(playerid,GRAU,"Benutzung: /makeadmin [ID] [Rank]");
format(string,sizeof(string),"Du hast %s zu einem Admin des Levels %d befördert.",otheridname,rank);
SendClientMessage(playerid,ROT,string);
format(string,sizeof(string),"Du wurdest von %s zu einem Admin des Levels %d befördert.",aname,rank);
SendClientMessage(otherid,ROT,string);
pInfo[otherid][pAdmin] = rank;// anpassen!
SpielerSpeichern(otherid);// anpassen!
}
return 1;
}

H4lo · 17. Mai 2012

Soll das unter OnPlayerCommandText??

Montero · 17. Mai 2012

if(strcmp(cmd, "/makeadmin", true) == 0)
{
if(PlayerInfo[playerid][pAdmin] == !!!!! || IsPlayerAdmin(playerid)) // Bei !!!! muss der Adminrang den derjenige der jmd zum admin macht sein.
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_GRAD2, "Benutze: /makeadmin [ID] [Level]");
return 1;
}
new para1 = ReturnUser(tmp);
tmp = strtok(cmdtext, idx);
new level = strval(tmp);
if(IsPlayerConnected(para1))
{
if(para1 != INVALID_PLAYER_ID)
{
GetPlayerName(para1, giveplayer, sizeof(giveplayer));
GetPlayerName(playerid, sendername, sizeof(sendername));
if(PlayerInfo[para1][pAdmin] < level)
{
format(string, sizeof(string), "Du wurdest durch %s zu einem Level %d Admin befördert", sendername, level);
SendClientMessage(para1, COLOR_LIGHTBLUE, string);
format(string, sizeof(string), "Du hast %s zu einem Level %d Admin befördert.", giveplayer,level);
SendClientMessage(playerid, COLOR_LIGHTBLUE, string);
}
else
{
format(string, sizeof(string), "Du wurdest durch %s zu einem Level %d Admin degradiert", sendername, level);
SendClientMessage(para1, COLOR_LIGHTBLUE, string);
format(string, sizeof(string), "Du hast %s zu einem Level %d Admin degradiert.", giveplayer,level);
SendClientMessage(playerid, COLOR_LIGHTBLUE, string);
}
PlayerInfo[para1][pAdmin] = level;
}
}
}
return 1;
}

H4lo · 17. Mai 2012

Habe jetzt 4 Errors. (habe einige behoben^^)

Zitat

C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 029: invalid expression, assumed zero
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : warning 215: expression has no effect
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : warning 215: expression has no effect
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 001: expected token: ";", but found ")"
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 029: invalid expression, assumed zero
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : 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.

Alles anzeigen

Buschmensch · 17. Mai 2012

nein der COMMAND soll ganz unten ins Script..
Aber du brauchst natürlich die Include's zcmd und sscanf2.

H4lo · 17. Mai 2012

Sry aber ich möchte noch nicht mit Sccanf und zcmd/ocmd arbeiten
Hat sonst keiner ein Vorschlag?
Vllt kennt ihr ja den Fehler hier für:

Zitat

Wenn ich nur makeadmin eingebe kommt da :
Du hast ID 0 zum Admin befördert.

Aber wenn ich /makeadmin 0 6 eingebe, kommt da, der Befehl existiert nicht.
Kannst du mir da weiterhelfen??

Zitat

if(strcmp(cmdtext, "/makeadmin", true) == 0)
{
if(IsPlayerAnAdmin(playerid,6))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, GRAU, "Benutze /makeadmin [SpielerID] [Level]");
return 1;
}
new level = strval(tmp);
tmp = strtok(cmdtext, idx);
new Spieler = ReturnUser(tmp);
new Spielerdatei[64];
new name[MAX_PLAYER_NAME];
new giveplayer[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
GetPlayerName(Spieler, giveplayer, sizeof(giveplayer));
format(Spielerdatei,sizeof(Spielerdatei),"/Account's/%s.txt",giveplayer);
dini_IntSet(Spielerdatei,"Adminlevel",level);
new string[124];
format(string,sizeof(string),"Du hast %d zu einem Admin des Levels %s befördert.",giveplayer, level);
SendClientMessage(playerid,ROT,string);
new string2[124];
format(string2,sizeof(string2),"Du wurdest von %d zu einem Admin des Levels %s befördert.",name, level);
SendClientMessage(Spieler,ROT,string2);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Diesen Befehl darfst du nicht benutzen!");
return 1;
}
}

Alles anzeigen

Buschmensch · 17. Mai 2012

Nimm dir das von Sohrab und passe es deinem Script an.

H4lo · 17. Mai 2012

Zitat von iMan

Habe jetzt 4 Errors. (habe einige behoben)

Zitat

C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 029: invalid expression, assumed zero
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : warning 215: expression has no effect
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : warning 215: expression has no effect
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 001: expected token: ";", but found ")"
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : error 029: invalid expression, assumed zero
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(242) : 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.

Alles anzeigen

Da hat mir keiner geantwortet
Weitere Vorschläge bitte.

Kaliber · 17. Mai 2012

So hier der komplette Code:
new idx;
new cmd[256] = strtok(cmdtext, idx);
if(strcmp(cmd, "/makeadmin", true) == 0)
{
if(IsPlayerAnAdmin(playerid,6))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp)) return SendClientMessage(playerid, GRAU, "Benutze /makeadmin [SpielerID] [Level]");
new level = strval(tmp);
tmp = strtok(cmdtext, idx);
new Spieler = ReturnUser(tmp);
new Spielerdatei[64];
new name[MAX_PLAYER_NAME];
new giveplayer[MAX_PLAYER_NAME];
GetPlayerName(playerid,name,sizeof(name));
GetPlayerName(Spieler, giveplayer, sizeof(giveplayer));
format(Spielerdatei,sizeof(Spielerdatei),"/Account's/%s.txt",giveplayer);
dini_IntSet(Spielerdatei,"Adminlevel",level);
new string[124];
format(string,sizeof(string),"Du hast %d zu einem Admin des Levels %s befördert.",giveplayer, level);
SendClientMessage(playerid,ROT,string);
new string2[124];
format(string2,sizeof(string2),"Du wurdest von %d zu einem Admin des Levels %s befördert.",name, level);
SendClientMessage(Spieler,ROT,string2);
return 1;
}
else return SendClientMessage(playerid,GRAU,"Diesen Befehl darfst du nicht benutzen!");
}

mfg.

H4lo · 17. Mai 2012

Ich hab jetzt 1 Error, den ich nicht verstehe.

Zitat

C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(240) : warning 219: local variable "idx" shadows a variable at a preceding level
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(241) : error 008: must be a constant expression; assumed zero
C:\Users\Arash\Desktop\Alle Ordner\World~of~Reallife\gamemodes\World~of~Reallife.pwn(565) : warning 203: symbol is never used: "idx"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Alles anzeigen

Kaliber · 17. Mai 2012

lösche das new idx; das hattest du anscheinend schon deklariert

mfg.