Probleme mit /rentcar System

Shoxx · 2. November 2015

Hallo,
ich habe vor Jahren mal ein Mietsystem gescriptet, was nun total alt ist und ich etwas überarbeiten möchte.

Damals war es so gescriptet, dass man sich in das auto setzt und dann mit /rentcar das auto bewegen kann..
Ist man dann ausgestiegen, und wollte wieder einsteigen musste man es wieder mieten...
Außerdem ist das Auto nach 1 Minute nicht benutzung respawnt.

NUN möchte ich es so, dass man sich reinsetzt das auto mieten kann und es dann für 30 minuten fahren kann. Sollte sich also jemand anderes reinsetzten sollte eine nachricht kommen wo drin steht Das Fahrzeug gehört XY oder so. Und wenn die 30 Minuten um sind sollte das Auto respawnen, auch wenn man im Wagen sitzt.

Hier mein Altes System, wie bearbeite ich es nun?

C: Miet1

if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[1])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 1 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
		{
		if(newstate == PLAYER_STATE_DRIVER)
	 	{
		//TextDrawHideForPlayer(playerid,Profico[playerid]);
		//TextDrawHideForPlayer(playerid,punkenun[playerid]);
  		TextDrawShowForPlayer(playerid, Speedo[playerid]);
		if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
		{
			SetTimerEx("Speedometer", 100, 1, "d", playerid);
			if(IsABike(GetPlayerVehicleID(playerid)))
			{
			    timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
			}
			else
			{
			    timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
			}
		}
	}
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[2])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 1 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
			{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[3])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 1 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
			{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[4])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 1 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
				{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[5])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 5 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
				{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[6])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 5 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
				{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[5])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 5 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}
				{
	if(newstate == PLAYER_STATE_DRIVER)
	       {
	new Vehicle = GetPlayerVehicleID(playerid);
	       if(Vehicle == rentroller[4])         // Die Abfrage ob er im rentroller ist
	{
	SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /Verlassen.");
	SendClientMessage(playerid,0xF60000F6,"Das Auto wird beim Aussteigen nach 5 Minute Respawnt!");
	//Die Nachricht
	TogglePlayerControllable(playerid,0);
	//Freezt ihn
	       return 1;
			}
		}
	}

Alles anzeigen

Befehl:

Code

//Vermietung
	if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[1])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 250) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-250);
				return 1;
			}
		}
	}
		if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[2])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 2500) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-2500);
				return 1;
			}
		}
	}
		if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[3])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 750) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-750);
				return 1;
			}
		}
	}
		if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[4])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 1000) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-1000);
				return 1;
			}
		}
	}
	if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[5])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 6000) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld! (6000$)");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-6000);
				return 1;
			}
		}
	}
	if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[6])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 6000) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet! (6000$)");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-6000);
				return 1;
			}
		}
	}
			if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[7])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 6000) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld! (6000$)");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-6000);
				return 1;
			}
		}
	}
			if (strcmp("/rentcar", cmdtext, true, 10) == 0)
	{
		new Vehicle = GetPlayerVehicleID(playerid);
		if(Vehicle == rentroller[8])
		{
			new Geld = GetPlayerMoney(playerid);
			if(Geld <= 6000) // So ist richtig
			{
	 			SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld! (6000$)");
			}
			else
			{
				SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
				TogglePlayerControllable(playerid,1);
				GivePlayerMoney(playerid,-6000);
				return 1;
			}
		}
	}
	if (strcmp("/verlassen", cmdtext, true, 10) == 0)
	{
	    if(ohnmacht[playerid]==0)
	    {
			RemovePlayerFromVehicle(playerid); // Schmeißt den Spieler herraus
	        TogglePlayerControllable(playerid,1); // Entfreezt ihn
	        KillTimer(hacken[playerid]);
		}
		else
		{
		    SendClientMessage(playerid, COLOR_RED,"Du bist gerade Ohmächtig und kannst das nicht machen");
	 	}
   		return 1;
	}
	//vermietung

Alles anzeigen

Jeffry · 3. November 2015

Der Code ist aber ziemlich abenteuerlich.

OnPlayerStateChange kannst du so machen:

