Scheine an Fahrzeugen zuordnen und abfragen

Jameso · 29. März 2018

Hey,
Erst einmal habe ich ein kleines Problem:

Ich bin dabei ein Fahrschulsystem zu machen und habe erst einmal nur den Auto-Schein. Jetzt möchte ich wenn man die Fahrstunde startet das sozusagen der Spieler in das Fahrschulauto kann
Das habe ich auch hinbekommen, aber dann steht da das er kein Auto-Schein hat.

Code

if(newstate == PLAYER_STATE_DRIVER)
 	{
		for(new p; p < sizeof(Fahrschulauto); p++)
		{
		    new vehid=GetPlayerVehicleID(playerid);
			if(vehid == Fahrschulauto[p])
			{
				if(pInfo[playerid][pFahrstunde] != 1)
				{
					SendClientMessage(playerid, -1,"{c45461}Du musst zuerst die Fahrstunde starten, um mit der Prüfung anzufangen.");
					return RemovePlayerFromVehicle(playerid);
				}
			}
		}
	}
    new vehicleid=GetPlayerVehicleID(playerid);
	if(WithoutMotor(vehicleid))
	{
	    if(newstate == PLAYER_STATE_DRIVER)
	    if(pInfo[playerid][pAutoschein]== 0)
	    {
  			SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Auto-Schein.");
			return RemovePlayerFromVehicle(playerid);
		}
		else
		{
	    	SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");

Alles anzeigen

Hier ist auch ein Video dazu damit man es besser versteht:

Externer Inhalt www.youtube.com

Inhalte von externen Seiten werden ohne Ihre Zustimmung nicht automatisch geladen und angezeigt.

Durch die Aktivierung der externen Inhalte erklären Sie sich damit einverstanden, dass personenbezogene Daten an Drittplattformen übermittelt werden. Mehr Informationen dazu haben wir in unserer Datenschutzerklärung zur Verfügung gestellt.

Mein zweites Problem ist ich steige in jedes Auto und jedes hat sozusagen den Autoschein außer Fahrräder soll ich so etwas auch 3x kopieren und es in einer Gruppe einteilen und dann kompliziert immer abfragen? Also nach meiner Art währen das bestimmt 250 Zeilen

Code

stock WithoutMotor(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 510,509,481: return 0;
    }
    return 1;
}

Hoffe einer von euch hat einen Plan

Pr0x · 29. März 2018

if(pInfo[playerid][pAutoschein]== 0)
zu
if(pInfo[playerid][pAutoschein]== 0 && pInfo[playerid][pFahrstunde] == 0)

Jedoch könnte er in dem Fall , wenn er die Fahrstunde gestartet hat, jedes Fahrzeug benutzen.

Jameso · 29. März 2018

Ja genau das wollte ich ja am Anfang machen aber das wäre dann wieder ein "Bug" Und mein Ziel ist es ein Fehlerfreies Script auf den Stand zu setzten.
Man kann doch nach der Sache mit pFahrstunde== 1 irgendwo ein return einsetzten dann macht er nicht weiter mit der Abfrage oder?

Mogly1 · 30. März 2018

versuch es mal so...

new vehicleid=GetPlayerVehicleID(playerid);
if(WithoutMotor(vehicleid))
{
if(newstate == PLAYER_STATE_DRIVER)
{
for(new p; p < sizeof(Fahrschulauto); p++)
{
if(vehicleid == Fahrschulauto[p])
{
if(pInfo[playerid][pFahrstunde] != 1)
{
SendClientMessage(playerid, -1,"{c45461}Du musst zuerst die Fahrstunde starten, um mit der Prüfung anzufangen.");
return RemovePlayerFromVehicle(playerid);
}
else
{
return SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
}
}
}
if(pInfo[playerid][pAutoschein]== 0)
{
SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Auto-Schein.");
return RemovePlayerFromVehicle(playerid);
}
else
{
SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
}
}
}

Jameso · 30. März 2018

Danke dir! ♥
Jetzt hat ja jedes Fahrzeug einen Autoschein außer Fahrräder wie frage ich ab ob er den gewissen schein dafür hat, also mit z.b nur Autoschein würde ich das ja hinkriegen aber bei mehreren Sachen habe ich Probleme

Code

stock sAutos(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 510: return 0;
    }
    return 1;
}

