Unrechable Code

John_Smith · 2. Januar 2012

Abend
hab ein kleines problem mit einer rollervermietung

Code

Zeile (934) : warning 225: unreachable code
Zeile (940) : warning 225: unreachable code
Zeile (946) : warning 225: unreachable code
Zeile (952) : warning 225: unreachable code
Zeile (975) : warning 225: unreachable code
Zeile (989) : warning 225: unreachable code
Zeile (1003) : warning 225: unreachable code
Zeile (1018) : warning 225: unreachable code

die zeilen

public OnPlayerExitVehicle(playerid, vehicleid)
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[0])
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;

if(Vehicle == rentroller[1]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;

if(Vehicle == rentroller[2]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;

if(Vehicle == rentroller[3]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;

if(Vehicle == rentroller[4]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;
}

public OnPlayerStateChange(playerid, newstate, oldstate)
{
if(newstate == PLAYER_STATE_DRIVER)
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[0])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

return 1;
}
}
return 1;

if(newstate == PLAYER_STATE_DRIVER) //hier
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[1])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

return 1;
}
}
return 1;

if(newstate == PLAYER_STATE_DRIVER) //hier
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[2])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

return 1;
}
}
return 1;

if(newstate == PLAYER_STATE_DRIVER) //hier
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[3])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

return 1;
}
}
return 1;

if(newstate == PLAYER_STATE_DRIVER) //hier
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[4])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

return 1;
}
}
return 1;
}

schonmal im vorraus vielen dank
mfg
John_Smith

/edit
errors genau markiert

Reloadet! · 2. Januar 2012

Makiere mal genau die zeilen ausserdem Returnst du zu oft in einer funktion

NumberOne · 2. Januar 2012

sind nur warnings sind nix schlimmes
oder buggt es ingame?

FirstNight · 2. Januar 2012

SetVehicleToRespawn(Vehicle);

probiere das mal , da du ja
new Vehicle = GetPlayerVehicleID(playerid);
schon vor definierst brauchst du normalerweise nicht nochmal
SetVehicleToRespawn(GetPlayerVehicleID(playerid));

John_Smith · 2. Januar 2012

die vermietung funktioniert nicht

die errors sind überall wo /hier steht
aber das komische meiner meinung nach ist ja wenn nur ein roller drinnen ist funktionierts aber wenns mehrere sind dann eben nicht

Goldkiller · 2. Januar 2012

Wenn du dir den Code mal genau anguckst ( zb bei OnPlayerExitVehicle ) , solltest du merken, dass nach einem return bei dir noch Code folgt.
Nach einem return kann aber kein code mehr in der Funktion aufgerufen werden, daher ist alles nach dem return sinnlos bzw es wird nie aufgerufen. Denn das return beendet die Funktion ( kann man sich jedenfalls so vorstellen ).
Ich zeig dir mal bei OnPlayerExitVehicle, wo die returns hin könnte ( oder du lässt sie weg ).

public OnPlayerExitVehicle(playerid, vehicleid)
{
new Vehicle = GetPlayerVehicleID(playerid);
if(Vehicle == rentroller[0])
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
return 1;
}

if(Vehicle == rentroller[1]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
return 1;
}

if(Vehicle == rentroller[2]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
return 1;
}

if(Vehicle == rentroller[3]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
return 1;
}
if(Vehicle == rentroller[4]) //hier
{
SetVehicleToRespawn(GetPlayerVehicleID(playerid));
}
return 1;
}

FirstNight · 2. Januar 2012

public OnPlayerStateChange(playerid, newstate, oldstate)
{
new Vehicle = GetPlayerVehicleID(playerid);

if(newstate == PLAYER_STATE_DRIVER)
{

if(Vehicle == rentroller[0])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);
}
if(Vehicle == rentroller[1])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

}
if(Vehicle == rentroller[2])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

}
if(Vehicle == rentroller[3])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

}
if(Vehicle == rentroller[4])
{
SendClientMessage(playerid,HELLGRAU,"Dieses Fahrzeug kannst du mieten! Tippe /mieten, wenn du nicht möchtest, dann /mietausstieg.");

TogglePlayerControllable(playerid,0);

}
}
return 1;
}

Ich würde dann so versuchen und das andere wie Goldkiller geschrieben hat da war etwas üeberschneidung drin

John_Smith · 2. Januar 2012

ja danke das hat geholfen aber es sind aber die anderen errors von
public OnPlayerStateChange(playerid, newstate, oldstate)
da

/edit
vielen dank euch allen es funktioniert

Atrox · 2. Januar 2012

Kleine Optimierung: Verwende eine if Schleife:
if(Vehicle == rentroller[1] || Vehicle == rentroller[2])
usw

Deadmau5 · 2. Januar 2012

Oder so
if(Vehicle == (rentroller[1] || rentroller[2]))

Goldkiller · 2. Januar 2012

Zitat von Deadmau5

Oder so
if(Vehicle == (rentroller[1] || rentroller[2]))

Das funktioniert niemals.
Warum ? Splitten wir das:
(rentroller[1] || rentroller[2])

rentroller[1]
rentroller[2]
Sind beides eigene Statements, da mit ODER ( ) verknüpft.Ist rentroller[1] oder rentroller[2] 0 , dann ist das Statement false.Wenn nicht,ist es für sich Wahr.Es ist zwar mit oder verknüpft, macht aber nichts. Ist eines der beiden wahr,ist das gesamte Statement ( rentroller[1] rentroller[2]) ebenfalls wahr. Wenn nicht,ist es falsch.
Heisst, das Gesamte Statement (rentroller[1] rentroller[2]) ist entweder 0 ( falsch ) oder 1 ( wahr ).
Funktioniert also nur in Ausnahmesitituationen, zb wenn rentroller[1] oder rentroller[2] Vehicleid 0 oder 1 ist. Zusätzlich muss dann auch Vehicle 0 oder 1 sein.

//Edit:
Um es nochmal zu verdeutlichen, wie der Ausgang immer sein würde.

Vehicle == ( 1 / wahr )
Vehicle == ( 0 / falsch )
if(Vehicle == (rentroller[1] || rentroller[2]))

breadfish.de 11. März 2022