C

if(newstate == PLAYER_STATE_DRIVER)
{
    if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
    {
        TextDrawShowForPlayer(playerid, Speedo[playerid]);
        //SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,
        //wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann
        //hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen
        //wird. Deine CPU wird dir das nicht danken ;)
        //Also besser so:
        speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.
        //Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch
        //alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.
        if(IsABike(GetPlayerVehicleID(playerid)))
        {
            timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
        }
        else
        {
            timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
        }
    }
    //Hier der Code für die Roller:
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");
            SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");
            TogglePlayerControllable(playerid,0);
            break;
        }
    }
}

Alles anzeigen

Den /rentcar Befehl kannst du dann so machen:

C

if(!strcmp("/rentcar", cmdtext, true))
{
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            new kosten;
            switch(i)
            {
                case 0: kosten = 100;
                case 1: kosten = 250;
                case 2: kosten = 2000;
                //...
                default: kosten = 500;
            }
            if(GetPlayerMoney(playerid) <= kosten) return SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
            SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
            TogglePlayerControllable(playerid,1);
            GivePlayerMoney(playerid,kosten);
            return 1;
        }
    }
    return SendClientMessage(playerid, 0xFF0000FF, "Error: Dies ist kein Auto, das gemietet werden kann.");
}

Alles anzeigen

Der /verlassen Befehl bleibt weitgehend gleich.

Wenn du es nun anpassen willst, dass das Fahrzeug für genau 30 Minuten aktiv ist, dann musst du eine Spielervariable anlegen, in der du den Wert speicherst, wann die Zeit vorbei ist und eine Variable die die Index ID beinhaltet, unter den Includes:

C

new rentTime[MAX_PLAYERS];
new rentIdx[MAX_PLAYERS];

Sowie eine Variable um das Fahrzeug als gemietet zu markieren (-1 = nicht vermietet):

C

new rentBlocked[MAX_VEHICLES] = {-1,...};

Unter OnGameModeInit startest du einen Prüf-Timer:

C

SetTimer("CheckRentCars", 5000, true);

[wiki]SetTimer[/wiki]

Und der Code dazu:

C

forward CheckRentCars();
public CheckRentCars()
{
    for(new i=0; i<MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i) && rentTime[i] != 0 && rentTime[i] < gettime())
        {
            new idx = rentIdx[playerid];
            SetVehicleToRespawn(rentroller[idx]);
            SendClientMessage(i, 0xFF0000FF, "Zeit vorbei.");
            rentTime[i] = 0;
        }
    }
    return 1;
}

Alles anzeigen

Den /rentcar Befehl musst du dann so anpassen:

C

if(!strcmp("/rentcar", cmdtext, true))
{
    if(rentTime[playerid] != 0) return SendClientMessage(playerid, 0xFF0000FF, "Error: Du hast bereits ein Auto gemietet.");
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            new kosten;
            switch(i)
            {
                case 0: kosten = 100;
                case 1: kosten = 250;
                case 2: kosten = 2000;
                //...
                default: kosten = 500;
            }
            if(GetPlayerMoney(playerid) <= kosten) return SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
            SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
            TogglePlayerControllable(playerid,1);
            GivePlayerMoney(playerid,kosten);
            rentTime[playerid] = gettime() + 30 * 60 * 1000; //30 Minuten zu je 60 Sekunden und 1000ms.
            rentIdx[playerid] = i;
            rentBlocked[i] = playerid;
            return 1;
        }
    }
    return SendClientMessage(playerid, 0xFF0000FF, "Error: Dies ist kein Auto, das gemietet werden kann.");
}

Alles anzeigen

Bei OnPlayerDisconnect zum Beispiel:

C

if(rentTime[playerid] != 0)
{
    new idx = rentIdx[playerid];
    SetVehicleToRespawn(rentroller[idx]);
    rentTime[playerid] = 0;
    rentBlocked[idx] = -1;
}

Und das OnPlayerStateChange sieht dann so aus:

C

