Script warning unreachable code

Critical(S)hit · 18. Juli 2013

Hey liebe community ich habe da ein problem ich habe ein Warning im script den ich nicht weg bekomme das teil des scriptes wo der ist :

Der warning : C:\Dokumente und Einstellungen\Justin.JU\Desktop\test\gamemodes\prohetzer.pwn(6260) : warning 225: unreachable code

if(gPlayerAccount[playerid] == 1)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;

123marvin123 · 18. Juli 2013

Geb uns mal den Code drüber

Firerfan · 18. Juli 2013

Welche der Zeilen ist es denn genau?

Critical(S)hit · 18. Juli 2013

der code genau :

Function OnPlayerSpawn(playerid)
{
StopAudioStreamForPlayer(playerid);
M_OnPlayerSpawn(playerid);
TextDrawShowForPlayer(playerid,Copyright);
TextDrawShowForPlayer(playerid,Datum);

if(PlayerInfo[playerid][pDonateRank] > 0)
{
GivePlayerWeapon(playerid,24,50);
}

GW_OnPlayerSpawn(playerid);
if(PlayerInfo[playerid][pTot] == 1 && PlayerInfo[playerid][pLevel] > 1)
{
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_YELLOW, "|________ Krankenhaus ________|");
SendClientMessage(playerid, COLOR_ORANGE, "Du bist gestorben , deswegen bist du im Krankenhaus");
SendClientMessage(playerid, COLOR_ORANGE, "Warte 60 Sekunden bis deine Behandlung fertig ist!");
SetPlayerSpawn(playerid);
if(IsPlayerInFrac(playerid, 1))
{
SetPlayerColor(playerid,NAMECOLOR_LSCOP);
}
else if(IsPlayerInFrac(playerid, 4))
{
SetPlayerColor(playerid, NAMECOLOR_MEDIC);
}
else
{
SetPlayerColor(playerid,TEAM_HIT_COLOR);
}
ResetPlayerWeapons(playerid);
return 1;
}
STDPlayer[playerid] = 0;
gTeam[playerid] = PlayerInfo[playerid][pTeam]; //Set the Team
if (gTeam[playerid] == 4) //medic / criminal reset team
{
gTeam[playerid] = 3;
}
if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
return 1;

if(gPlayerAccount[playerid] == 1) <-------------------- der warning
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;

YaBroo · 18. Juli 2013

Das heißt des öfteren das ein Return falsch gesetzt wurde sprich falsch returnt wird, zeig mal am besten den ganzen Code.

//Edit
Absicht oder fehlt da eine Klammer. Du meintest das ist der ganze Code

Firerfan · 18. Juli 2013

1. Benutze doch bitte die Pawn-Tags, wie Du es im vorherigen Beitrag auch getan hast.
2. Markiere bitte die Zeile, in der der Fehler auftritt.

Critical(S)hit · 18. Juli 2013

Firerfan is jetzt und wo der fehler ist is ein <------------------- bzw in reihe 56

YaBroo · 18. Juli 2013

if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
return 1;
if(gPlayerAccount[playerid] == 1)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;
}

was soll das denn!?
Function OnPlayerSpawn(playerid)
{
StopAudioStreamForPlayer(playerid);
M_OnPlayerSpawn(playerid);
TextDrawShowForPlayer(playerid,Copyright);
TextDrawShowForPlayer(playerid,Datum);

if(PlayerInfo[playerid][pDonateRank] > 0)
{
GivePlayerWeapon(playerid,24,50);
}
GW_OnPlayerSpawn(playerid);
if(PlayerInfo[playerid][pTot] == 1 && PlayerInfo[playerid][pLevel] > 1)
{
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_YELLOW, "|________ Krankenhaus ________|");
SendClientMessage(playerid, COLOR_ORANGE, "Du bist gestorben , deswegen bist du im Krankenhaus");
SendClientMessage(playerid, COLOR_ORANGE, "Warte 60 Sekunden bis deine Behandlung fertig ist!");
SetPlayerSpawn(playerid);
if(IsPlayerInFrac(playerid, 1))
{
SetPlayerColor(playerid,NAMECOLOR_LSCOP);
}
else if(IsPlayerInFrac(playerid, 4))
{
SetPlayerColor(playerid, NAMECOLOR_MEDIC);
}
else
{
SetPlayerColor(playerid,TEAM_HIT_COLOR);
}
ResetPlayerWeapons(playerid);
return 1;
}
STDPlayer[playerid] = 0;
gTeam[playerid] = PlayerInfo[playerid][pTeam]; //Set the Team
if (gTeam[playerid] == 4) //medic / criminal reset team
{
gTeam[playerid] = 3;
}
if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
}
if(gPlayerAccount[playerid] == 1)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;
}

