[INCLUDE] Unique Random Number

Jameso · 21. Dezember 2018

Ja oke, dass ist ja so machbar aber ich möchte auch bei anderen Sachen eine Zahl generieren. Bei einem anderen Autohaus und Zulassungsstelle.

Wie mache ich das so das ich mehrere Sachen einbauen kann also das ich StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 0); öfters
benutzen kann.

irgendwie geht das nicht.

Code

if(dialogid == DIALOG_AUTOHAUS5) //ZULASSUNG JA LASTSCHRIFT
	{
		if(response)
		{
			new vid = GetPlayerVehicleID(playerid);
			for(new i = 0; i < sizeof(Autohausauto1); i++)
			{
	   			if(IsPlayerInVehicle(playerid, Autohausauto1[i]))
				{
					if(vid == Autohausauto1[i])
					{
					    if(pInfo[playerid][pBankgeld] < Autohausauto1_preis[i])
					    {
					        RemovePlayerFromVehicle(playerid);
	 						new string[100];
							format(string, sizeof(string), "{bc4343}Du brauchst $%s auf deinem Konto damit du dir den %s kaufen kannst.",sMoney(Autohausauto1_preis[i]),Autohausauto1_name[i]);
	    					SendClientMessage(playerid,-1,string);
						}
						else
						{
						    RemovePlayerFromVehicle(playerid);

							for(new d = 0; d < sizeof(PCar[]); d++)
							{
								//Prüfe, ob Fahrzeug existiert. Wenn nicht, nutze den Slot und lege das Fahrzeug an.
								SetPVarInt(playerid,"_tmp_veh",d);
								StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 0);
							}
						}
  	  				}
				}
			}
		}
		else
		{
		    RemovePlayerFromVehicle(playerid);
		}
		return 1;
	}

Alles anzeigen

Code

ublic OnRandomNumberGenerated(playerid, const pool, const token, const status)
{
	if(status == -1) return printf("![!]! Der Random Pool (%d), ist leer ![!]!",pool);
	switch(pool)
	{
	    case NUMBER_PLATE_POOL:
	    {
         	switch(token)
	        {
	            case 0:
	            {
					ApplyNumberPlate(playerid, status);
	            }
	        }
	    }
	}
	return 1;
}


stock ApplyNumberPlate(playerid, number) //number ist die eindeutig zufällige Zahl
{
	for(new i = 0; i < sizeof(Autohausauto1); i++)
	{
    	new d = GetPVarInt(playerid,"_tmp_veh"), SP = random(sizeof(SAHSpawn)), FB = random(127);
	    DeletePVar(playerid, "_tmp_veh");
	    PCar[playerid][d][P_KZ] = number;
	    PCar[playerid][d][P_uID] = pInfo[playerid][db_id];
	    PCar[playerid][d][P_X] = SAHSpawn[SP][0];
	    PCar[playerid][d][P_Y] = SAHSpawn[SP][1];
	    PCar[playerid][d][P_Z] = SAHSpawn[SP][2];
	    PCar[playerid][d][P_R] = SAHSpawn[SP][3];
	    PCar[playerid][d][P_C1] = FB;
	    PCar[playerid][d][P_C2] = FB;
	    PCar[playerid][d][P_MID] = GetVehicleModel(Autohausauto1[i]);
	    PCar[playerid][d][P_dbID] = pInfo[playerid][db_id];
	    PCar[playerid][d][P_VID] = CreateVehicle(PCar[playerid][d][P_MID],PCar[playerid][d][P_X] , PCar[playerid][d][P_Y], PCar[playerid][d][P_Z], PCar[playerid][d][P_R], PCar[playerid][d][P_C1], PCar[playerid][d][P_C2], -1);
	    new query[256];
	    mysql_format(handle, query, sizeof(query), "INSERT INTO personalcars (P_X, P_Y, P_Z, P_R, P_C1, P_C2, P_MID, P_uID) VALUES ('%f','%f','%f','%f','%d','%d','%d','%d')",PCar[playerid][d][P_X],PCar[playerid][d][P_Y],PCar[playerid][d][P_Z],PCar[playerid][d][P_R],PCar[playerid][d][P_C1],PCar[playerid][d][P_C2],PCar[playerid][d][P_MID],PCar[playerid][d][P_uID]);
	    mysql_pquery(handle,query);
	}
}

Alles anzeigen

Kaliber · 21. Dezember 2018

Zitat von BrightLeaN

öfters
benutzen kann.

Das habe ich doch auch erklärt, du änderst einfach den Token:

StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 1);

und dann kannst du im Callback:

C