if(newstate == PLAYER_STATE_DRIVER)
{
    if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
    {
        TextDrawShowForPlayer(playerid, Speedo[playerid]);
        //SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,
        //wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann
        //hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen
        //wird. Deine CPU wird dir das nicht danken ;)
        //Also besser so:
        speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.
        //Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch
        //alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.
        if(IsABike(GetPlayerVehicleID(playerid)))
        {
            timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
        }
        else
        {
            timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
        }
    }
    //Hier der Code für die Roller:
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            if(rentBlocked[i] == -1)
            {
                SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");
                SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");
                TogglePlayerControllable(playerid,0);
            }
            else
            {
                SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug gehört bereits einem Spieler.");
                RemovePlayerFromVehicle(playerid);
            }
            break;
        }
    }
}

Alles anzeigen

Beachte bitte die Kommentare im Code und passie die Stellen entsprechend an.

Shoxx · 4. November 2015

Zitat von Jeffry

Der Code ist aber ziemlich abenteuerlich.

OnPlayerStateChange kannst du so machen:

C

if(newstate == PLAYER_STATE_DRIVER)
{
    if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
    {
        TextDrawShowForPlayer(playerid, Speedo[playerid]);
        //SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,
        //wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann
        //hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen
        //wird. Deine CPU wird dir das nicht danken ;)
        //Also besser so:
        speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.
        //Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch
        //alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.
        if(IsABike(GetPlayerVehicleID(playerid)))
        {
            timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
        }
        else
        {
            timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
        }
    }
    //Hier der Code für die Roller:
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");
            SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");
            TogglePlayerControllable(playerid,0);
            break;
        }
    }
}

Alles anzeigen

Den /rentcar Befehl kannst du dann so machen:

C

if(!strcmp("/rentcar", cmdtext, true))
{
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            new kosten;
            switch(i)
            {
                case 0: kosten = 100;
                case 1: kosten = 250;
                case 2: kosten = 2000;
                //...
                default: kosten = 500;
            }
            if(GetPlayerMoney(playerid) <= kosten) return SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
            SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
            TogglePlayerControllable(playerid,1);
            GivePlayerMoney(playerid,kosten);
            return 1;
        }
    }
    return SendClientMessage(playerid, 0xFF0000FF, "Error: Dies ist kein Auto, das gemietet werden kann.");
}

Alles anzeigen

Der /verlassen Befehl bleibt weitgehend gleich.

Wenn du es nun anpassen willst, dass das Fahrzeug für genau 30 Minuten aktiv ist, dann musst du eine Spielervariable anlegen, in der du den Wert speicherst, wann die Zeit vorbei ist und eine Variable die die Index ID beinhaltet, unter den Includes:

C

new rentTime[MAX_PLAYERS];
new rentIdx[MAX_PLAYERS];

Sowie eine Variable um das Fahrzeug als gemietet zu markieren (-1 = nicht vermietet):

C

new rentBlocked[MAX_VEHICLES] = {-1,...};

Unter OnGameModeInit startest du einen Prüf-Timer:

C

SetTimer("CheckRentCars", 5000, true);

[wiki]SetTimer[/wiki]

Und der Code dazu:

C

forward CheckRentCars();
public CheckRentCars()
{
    for(new i=0; i<MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i) && rentTime[i] != 0 && rentTime[i] < gettime())
        {
            new idx = rentIdx[playerid];
            SetVehicleToRespawn(rentroller[idx]);
            SendClientMessage(i, 0xFF0000FF, "Zeit vorbei.");
            rentTime[i] = 0;
        }
    }
    return 1;
}

Alles anzeigen

Den /rentcar Befehl musst du dann so anpassen:

C

if(!strcmp("/rentcar", cmdtext, true))
{
    if(rentTime[playerid] != 0) return SendClientMessage(playerid, 0xFF0000FF, "Error: Du hast bereits ein Auto gemietet.");
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            new kosten;
            switch(i)
            {
                case 0: kosten = 100;
                case 1: kosten = 250;
                case 2: kosten = 2000;
                //...
                default: kosten = 500;
            }
            if(GetPlayerMoney(playerid) <= kosten) return SendClientMessage(playerid,0xF60000F6,"Du hast nicht genügend Geld!");
            SendClientMessage(playerid,0xF60000F6,"Du hast dieses Auto gemietet!");
            TogglePlayerControllable(playerid,1);
            GivePlayerMoney(playerid,kosten);
            rentTime[playerid] = gettime() + 30 * 60 * 1000; //30 Minuten zu je 60 Sekunden und 1000ms.
            rentIdx[playerid] = i;
            rentBlocked[i] = playerid;
            return 1;
        }
    }
    return SendClientMessage(playerid, 0xFF0000FF, "Error: Dies ist kein Auto, das gemietet werden kann.");
}

