ID 0 Bug beim Tanksystem

Frank Bullitt · 21. Dezember 2010

Hallo,

ich habe das Tanksystem vom Godfatherscript etwas umgebaut und für 0.3c angepasst.
Aber 2 if-Abfragen in der public CheckGas() werden nur mit ID 0 ausgeführt.
Hier die public:

public CheckGas()
{
for(new i=0;i<MAX_PLAYERS;i++)
{
for(new v=0;v<MAX_VEHICLES;v++)
{
if(Gas[v] >= 1)
{
new engine, lights, alarm, doors, bonnet, boot, objective;
GetVehicleParamsEx(v, engine, lights, alarm, doors, bonnet, boot, objective);
if(engine == 1)
{
if(Gas[v] <= 10) { if(IsPlayerInVehicle(i, v) && GetPlayerState(i) == PLAYER_STATE_DRIVER) { PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0); } } // <--- Abfrage 1, die nur für ID 0 funktioniert
if(IsAPlane(v) || IsABoat(v)) { Gas[v]++; }
Gas[v]--;
}
}
else
{
new engine,lights,alarm,doors,bonnet,boot,objective;
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
if(IsPlayerInVehicle(i, v) && GetPlayerState(i) == PLAYER_STATE_DRIVER) // <--- Abfrage 2, die nur für ID 0 funktioniert
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}
}
}
}
return 1;
}
Ich habe schon 2 for-Schleifen drin, aber scheint trotzdem nur für ID 0 zu funktionieren.

Ich hoffe auf schnelle Antwort.

MfG

eli · 21. Dezember 2010

probi es so:
public CheckGas(playerid)
{
new string[256];
for(new i=0;i<MAX_PLAYERS;i++)
{
if(IsPlayerConnected(i))
{
if(GetPlayerState(i) == PLAYER_STATE_DRIVER)
{
new vehicle = GetPlayerVehicleID(i);
new Model = GetVehicleModel(GetPlayerVehicleID(i));
if(Gas[vehicle] >= 1)
{
if(Gas[vehicle] <= 10) { PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0); }
if(gGas[i] == 1) {
format(string, sizeof(string), "~r~~n~~n~~n~~n~~n~~n~~n~~n~~n~Benzin:~w~ %d%",Gas[vehicle]);
GameTextForPlayer(i,string,15500,3); }
if(IsAPlane(vehicle) || IsABoat(vehicle) || Model == 462 || Model == 481) { Gas[vehicle]++; }
Gas[vehicle]--;
}
else
{
new vID = GetPlayerVehicleID(playerid);
NoFuel[vID] = 1;
new vid = GetPlayerVehicleID(playerid);
new engine,lights,alarm,doors,bonnet,boot,objective;
GetVehicleParamsEx(vid,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(vid,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
motor[vid] = false;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Kein Benzin mehr~n~Dein motor geht aus",3000,3);
}
if(Gas[vehicle] > GetGasMax(GetVehicleModel(vehicle)))
{
Gas[vehicle] = GetGasMax(GetVehicleModel(vehicle));
}
}
}
}
return 1;
}

Mach bei public fillup:
new vID = GetPlayerVehicleID(playerid);
NoFuel[vID] = 0;
und bei /motor an wenn du das hast:
new vID = GetPlayerVehicleID(playerid);
if(NoFuel[vID] == 1)
{
SendClientMessage(playerid, COLOR_RED, "Du hast keinen tank mehr du kannst den motor nicht starten!");
return 1;
}
//edit

Frank Bullitt · 21. Dezember 2010

Erstmal danke für deine schnelle Antwort, aber ich wollte das eigentlich so haben, wie ich's gemacht habe

Wollte nur eine Lösung für das Problem.

LiamRowley · 21. Dezember 2010

Ist es normal, dass du bei der Zeile wo das nur für die ID 0 funktioniert, ein ";" fehlt?

Hier ist's so:

if(IsPlayerInVehicle(i, v) && GetPlayerState(i) == PLAYER_STATE_DRIVER)

// Edit, Dummheit gehört bestraft. XD ich lag falsch. ;S

eli · 21. Dezember 2010

probir:
if(!IsPlayerInVehicle(i, v)) // <--- Abfrage 2, die nur für ID 0 funktioniert
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}

Frank Bullitt · 21. Dezember 2010

Zitat von eli

probir:
if(!IsPlayerInVehicle(i, v)) // <--- Abfrage 2, die nur für ID 0 funktioniert
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}

Alles anzeigen

So ist's falsch, da wird abgefragt ob ich in keinem Auto sitze und wenn ja, wird mir die Meldung angezeigt die eigentlich kommen soll, wenn ich der Fahrer eines Fahrzeuges bin und der Tank leer ist.

eli · 21. Dezember 2010

Probire das:
if(IsPlayerInVehicle(i, v) && GetPlayerState(i) != PLAYER_STATE_DRIVER) // <--- Abfrage 2, die nur für ID 0 funktioniert
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}

Frank Bullitt · 21. Dezember 2010

Zitat von eli

Probire das:
if(IsPlayerInVehicle(i, v) && GetPlayerState(i) != PLAYER_STATE_DRIVER) // <--- Abfrage 2, die nur für ID 0 funktioniert
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}

Alles anzeigen

