Vehicle Command /Lock

Flumiggy[Scripting] · 13. August 2019

hey Breadfish community,

Ich bräuchte bitte mal eure hilfe

Ich habe anhand eines Tut. ein [Command("Lock")] erstellt
dieser wie man sieht abfragt ob ich in der nähe eines Autos bin oder nicht mit 7 einheiten gesetzt.

er schließt auch sauber ab mit dem command und beim wiederholen öffnet er das auto wieder
dennoch würde ich gerne an meinen client eine msg schicken das er einmal das Fahrzeug gespeert hat und einmal geöffnet hat

ich bekomme das nicht wirklich hin . weil eine Abfrage mit vehicle.Open finde ich nicht

CSS

[Command ("lock")]
        public void CMD_LockVehicle(Client client)
        {
            bool was_vehicle_found = false;


            foreach(Vehicle vehicle in NAPI.Pools.GetAllVehicles())
            {


                if (vehicle.Position.DistanceTo2D(client.Position) <= 7)
                {
                    vehicle.Locked = !vehicle.Locked;
                    was_vehicle_found = true;
                    client.SendChatMessage(" **Du hast das Auto zugesperrt !** ");


                    break;
                }


            }


            if (was_vehicle_found)
                return;


             client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");


        }

Alles anzeigen

LeonMrBonnie · 13. August 2019

Verwende doch einfach die Vehicle.Locked Variable?

Flumiggy[Scripting] · 13. August 2019

CSS

vehicle.Locked = !vehicle.Locked;
was_vehicle_found = true;
client.SendChatMessage($" **Du hast das Auto {vehicle.Locked} !** ");

Dann gibt er mir nur true and flase an also nciht ganz wie es ich mir wünsche

LeonMrBonnie · 13. August 2019

Dann mach einfach ein if und Else, und sende jeweils die passende Nachricht.

Flumiggy[Scripting] · 13. August 2019

CSS

[Command ("lock")]
        public void CMD_LockVehicle(Client client)
        {
            bool was_vehicle_found = false;


            foreach(Vehicle vehicle in NAPI.Pools.GetAllVehicles())
            {


                if (vehicle.Position.DistanceTo2D(client.Position) <= 7)
                {
                   if (vehicle.Locked = !vehicle.Locked) 
                   {
                    was_vehicle_found = true;
                    client.SendChatMessage(" **Du hast das Auto geschlossen !** ");
                   }
                   else
                   {
                    was_vehicle_found = true;
                    client.SendChatMessage(" **Du hast das Auto geöffnet !** ");
                   }                
                }
                break;
            }


            if (was_vehicle_found)
                return;


             client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");


        }

Alles anzeigen

Danke dir

Kaliber · 14. August 2019

Das break ist falsch gesetzt.

Das solltest du so schreiben:

C

foreach(Vehicle vehicle in NAPI.Pools.GetAllVehicles())
            {
                if (vehicle.Position.DistanceTo2D(client.Position) <= 7)
                {
                   if (vehicle.Locked = !vehicle.Locked) 
                   {
                    was_vehicle_found = true;
                    client.SendChatMessage(" **Du hast das Auto geschlossen !** ");
                   }
                   else
                   {
                    was_vehicle_found = true;
                    client.SendChatMessage(" **Du hast das Auto geöffnet !** ");
                   }    
                   break;            
                }
            }

Alles anzeigen

EinfachMax · 14. August 2019

C

using System.Linq; // Falls nicht schon getan


[Command ("lock")]
public void CMD_LockVehicle(Client client)
{
	var vehicle = NAPI.Pools.GetAllVehicles().FirstOrDefault(x => x.Position.DistanceTo(client.Position) <= 7);


	if(vehicle == null)
	{
		client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");
		return;
	}


	vehicle.Locked = !vehicle.Locked;
	client.SendChatMessage(" **Du hast das Auto " + (vehicle.Locked ? "zugesperrt" : "aufgesperrt") + " !** ");	
}

Alles anzeigen

Mach es mit Linq und bisschen aufgeräumter, weiß nicht ob der Code so direkt funktioniert - ist einige Monate her das ich auf Rage was gemacht habe.

Gruß

Warren · 26. August 2019

Zitat von Max Jackson

C

using System.Linq; // Falls nicht schon getan


[Command ("lock")]
public void CMD_LockVehicle(Client client)
{
	var vehicle = NAPI.Pools.GetAllVehicles().FirstOrDefault(x => x.Position.DistanceTo(client.Position) <= 7);


	if(vehicle == null)
	{
		client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");
		return;
	}


	vehicle.Locked = !vehicle.Locked;
	client.SendChatMessage(" **Du hast das Auto " + (vehicle.Locked ? "zugesperrt" : "aufgesperrt") + " !** ");	
}

Alles anzeigen

Mach es mit Linq und bisschen aufgeräumter, weiß nicht ob der Code so direkt funktioniert - ist einige Monate her das ich auf Rage was gemacht habe.

Gruß

Aufgeräumt?

Code

using System.Linq; // Falls nicht schon getan
[Command ("lock")]
public void CMD_LockVehicle(Client client)
{
	var vehicle = NAPI.Pools.GetAllVehicles().FirstOrDefault(x => x.Position.DistanceTo(client.Position) <= 7);


	if(!vehicle)
		return client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");

	vehicle.Locked = !vehicle.Locked;
	client.SendChatMessage(" **Du hast das Auto " + (vehicle.Locked ? "zugesperrt" : "aufgesperrt") + " !** ");	
}

Alles anzeigen

EinfachMax · 26. August 2019

Zitat von Warren

Aufgeräumt?

Code

using System.Linq; // Falls nicht schon getan
[Command ("lock")]
public void CMD_LockVehicle(Client client)
{
	var vehicle = NAPI.Pools.GetAllVehicles().FirstOrDefault(x => x.Position.DistanceTo(client.Position) <= 7);


	if(!vehicle)
		return client.SendChatMessage("~r~Es ist kein Auto in deiner Nähe.");
	vehicle.Locked = !vehicle.Locked;
	client.SendChatMessage(" **Du hast das Auto " + (vehicle.Locked ? "zugesperrt" : "aufgesperrt") + " !** ");	
}

Alles anzeigen

Aufgeräumt und einfach Zeilen weglassen ist ein gewaltiger Unterschied. Oftmals ist es viel aufgeräumter die geschweiften Klammern zu setzen, als ohne. (Thema Übersicht)
Den Zweck von deinem Post habe ich aber trotzdem nicht verstanden, zum einen trägt er nicht zum Thema bei und zum anderen scheint es so als hättest du noch nie wirklich für RAGE programmiert.

In PAWN kannst du ein "SendCientMessage" einfach nach einem return schreiben, das geht bei C#/ RAGE nicht, bleib bei PAWN wenn du bisschen Klugscheißen willst.

Vehicle Command /Lock

dennismitzwein 9. November 2020

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

CreatedVehicle und VehicleOwner nicht im Kontext da.

kann nicht connecten