Alles anzeigen

Bei OnPlayerDisconnect zum Beispiel:

C

if(rentTime[playerid] != 0)
{
    new idx = rentIdx[playerid];
    SetVehicleToRespawn(rentroller[idx]);
    rentTime[playerid] = 0;
    rentBlocked[idx] = -1;
}

Und das OnPlayerStateChange sieht dann so aus:

C

if(newstate == PLAYER_STATE_DRIVER)
{
    if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
    {
        TextDrawShowForPlayer(playerid, Speedo[playerid]);
        //SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,
        //wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann
        //hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen
        //wird. Deine CPU wird dir das nicht danken ;)
        //Also besser so:
        speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);
        //Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.
        //Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch
        //alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.
        if(IsABike(GetPlayerVehicleID(playerid)))
        {
            timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
        }
        else
        {
            timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
        }
    }
    //Hier der Code für die Roller:
    new veh = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(rentroller); i++)
    {
        if(veh == rentroller[i])
        {
            if(rentBlocked[i] == -1)
            {
                SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");
                SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");
                TogglePlayerControllable(playerid,0);
            }
            else
            {
                SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug gehört bereits einem Spieler.");
                RemovePlayerFromVehicle(playerid);
            }
            break;
        }
    }
}

Alles anzeigen

Beachte bitte die Kommentare im Code und passie die Stellen entsprechend an.

Alles anzeigen

Vielen Dank dafür !
Das einzige was bisher komisch ist ist, dass wenn ich das Auto miete, dort steht das das auto bereits einem spieler gehört obwohl es ja mir ist.

Jeffry · 4. November 2015

Ah, beim Einsteigen? Das habe ich vergessen, dem Spieler auch eine Nachricht zu senden, sorry.
Das wäre dann so:

C

if(newstate == PLAYER_STATE_DRIVER)
{
	if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))
	{
		TextDrawShowForPlayer(playerid, Speedo[playerid]);
		//SetTimerEx("Speedometer", 100, 1, "d", playerid);
		//Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,
		//wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann
		//hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen
		//wird. Deine CPU wird dir das nicht danken ;)
		//Also besser so:
		speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);
		//Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.
		//Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch
		//alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.
		if(IsABike(GetPlayerVehicleID(playerid)))
		{
			timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);
		}
		else
		{
			timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);
		}
	}
	//Hier der Code für die Roller:
	new veh = GetPlayerVehicleID(playerid);
	for(new i=0; i<sizeof(rentroller); i++)
	{
		if(veh == rentroller[i])
		{
			if(rentBlocked[i] == -1)
			{
				SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");
				SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");
				TogglePlayerControllable(playerid,0);
			}
			else if(rentBlocked[i] != playerid)
			{
				SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug gehört bereits einem Spieler.");
				RemovePlayerFromVehicle(playerid);
			}
			else
			{
				SendClientMessage(playerid, 0xF60000F6, "Willkommen.");
			}
			break;
		}
	}
}

Alles anzeigen

Shoxx · 4. November 2015

Zitat von Jeffry

Ah, beim Einsteigen? Das habe ich vergessen, dem Spieler auch eine Nachricht zu senden, sorry.
Das wäre dann so:

C

