AHK

Timer ausstellen oder eine Variable setzen.

global startjob := true
SetTimer, TEST, 100


TEST:
   if (startjob && IsPlayerDriver() == 1 && GetVehicleModelId() == 428)
   {
      startjob := false
      SendChat("/startjob")
   }
   else if (IsPlayerDriver() == 0)
      startjob := true


return

So in der Art, je nachdem wann du die Variable dann resetten willst.

So unnötig verschachtelt braucht man es nicht.

Zitat von eLias_

Code

global jobStarted := false


SetTimer, Test, 100


Test:
{
	if (IsPlayerInAnyVehicle()) {
		if (IsPlayerDriver()) {
			if (GetVehicleModelId() == 428) {
				if (!jobStarted) {
					SendChat("/startjob")
					jobStarted := true
				}
			} else {
				if (jobStarted) {
					jobStarted := false
				}
			}
		} else {
			if (jobStarted) {
				jobStarted := false
			}
		}
	} else {
		if (jobStarted) {
			jobStarted := false
		}
	}
}
return

Alles anzeigen

Du hast 3 mal den gleichen Codeblock. Zudem braucht man IsPlayerInAnyVehicle nicht, da IsPlayerDriver das schon impliziert. Und If-Abfragen unterstützen nicht umsonst Expressions, in denen man den &&-Operator nutzen kann.

Mir ist gerade noch aufgefallen, dass man bei der ersten Bedingung IsPlayerDriver auch nicht braucht, da GetVehicleModelId sowieso 0 zurückgibt, wenn man in keinem Fahrzeug sitzt.
Bei der zweiten Bedingung könnte man noch !startjob mit einfügen, damit IsPlayerDriver nicht unnötig häufig aufgerufen wird. Zudem würde ich ebenfalls aus Performance Gründen den Timer eher auf 500 setzen oder ansonsten 300 wenn einem das zu lange dauert:

global startjob := true
SetTimer, TEST, 500


TEST:
	if (startjob && GetVehicleModelId() == 428)
	{
		startjob := false
		SendChat("/startjob")
	}
	else if (!startjob && IsPlayerDriver() == 0)
		startjob := true
return