Code

stock sMotorrad(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 481: return 0;
    }
    return 1;
}

Jeffry · 30. März 2018

if(sAutos(vehicleid))
{
if(pInfo[playerid][pAutoschein] == 0)
{
SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Auto-Schein.");
RemovePlayerFromVehicle(playerid);
}
else
{
SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
}
}
else if(sMotorrad(vehicleid))
{
if(pInfo[playerid][pMotorradschein] == 0)
{
SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Motorrad-Schein.");
RemovePlayerFromVehicle(playerid);
}
else
{
SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
}
}

Jameso · 30. März 2018

Er sagt mir bei jedem Fahrzeug das ich kein Autoschein habe, komisch

Code

if(sAuto(vehicleid))
			{
				if(pInfo[playerid][pAutoschein] == 0)
				{
					SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Auto-Schein.");
					RemovePlayerFromVehicle(playerid);
				}
				else
				{
					SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
				}
			}
			else if(sMotorrad(vehicleid))
			{
				if(pInfo[playerid][pMotorradschein] == 0)
				{
					SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Motorrad-Schein.");
					RemovePlayerFromVehicle(playerid);
				}
				else
				{
					SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
				}
			}
			else if(sBoot(vehicleid))
			{
				if(pInfo[playerid][pBootschein] == 0)
				{
					SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Boot-Schein.");
					RemovePlayerFromVehicle(playerid);
				}
				else
				{
					SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
				}
			}
   			else if(sLKW(vehicleid))
			{
				if(pInfo[playerid][pLKWschein] == 0)
				{
					SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen LKW-Schein.");
					RemovePlayerFromVehicle(playerid);
				}
				else
				{
					SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
				}
			}
			else if(sFlugzeug(vehicleid))
			{
				if(pInfo[playerid][pFlugschein] == 0)
				{
					SendClientMessage(playerid, -1,"{c45461}Du hast noch keinen Flug-Schein.");
					RemovePlayerFromVehicle(playerid);
				}
				else
				{
					SendClientMessage(playerid,-1,"{FAFAFA}Benutze den Befehl{009BFF} /Motor{FAFAFA} um den Motor zu starten. Mit den Befehl {009BFF}/Licht{FAFAFA} schaltest du das Licht ein.");
				}
			}


		}
	}

Alles anzeigen

Douq · 30. März 2018

Zeig mal deine Funktion sAuto

Jeffry · 30. März 2018

Sollte deine "sAutos"-Funktion (und auch die anderen) noch so sein, wie oben, dann vertausche return 1 und return 0.
Es darf nur am Ende ein "return 0" stehen, bei allen anderen Fällen (die dann ja positiv sind), muss return 1 stehen.
Bzw. false (0) und true (1).

Jameso · 30. März 2018

Da ist Return 0 und 1

Code

stock sMotorrad(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 461,463,468,471,521,522,523,581,586: return 0;
    }
    return 1;
}


stock sAuto(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 549,546,547,545,540,541,542,543,533,534,531,535,536,539,527,528,529,530,526,516,517,
			 500,518,502,503,504,505,506,507,489,490,491,492,494,495,496,482,483,485,
			 477,478,479,480,474,475,466,470,467,459,458,451,400,401,402,404,405,409,
			 410,411,412,413,415,418,419,420,421,422,424,426,429,434,436,438,439,440,
			 556,557,558,559,560,561,562,550,551,552,554,555,442,443,444,585,587,588,
			 580,582,583,579,575,576,574,445,565,566,567,568,571,572,589,596,597,
			 598,599,600,601,602,603,604,605: return 0;
    }
    return 1;
}


stock sLKW(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 532,524,525,514,515,508,499,498,486,403,406,407,408,414,416,423,427,
			 578,573,428,431,432,433,437,455,456,544,609: return 0;
    }
    return 1;
}


stock sFlugzeug(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 577,563,553,548,497,417,425,447,460,469,487,488,511,512,513,519,520,
			 592,593: return 0;
    }
    return 1;
}


stock sBoot(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 493,430,446,452,453,454,472,473,476,484,595: return 0;
    }
    return 1;
}

Alles anzeigen

Jeffry · 30. März 2018