Critical(S)hit · 18. Juli 2013

was soll was

Beavis · 18. Juli 2013

Es wird an den ersten Return liegen
Entfern das mal und Compile

Critical(S)hit · 18. Juli 2013

Beavis dann stürtzt pawno ab

YaBroo · 18. Juli 2013

Function OnPlayerSpawn(playerid)
{
StopAudioStreamForPlayer(playerid);
M_OnPlayerSpawn(playerid);
TextDrawShowForPlayer(playerid,Copyright);
TextDrawShowForPlayer(playerid,Datum);

if(PlayerInfo[playerid][pDonateRank] > 0)
{
GivePlayerWeapon(playerid,24,50);
}
GW_OnPlayerSpawn(playerid);
if(PlayerInfo[playerid][pTot] == 1 && PlayerInfo[playerid][pLevel] > 1)
{
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_LIGHTRED, " ");
SendClientMessage(playerid, COLOR_YELLOW, "|________ Krankenhaus ________|");
SendClientMessage(playerid, COLOR_ORANGE, "Du bist gestorben , deswegen bist du im Krankenhaus");
SendClientMessage(playerid, COLOR_ORANGE, "Warte 60 Sekunden bis deine Behandlung fertig ist!");
SetPlayerSpawn(playerid);
if(IsPlayerInFrac(playerid, 1))
{
SetPlayerColor(playerid,NAMECOLOR_LSCOP);
}
else if(IsPlayerInFrac(playerid, 4))
{
SetPlayerColor(playerid, NAMECOLOR_MEDIC);
}
else
{
SetPlayerColor(playerid,TEAM_HIT_COLOR);
}
ResetPlayerWeapons(playerid);
}
STDPlayer[playerid] = 0;
gTeam[playerid] = PlayerInfo[playerid][pTeam]; //Set the Team
if (gTeam[playerid] == 4) //medic / criminal reset team
{
gTeam[playerid] = 3;
}
if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
return 1;
}
if(gPlayerAccount[playerid] == 1)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;
}

Ja was soll das hier ->
if(gPlayerLogged[playerid] == 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
return 1;
if(gPlayerAccount[playerid] == 1)

Critical(S)hit · 18. Juli 2013

Das script hat ein alter scripter vonem server genommen / gemacht ka und daer kz mehr hatte habe ichs übernommen ich weiß auch ncih was der da getan hat

//EDIT: jetzt is kein warning nichts mehr ;

SendClientMessage(playerid, COLOR_LIGHTRED, "** Du musst dich vorher einloggen oder dir einen Account erstellen! ***");
Kick(playerid);
<--- habe dort das return 1; weg gemacht aber muss da nich ne -> } <- hin?

if(gPlayerAccount[playerid] == 1)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
else if(gPlayerAccount[playerid] == 0)
{
ForceClassSelection(playerid);
TogglePlayerSpectating(playerid, 0);
}
return 1;

YaBroo · 18. Juli 2013

Warum nimmst du nicht einfach das von mir gemachte und c&p's es einfach.
Ja das return kann raus muss aber nicht & da kommt wie ichs scho 2x gemacht habe eine geschweifte klammer hin.

breadfish.de 12. März 2022