Also, das Ausrufezeichen (!) steht für nicht.
Also würde man damit jetzt abfragen, ob der Spieler in einem Fahrzeug ist und nicht der Fahrer ist.
Ich will das ja so, dass man im Fahrzeug sein muss und der Fahrer sein muss.

eli · 21. Dezember 2010

Also bei mir z.B ist es bei motor an/aus so:
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht der Fahrer!");
return 1;
}
wenn ich es so habe dan kann es nur der fahrer machen und wenn ich es so mache:
if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
{
SendClientMessage(playerid, COLOR_RED, "Du bist nicht der Fahrer!");
return 1;
}
kann es der beifahrer usw machen nur der fahre nicht

LiamRowley · 21. Dezember 2010

@ an eli:
Wenn man keine Ahnung hat, einfach mal nichts schreiben.
Lass es, bevor du noch 10x schreibst. Und die Antwort falsch ist.

Sean Mahoney,

kann es sein, dass du vielleicht irgendeine Klammer falsch gesetzt habe?
Du hast im Script:

Gas[v]--;
}
}
else
{

stehen.
Ich habe:

Gas[vehicle]--;
}
else
{

Vielleicht daran?

Frank Bullitt · 21. Dezember 2010

Hm, kann ich mir eigentlich nicht vorstellen..
Hab alles überprüft kann jedoch keinen Fehler finden und wenn's tatsächlich ein Klammerfehler wäre, würde die Funktion wahrscheinlich garnicht funktionieren, aber für ID 0 geht es ja..

LiamRowley · 21. Dezember 2010

Vielleicht liegt's ja sogar dadran?
nen Versuch ist's wert.
Wenns nicht geht, hab ich auch keine Ahnung mehr.

Frank Bullitt · 21. Dezember 2010

Zitat von LiamRowley

Vielleicht liegt's ja sogar dadran?
nen Versuch ist's wert.
Wenns nicht geht, hab ich auch keine Ahnung mehr.

Hast recht, ich werde es mal versuchen und melde mich nachher wieder

LiamRowley · 21. Dezember 2010

Dann probier mal schön.

Frank Bullitt · 21. Dezember 2010

Hab die Klammern anders gesetzt.
Resultat: Geht immernoch nur für ID 0

eli · 21. Dezember 2010

probire es mal so:
public CheckGas(playerid)
{
for(new i=0;i<MAX_PLAYERS;i++)
{
for(new v=0;v<MAX_VEHICLES;v++)
{
if(Gas[v] >= 1)
{
new engine, lights, alarm, doors, bonnet, boot, objective;
GetVehicleParamsEx(v, engine, lights, alarm, doors, bonnet, boot, objective);
if(engine == 1)
{
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
if(Gas[vehicle] <= 10) { PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0); }
}
if(IsAPlane(v) || IsABoat(v)) { Gas[v]++; }
Gas[v]--;
}
}
else
{
new engine,lights,alarm,doors,bonnet,boot,objective;
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}
}
}
return 1;
}

Frank Bullitt · 21. Dezember 2010

Zitat von eli

probire es mal so:
public CheckGas(playerid)
{
for(new i=0;i<MAX_PLAYERS;i++)
{
for(new v=0;v<MAX_VEHICLES;v++)
{
if(Gas[v] >= 1)
{
new engine, lights, alarm, doors, bonnet, boot, objective;
GetVehicleParamsEx(v, engine, lights, alarm, doors, bonnet, boot, objective);
if(engine == 1)
{
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
if(Gas[vehicle] <= 10) { PlayerPlaySound(i, 1085, 0.0, 0.0, 0.0); }
}
if(IsAPlane(v) || IsABoat(v)) { Gas[v]++; }
Gas[v]--;
}
}
else
{
new engine,lights,alarm,doors,bonnet,boot,objective;
GetVehicleParamsEx(v,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(v,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
NoFuel[i] = 1;
GameTextForPlayer(i,"~w~~n~~n~~n~~n~~n~~n~~n~~n~Tank leer",1500,3);
}
}
}
return 1;
}

Alles anzeigen

So mit den Klammern hatte ich es auch probiert, aber geht ja nicht
Und die Ausrufezeichen gehören da nicht hin, bei GetPlayerState, weil wie gesagt, Ausrufezeichen bedeuten Verneinung

eli · 21. Dezember 2010

die ! können manchmal helfen! probir es doch mal...
ich habe früher auch immer nach gefragt und dan haben die meisten gesagt mach != dan funktionirt es
Ich probire es jetzt mal selbst

LiamRowley · 21. Dezember 2010

Zitat von eli

die ! können manchmal helfen! probir es doch mal...
ich habe früher auch immer nach gefragt und dan haben die meisten gesagt mach != dan funktionirt es
Ich probire es jetzt mal selbst

1. Probier
Hast immer wieder probir.

2. Es kann sein, dass es hilft.
Jedoch kommt es auch auf die Art an.

Frank Bullitt · 21. Dezember 2010

Zitat von Sean Mahoney

Also, das Ausrufezeichen (!) steht für nicht.
Also würde man damit jetzt abfragen, ob der Spieler in einem Fahrzeug ist und nicht der Fahrer ist.
Ich will das ja so, dass man im Fahrzeug sein muss und der Fahrer sein muss.

In diesem Fall würde es verneinen und somit garnicht funktionieren..