if(newstate == PLAYER_STATE_DRIVER){	if(!IsANoTachoVehicle(GetPlayerVehicleID(playerid)))	{		TextDrawShowForPlayer(playerid, Speedo[playerid]);		//SetTimerEx("Speedometer", 100, 1, "d", playerid);		//Ich habe diesen Timer auskommentiert, da er jedes mal gestartet würde,		//wenn du in ein Fahrzeug einsteigst, das einen Tacho hat, sprich irgendwann		//hast du 1000 Timer am Laufen, von denen jeder 10x pro Sekunde aufgerufen		//wird. Deine CPU wird dir das nicht danken ;)		//Also besser so:		speedoTimer[playerid] = SetTimerEx("Speedometer", 100, 1, "d", playerid);		//Und den Timer auch wieder beenden, wenn der Speedometer nicht mehr angezeigt wird.		//Oder noch besser ist es, einen einzigen globalen Timer mit einer Schleife durch		//alle Spieler zu machen, der Timer wird dann bei OnGameModeInit gestartet.		if(IsABike(GetPlayerVehicleID(playerid)))		{			timer2[playerid] = SetTimerEx("TankTimer",60000,1,"i",playerid);		}		else		{			timer2[playerid] = SetTimerEx("TankTimer",20000,1,"i",playerid);		}	}	//Hier der Code für die Roller:	new veh = GetPlayerVehicleID(playerid);	for(new i=0; i<sizeof(rentroller); i++)	{		if(veh == rentroller[i])		{			if(rentBlocked[i] == -1)			{				SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug kannst du mieten! Tippe /rentcar, wenn du nicht möchtest, dann /verlassen.");				SendClientMessage(playerid,0xF60000F6,"Das Auto wird nach 30 Minuten respawnt.");				TogglePlayerControllable(playerid,0);			}			else if(rentBlocked[i] != playerid)			{				SendClientMessage(playerid,0xF60000F6,"Dieses Fahrzeug gehört bereits einem Spieler.");				RemovePlayerFromVehicle(playerid);			}			else			{				SendClientMessage(playerid, 0xF60000F6, "Willkommen.");			}			break;		}	}}

Was auch nicht funktioniert ist, dass das Mietverhältnis aufgelöst und das Fahrzeug respawnt.

Jeffry · 4. November 2015

Kannst du bitte diese sinnlosen Voll-Zitate samt Code unterlassen? Das siehst du doch selbst, dass das abartig aussieht.

Zum Problem:
Wie siehst dein Code zu dem Timer CheckRentCars aus?

Shoxx · 4. November 2015

Zitat von Jeffry

Kannst du bitte diese sinnlosen Voll-Zitate samt Code unterlassen? Das siehst du doch selbst, dass das abartig aussieht.

Zum Problem:
Wie siehst dein Code zu dem Timer CheckRentCars aus?

So wie du ihn geschrieben hast.

Jeffry · 5. November 2015

Ahh ja, gettime arbeitet mit Sekunden, habe es mit GetTickCount vertauscht.

C

rentTime[playerid] = gettime() + 30 * 60 * 1000; //30 Minuten zu je 60 Sekunden und 1000ms.

Zu

C

rentTime[playerid] = gettime() + 30 * 60; //30 Minuten zu je 60 Sekunden.

Und nochmal der Hinweis:
Lasse die Vollzitate, es macht doch keinen Sinn, dass zwei mal der gleiche Text hintereinander da steht. Ich habe es schon gesehen, dass diese Vollzitate samt Code von Mods entfernt wurden, also unterlasse es bitte und nutze nur den ANTWORTEN Button, das reicht, ist übersichtlicher und du bekommst keine Verwarnung.

Shoxx · 5. November 2015

Okay, nun ist mir noch aufgefallen, dass ich auch ohne in einem Fahrzeug zu sitzen /rentcar eingeben kann. Dort steht dann auch Du hast dieses Fahrzeug gemietet...

Jeffry · 5. November 2015

Wie gesagt, die Codes musst du natürlich anpassen, wie du es haben willst.
Wenn man in einem Fahrzeug als Fahrer sein muss, dann:

C

if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return SendClientMessage(playerid, 0xFF0000FF, "Error: Du musst in einem Fahrzeug sein.");

Das sind minimale Anpassungen, die jeder anders haben will, daher habe ich das weg gelassen.

Probleme mit /rentcar System

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

Park Befehl macht Probleme

Problem bei BIZ System Arrays

Dynamisches Autohaus

Create3DTextlabel und Anit Cheat System Fragen