//Im Callback OnRandomNumberGenerated
public OnRandomNumberGenerated(playerid, const pool, const token, const status)
{
	if(status == -1) return printf("Der Random Pool (%d), ist leer!",pool);
	switch(pool)
	{
	    case SMALL_EXAMPLE_POOL:
	    {
         	switch(token)
	        {
	            case 0: ApplyNumberPlate(playerid, status);
                    case 1: AnotherFunction(playerid, status); //<< wird dann nur aufgerufen bei token 1
	        }
	    }
	}
	return 1;
}

Alles anzeigen

//Edit:

Zitat von BrightLeaN

irgendwie geht das nicht.

Was machst du da auch?! Wieso änderst du den Code?
Warum löschst du einfach ein break und die if-Abfrage in der Schleife?!
Wieso packst du in die andere Funktion eine Schleife mit i die vorher gar nicht da war?!?!
Nutze dort einfach: GetPlayerVehicleID(playerid) anstatt Autohausauto1[i], du hast ja schon vorher gecheckt, ob er da wirklich drinnen sitzt.

So kann das natürlich nicht funktionieren...wenn da keine Code-Logik dahinter steht..

Nimm doch einfach den Code, wie ich ihn geschrieben hatte...

Jameso · 21. Dezember 2018

Zitat von Kaliber

Das habe ich doch auch erklärt, du änderst einfach den Token:

Verstanden.

Guck mal in der Zeile 6. da ist auch die Schleife und ich brauche diese ansonsten geht das nicht. (Der Code den ich vorher geschickt habe)
Und hier in Zeile 15. brauch ich das, hier fragt er ab welches Fahrzeug von den Austellungsfahrzeugen man kaufen will.
ich kann den Code nicht benutzen den du geschrieben hast weil i nicht definiert ist, deswegen die Schleife.
das break; habe ich aus Versehen mit gelöscht.

also die Schleife war auch schon vorher da beim Originalen

Code

stock ApplyNumberPlate(playerid, number) //number ist die eindeutig zufällige Zahl
{
	for(new i = 0; i < sizeof(Autohausauto1); i++)
	{
    	new d = GetPVarInt(playerid,"_tmp_veh"), SP = random(sizeof(SAHSpawn)), FB = random(127);
	    DeletePVar(playerid, "_tmp_veh");
	    PCar[playerid][d][P_KZ] = number;
	    PCar[playerid][d][P_uID] = pInfo[playerid][db_id];
	    PCar[playerid][d][P_X] = SAHSpawn[SP][0];
	    PCar[playerid][d][P_Y] = SAHSpawn[SP][1];
	    PCar[playerid][d][P_Z] = SAHSpawn[SP][2];
	    PCar[playerid][d][P_R] = SAHSpawn[SP][3];
	    PCar[playerid][d][P_C1] = FB;
	    PCar[playerid][d][P_C2] = FB;
	    PCar[playerid][d][P_MID] = GetVehicleModel(Autohausauto1[i]);
	    PCar[playerid][d][P_dbID] = pInfo[playerid][db_id];
	    PCar[playerid][d][P_VID] = CreateVehicle(PCar[playerid][d][P_MID],PCar[playerid][d][P_X] , PCar[playerid][d][P_Y], PCar[playerid][d][P_Z], PCar[playerid][d][P_R], PCar[playerid][d][P_C1], PCar[playerid][d][P_C2], -1);
	    new query[256];
	    mysql_format(handle, query, sizeof(query), "INSERT INTO personalcars (P_X, P_Y, P_Z, P_R, P_C1, P_C2, P_MID, P_uID) VALUES ('%f','%f','%f','%f','%d','%d','%d','%d')",PCar[playerid][d][P_X],PCar[playerid][d][P_Y],PCar[playerid][d][P_Z],PCar[playerid][d][P_R],PCar[playerid][d][P_C1],PCar[playerid][d][P_C2],PCar[playerid][d][P_MID],PCar[playerid][d][P_uID]);
	    mysql_pquery(handle,query);
	}
}

Alles anzeigen

Kaliber · 21. Dezember 2018

Zitat von BrightLeaN

ich kann den Code nicht benutzen den du geschrieben hast weil i nicht definiert ist, deswegen die Schleife.

Lösche die Schleife!

Zitat von Kaliber

Nutze dort einfach: GetPlayerVehicleID(playerid) anstatt Autohausauto1[i], du hast ja schon vorher gecheckt, ob er da wirklich drinnen sitzt.

Jameso · 21. Dezember 2018

Auto wird nicht gespawnt. Randomzahl auch nicht generiert soweit kommt es nicht.

Code