Zitat von BrightLeaN

Da ist Return 0 und 1

Jeweils umdrehen, sonst klappt es nicht.
1 = true
0 = false

Du willst ja sagen, wenn das Fahrzeug ein Auto (etc) ist, dann ist alles ok (= true = wahr = 1).

Jameso · 2. April 2018

Entweder bin ich doof oder es funktioniert nicht

Code

stock sFlugzeug(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 577,563,553,548,497,417,425,447,460,469,487,488,511,512,513,519,520,592,
		593: return 1;
    }
    return 0;
}


stock sBoot(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 493,430,446,452,453,454,472,473,476,484,595: return 1;
    }
    return 0;
}

Alles anzeigen

Jeffry · 2. April 2018

Wenn die anderen Funktionen auch so aussehen, dann ist das korrekt und sollte auch funktionieren.
Lass dir ansonsten mal die Model-ID per print ausgeben, falls es nicht klappt, und prüfen dann, ob die vielleicht doch im switch drin steht.
Vor den if-Abfragen:
printf("vehicleid: %d / Model-ID: %d / sAuto: %d", vehicleid, GetVehicleModel(vehicleid), sAuto(vehicleid));

Jameso · 2. April 2018

//EDIT

Habe nochmal alles nachgeschaut eine Funktion habe ich vergessen geht doch danke!

Also erst einmal funktioniert alles gut nur das der Spieler ganz normal bei der Fahrstunde aussteigen kann was nicht der Fall sein soll, also wenn er das Auto verlässt will ich das die Fahrstunde gestoppt wird und das Auto wieder zum Spawn geschickt wird, wie soll ich anfangen bei onplayervehicleexit oder kann das noch hier mit rein? ( ist das erste mal das ich damit arbeite das jemand ein Fahrzeug verlässt )

Code

public OnPlayerEnterCheckpoint(playerid)
{
    switch(IsInCheckpoint(playerid,FS_Points<FAHRSCHUL_ROUTE>))
    {
        case FINAL_CHECKPOINT:
        {
            if(isPlayerInFSCar(playerid))
            {
                GameTextForPlayer(playerid, "~w~Fahrstunde\n~g~Bestanden", 5000, 3);
                SetVehicleToRespawn(GetPlayerVehicleID(playerid));
                ClearCheckpoint<FAHRSCHUL_ROUTE>(playerid);
				pInfo[playerid][aFahrstunde] =-1;
            }
            else SendClientMessage(playerid,-1,"{c45461}Du befindest dich in keinem Fahrschul-Auto.");
        }
        case NORMAL_CHECKPOINT:
        {
            if(isPlayerInFSCar(playerid))
            {
                NextCheckpoint(playerid, FS_Points<FAHRSCHUL_ROUTE>, 5.0);
            }
            else SendClientMessage(playerid,-1,"* Du bist in keinem FS Auto!");
	}
    }
    return 1;
}

Alles anzeigen

Jeffry · 2. April 2018

Frage das bei OnPlayerStateChange ab.
Wenn oldstate == PLAYER_STATE_DRIVER und der Spieler im Fahrstunde-Modus ist.

Jameso · 2. April 2018

Also, irgendwie verstehe ich es nicht warum oldstate nicht definiert ist?

Code

public OnPlayerExitVehicle(playerid, vehicleid)
{
	if(pInfo[playerid][aFahrstunde] == 1);
	{
		if(oldstate == PLAYER_STATE_DRIVER) return RemovePlayerFromVehicle(playerid);
	}
	return 1;
}

Code

(2012) : error 036: empty statement
(2014) : error 017: undefined symbol "oldstate"
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




2 Errors.

Jeffry · 2. April 2018

Zitat von BrightLeaN

OnPlayerExitVehicle

Zitat von Jeffry

OnPlayerStateChange

Zitat von BrightLeaN

if(pInfo[playerid][aFahrstunde] == 1);

Und hier das Semikolon entfernen.

Jameso · 2. April 2018

oh idiot sollte schlafen gehen haha danke

Jeffry · 2. April 2018

Kein Thema, kommt vor, manchmal sieht man den Wald vor lauter Bäumen nicht mehr. Kenne ich nur zu gut.

Beitrag von Harry. (29. März 2018)