SA-MP Forums Archive
Fuel System Textdraw - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: Fuel System Textdraw (/showthread.php?tid=652763)


Fuel System Textdraw - JaskaranSingh - 18.04.2018

I created a textdraw for fuel system. Everything is working fine for one player. But after driving the second vehicle, the previous value of fuel is overlapping with new one and both the values are decreasing together.

PHP код:
new fueltimer[MAX_VEHICLES]; 
PHP код:
public OnPlayerConnect(playerid)
{
    VehicleMeter[playerid] = TextDrawCreate(498.000000, 140.000000, " ");
    TextDrawBackgroundColor(VehicleMeter[playerid], 255);
    TextDrawFont(VehicleMeter[playerid], 1);
    TextDrawLetterSize(VehicleMeter[playerid], 0.389999, 1.299999);
    TextDrawColor(VehicleMeter[playerid], 0xFFFFFFFF);
    TextDrawSetOutline(VehicleMeter[playerid], 1);
    TextDrawSetProportional(VehicleMeter[playerid], 1);
    
    return 1;
} 
PHP код:
public OnPlayerStateChange(playerid, newstate, oldstate)
{
    new vid = GetPlayerVehicleID(playerid);
    if(newstate == PLAYER_STATE_DRIVER)
    {
        TextDrawShowForPlayer(playerid, VehicleMeter[playerid]);
        fueltimer[vid] = SetTimerEx("DecreaseFuel", 1000, 1, "ii", playerid, vid);
    }
    else
    {
        TextDrawHideForPlayer(playerid, VehicleMeter[playerid]);
        KillTimer(fueltimer[vid]);
    }
    return 1;
} 
PHP код:
public OnPlayerExitVehicle(playerid, vehicleid)
{
    KillTimer(fueltimer[vehicleid]);
    TextDrawHideForPlayer(playerid, VehicleMeter[playerid]);
    return 1;
} 
Function:
PHP код:
public DecreaseFuel(playerid, vehicleid) // Decreases fuel by 0.035 (Whole fuel finishes in almost 50 minutes)
{
    new engine, lights, alarm, doors, bonnet, boot, objective, string[128];
    GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
    if(IsBicycle(vehicleid) == 1 || IsValidDealershipVehicle(vehicleid) == 1 || engine == 0)
        return 1;
    else
    {
        if(Vehicle[vehicleid][vFuel] <= 0)
        {
            ToggleEngine(vehicleid, VEHICLE_PARAMS_OFF);
            SendClientMessage(playerid, COLOR_LIGHTRED, "Your vehicle is out of fuel.");
            KillTimer(fueltimer[vehicleid]);
        }
        format(string, sizeof(string), "Fuel: %d%%", floatround(Vehicle[vehicleid][vFuel]));
        TextDrawSetString(VehicleMeter[playerid], string);
        Vehicle[vehicleid][vFuel] -= 0.035;
    }
    return 1;
} 


Re: Fuel System Textdraw - JaskaranSingh - 19.04.2018

Anyone?

Re: Fuel System Textdraw - jasperschellekens - 19.04.2018

That's why you should use player textdraws.. Also destroy them onplayerexitvehicle instead of hiding them. Create the textdraw again for the new vehicle onplayerentervehicle. You should also restore all the values you use to calculate when the player enters a vehicle.

Re: Fuel System Textdraw - JaskaranSingh - 19.04.2018

PHP код:
public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
    fueltimer[vehicleid] = SetTimerEx("DecreaseFuel", 1000, 1, "ii", playerid, vehicleid);
    VehicleMeter[playerid] = CreatePlayerTextDraw(playerid, 498.000000, 140.000000, " ");
    PlayerTextDrawBackgroundColor(playerid, VehicleMeter[playerid], 255);
    PlayerTextDrawFont(playerid, VehicleMeter[playerid], 1);
    PlayerTextDrawLetterSize(playerid, VehicleMeter[playerid], 0.389999, 1.299999);
    PlayerTextDrawColor(playerid, VehicleMeter[playerid], 0xFFFFFFFF);
    PlayerTextDrawSetOutline(playerid, VehicleMeter[playerid], 1);
    PlayerTextDrawSetProportional(playerid, VehicleMeter[playerid], 1);
    return 1;
} 
PHP код:
public OnPlayerExitVehicle(playerid, vehicleid)
{
    KillTimer(fueltimer[vehicleid]);
    PlayerTextDrawDestroy(playerid, VehicleMeter[playerid]);
    return 1;
} 
It's still giving the same problems.

Re: Fuel System Textdraw - CodeStyle175 - 19.04.2018

wtf are you even doing, to you understand?
when you write code, first and most important thing is to understand what this code is doing.

Re: Fuel System Textdraw - Pottus - 19.04.2018

Quote:
Originally Posted by jasperschellekens
Посмотреть сообщение
That's why you should use player textdraws.. Also destroy them onplayerexitvehicle instead of hiding them. Create the textdraw again for the new vehicle onplayerentervehicle. You should also restore all the values you use to calculate when the player enters a vehicle.
Unless you have a lot of player textdraws it makes no sense to delete them and even if you did have a lot it is difficult to approach the limit. In certain cases where TD's would be generated from iterated loops yeah I could see that method being beneficial. However your suggestion makes absolutely no sense in this case use the right technique for the job!

Re: Fuel System Textdraw - JaskaranSingh - 19.04.2018

Quote:
Originally Posted by CodeStyle175
Посмотреть сообщение
wtf are you even doing, to you understand?
when you write code, first and most important thing is to understand what this code is doing.
That is the problem. Everything that is using timers on my server is doing like this. Everything works well for the first player. For the second player, everything runs twice. For the third, thrice and so on. WTH is going on?