stock ApplyNumberPlate(playerid, number) //number ist die eindeutig zufällige Zahl
{
	new d = GetPVarInt(playerid,"_tmp_veh"), SP = random(sizeof(SAHSpawn)), FB = random(127);
	DeletePVar(playerid, "_tmp_veh");
	PCar[playerid][d][P_KZ] = number;
	PCar[playerid][d][P_uID] = pInfo[playerid][db_id];
	PCar[playerid][d][P_X] = SAHSpawn[SP][0];
	PCar[playerid][d][P_Y] = SAHSpawn[SP][1];
	PCar[playerid][d][P_Z] = SAHSpawn[SP][2];
	PCar[playerid][d][P_R] = SAHSpawn[SP][3];
	PCar[playerid][d][P_C1] = FB;
	PCar[playerid][d][P_C2] = FB;
	PCar[playerid][d][P_MID] = GetPlayerVehicleID(playerid);
	PCar[playerid][d][P_dbID] = pInfo[playerid][db_id];
	PCar[playerid][d][P_VID] = CreateVehicle(PCar[playerid][d][P_MID],PCar[playerid][d][P_X] , PCar[playerid][d][P_Y], PCar[playerid][d][P_Z], PCar[playerid][d][P_R], PCar[playerid][d][P_C1], PCar[playerid][d][P_C2], -1);
	new query[256];
	mysql_format(handle, query, sizeof(query), "INSERT INTO personalcars (P_X, P_Y, P_Z, P_R, P_C1, P_C2, P_MID, P_uID) VALUES ('%f','%f','%f','%f','%d','%d','%d','%d')",PCar[playerid][d][P_X],PCar[playerid][d][P_Y],PCar[playerid][d][P_Z],PCar[playerid][d][P_R],PCar[playerid][d][P_C1],PCar[playerid][d][P_C2],PCar[playerid][d][P_MID],PCar[playerid][d][P_uID]);
	mysql_pquery(handle,query);
}

Alles anzeigen

Code

for(new d = 0; d < sizeof(PCar[]); d++)
							{
								//Prüfe, ob Fahrzeug existiert. Wenn nicht, nutze den Slot und lege das Fahrzeug an.
								SetPVarInt(playerid,"_tmp_veh",d);
								StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 0);
								break;
							}

Kaliber · 21. Dezember 2018

Zitat von Kaliber

for(new d; d < sizeof(PCar[]); d++)
{
if(PCar[playerid][d][P_MID] == 0)
{
//Prüfe, ob Fahrzeug existiert. Wenn nicht, nutze den Slot und lege das Fahrzeug an.
SetPVarInt(playerid,"_tmp_veh",d);
StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 0);
break;
}
}

Alles anzeigen

Schau mal..so sieht meine Code aus..

...siehst du den Unterschied?

Bei dir ist keine if-Abfrage..nur ein break..was keinen Sinn macht...

//Edit:
Und sag mal willst du mich hier trollen oder so?!

Zitat von BrightLeaN

PCar[playerid][d][P_MID] = GetPlayerVehicleID(playerid);

Ich sagte, du sollst es ersetzen..und nicht das GetVehicleModel drum herum löschen...

Jameso · 21. Dezember 2018

Zitat von Kaliber

Und sag mal willst du mich hier trollen oder so?!

Nein

Hab alles hinbekommen danke dir und ich hab nicht getrollt
Ist das nicht ein "Sicherheit" Fehler. Man kann sich doch bestimmt mit Sobeit oder anderen Vorteilsprogrammen in einen anderen Fahrzeug teleportieren
GetVehicleModel(GetPlayerVehicleID(playerid)); dann wird ja das Model dort eingetragen (und ja ich weiß vorher wurde ja schon abgefragt)

Kann man das theoretisch machen?

Kaliber · 21. Dezember 2018

Zitat von BrightLeaN

Kann man das theoretisch machen?

Nein, da ja nur eine Millisekunde vergeht bis zur vorherigen Abfrage.

Und dort hast du ja geprüft, ob er wirklich in dem Fahrzeug ist oder nicht.

Das hatte ich auch oben geschrieben xD

Goldkiller · 14. Januar 2019

Unabhängig davon ob Jemand die SQLite Variante nutzt, sind da paar Memory leaks. (Z. 82, Z.90, Z. 100).
Wenn die INSERT-Statements ein Problem sind was die Laufzeit betrifft, dann würde ich mir mal Transactions ansehen ( Transaction, Begin, Commit, Autocommit mal als Keywords.)

Kaliber · 14. Januar 2019

Zitat von Goldkiller

sind da paar Memory leaks. (Z. 82, Z.90, Z. 100).

Z. 82 = Create Table
Z.90 = Insert
Z.100 = Offensichtlich Leer

Soweit ich das mal gelesen hatte, muss man da nichts clearen, weil da eh nichts existiert.

Zitat von Goldkiller

Wenn die INSERT-Statements ein Problem sind was die Laufzeit betrifft, dann würde ich mir mal Transactions ansehen

Ja, das gar keine schlechte Idee, schau ich mal, ob das die Performance verbessert, danke