Beiträge von Kaliber

Zitat von Nima.Yaghobi

wie finde ich die Koordinaten dafür ?

Mit einem TextDraw-Editor.

https://forum.sa-mp.com/showthread.php?t=543002

https://forum.sa-mp.com/showthread.php?p=3927120

https://forum.sa-mp.com/showthread.php?t=376758

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.

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:

//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;
}

//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...

Schreibst das dann einfach so:

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;
	}
}

Und der ganze Rest, kommt dann in das Callback, z.B. so:

//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);
	        }
	    }
	}
	return 1;
}

Und dann packst das einfach alles in die Funktion:

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] = 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);
}

Zu deiner 1. Frage mit dem Enter, schreib das mal so:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
	if(!IsPlayerInAnyVehicle(playerid) && (newkeys & KEY_SECONDARY_ATTACK))// Enter
	{
	    // Öffentliche Gebäude
	    if(GetPVarInt(playerid,"InBuilding")!=-1)
	    {
	        new e=GetPVarInt(playerid,"InBuilding");
     		if(IsPlayerInRangeOfPoint(playerid,3.0,Building[e][IX],Building[e][IY],Building[e][IZ]))
	    	{
		        SetPlayerPos(playerid,Building[e][EX],Building[e][EY],Building[e][EZ]);
	            SetPlayerInterior(playerid,0);
	            SetPlayerVirtualWorld(playerid,0);
	            SetPVarInt(playerid,"InBuilding",-1);
				return 1;
	        }
	    }
		else
		{
		    for(new e;e<sizeof (Building);e++)
		    {
		        if(IsPlayerInRangeOfPoint(playerid,3.0,Building[e][EX],Building[e][EY],Building[e][EZ]))
		        {
		            SetPlayerPos(playerid,Building[e][IX],Building[e][IY],Building[e][IZ]);
		            if(Building[e][Interior]!=0)SetPlayerInterior(playerid,Building[e][Interior]);
		            if(Building[e][Welt]!=0)SetPlayerVirtualWorld(playerid,Building[e][Welt]);
		            SetPVarInt(playerid,"InBuilding",e);
					return 1;
		        }
		    }
		}
		// Biz
	 	if(GetPVarInt(playerid,"InBiz")!=-1)
	    {
	        new b=GetPVarInt(playerid,"InBiz");
	        new bi=bInfo[b][bInterior];
    		if(IsPlayerInRangeOfPoint(playerid,3.0,BizInts[bi][IX],BizInts[bi][IY],BizInts[bi][IZ]))
	    	{
	    	    SetPlayerPos(playerid,bInfo[b][bX],bInfo[b][bY],bInfo[b][bZ]);
	            SetPlayerInterior(playerid,0);
	            SetPlayerVirtualWorld(playerid,0);
	            SetPVarInt(playerid,"InBiz",-1);
	            return 1;
	    	}
		}
		else
		{
			for(new b;b<sizeof(bInfo);b++)
		    {
		        printf("A: %d",b);
		        if(IsPlayerInRangeOfPoint(playerid,5.0,bInfo[b][bX],bInfo[b][bY],bInfo[b][bZ]))
		        {
		            if(bInfo[b][biztype]==3)return SendClientMessage(playerid,-1,"Diesen BizTyp kannst du nicht betreten!");
		            new bi=bInfo[b][bInterior];
		            SetPlayerPos(playerid,BizInts[bi][IX],BizInts[bi][IY],BizInts[bi][IZ]);
		            SetPlayerInterior(playerid,BizInts[bi][Interior]);
		            SetPlayerVirtualWorld(playerid,BizInts[bi][Welt]);
	                SetPVarInt(playerid,"InBiz",b);
                 	new string[250];
                 	format(string,sizeof(string),"Binfo Interior: %d | BizInfo Y: %f | BizInfo Interior: %d",bi,BizInts[bi][IY],BizInts[bi][Interior]);
                 	printf(string);
	                return 1;
		        }
	     	}
		}
	}
	return 1;
}

Zu deiner 2. Frage naja, was heißt Fehler, es ist halt komplett unnötig, du verwendest ja diese Dinge nicht.
Solltest du so schreiben:

stock const Float:SAHSpawn[][] =
{
	{770.2623,-1360.5071,13.4866,90.3064}, // 1
	{769.7510,-1364.0002,13.4859,89.2629}, // 2
	{769.6163,-1367.4791,13.4856,89.5874}, // 3
	{769.5392,-1370.8284,13.4845,89.2213}, // 4
	{769.7069,-1374.3871,13.4850,89.7537}, // 5
	{769.9338,-1378.0271,13.4861,89.0365}, // 6
	{769.6988,-1381.5538,13.4854,89.0018}, // 7
	{758.9918,-1381.4570,13.4671,270.7707}, // 8
	{758.7995,-1378.0237,13.4667,269.7128}, // 9
	{759.1549,-1374.4269,13.4650,268.9954}, // 10
	{758.9277,-1370.9075,13.4665,268.7696}, // 11
	{759.0135,-1367.4722,13.4659,269.2463}, // 12
	{758.9769,-1363.9332,13.4660,270.8068}, // 13
	{758.6721,-1360.5215,13.4664,269.2405} // 14
};

Diese 15 bezieht sich übrigens nicht auf die Anzahl der Einträge, sondern auf die Länge.
Sprich du hast X, Y, Z, A, aber da sagst du nein, du hast nicht 4, sondern 15. Der Ganze Rest ist halt 0.

Du hast nicht richtig gelesen.

Das startet nur die Generierung.

Du musst dass dann im Callback zuweisen.

//Da du noch d als Parameter hast, musst du den noch übergeben:
SetPVarInt(playerid,"_tmp_veh",d);
StartGenerateUniqueRandomNumber(playerid, handle, NUMBER_PLATE_POOL, 0);


//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:
	            {
                        new d = GetPVarInt(playerid,"_tmp_veh");
                        DeletePVar(playerid, "_tmp_veh");
			PCar[playerid][d][P_KZ] = status;
                        //Und dann hier eine Funktion aufrufen, wo es weiter gehen soll :)
	            }
	        }
	    }
	}
	return 1;
}

Das ist asynchron, so wie eben ein mysql Query.

Habe ich ja oben alles erklärt

Wo steht das?

Wie sieht denn deine Funktion aus zum Erstellen des Pools?

//Edit: Wie öffnest du auch die Datenbank? Das ist doch nicht phpmyadmin?

Es kann auch sein, dass dir nur 1000 angezeigt werden.
Kannst ja mal SELECT COUNT(*) FROM `t_0` machen, ich wette, da steht dann 8998.

Zitat von BrightLeaN

Habe es ja noch nicht mal benutzt und kriege die Handle fehler.

Der Fehler kommt nur, weil ich im Include auch zufällig den Namen db verwendet habe.

Nenne das einfach von new MySQL:db zu new MySQL:handle um.

und Verwende dann in deinem Skript überall handle, anstatt db.

Im Thread unter Beispiel, habe ich das erklärt

Im Spoiler, da kannst du das aufklappen.

Einfach dein handle angeben

Genau, du musst da kein handle neu deklarieren, du hast ja bereits eins, einfach das verwenden und übergeben

//Edit: Wenn es von Anfang an so heißt, dann nenn es um oder im Include

Zitat von Hybris

Für die Kennzeichen/Telefonnummer würde ich einfach aufsteigende Zahlenfolge nutzen.

Das freut mich

Zitat von Hybris

Selbst ein Index ist wahrscheinlich overkill, wenn wir von 1000 Zahlen reden, aber gut.

Ja, das versuche ich dir ja gerade zu vermitteln.
Desweiteren kannst du in Pawn nicht mal dynamisch Speicher allozieren, also würde das so gar nicht gehen.
Schon gar nicht bei mehreren Pools.
(Ja Memory Plugin außen vor, dennoch hat man dann noch O(n) Zugriffe auf das Array)

Zitat von Hybris

Wenn man für die Zufallszahl einen Index erstellt, dann ist der Check auf Duplikate relativ schnell

Wie willst du das machen?
Du musst ja trotzdem alle vorherigen Zahlen durchgehen, dein Ansatz ist der naive Ansatz, wie oben erklärt.
Und wenn du einen Pool von 10.000 Zahlen hast, dann viel Spaß. Dein Array musst du ja auch irgendwie deklarieren und das bezieht auch Speicher.

Zitat von Hybris

da es sonst zu viel Speicher frisst.

Nah jede Zahl frisst nur 4 Bytes.
Also selbst wenn du einen Pool aus 1.000.000 Zahlen hast, frisst das ((1.000.000*4)/1024) = 3.906,25kB = ~4MB

Also das ist ein RIESEN Pool, ich wüsste nicht, wofür man so viele eindeutig zufällige Zahlen bräuchte und man benötigt nur 4MB, was jeder Rechner locker zur Verfügung hat, immerhin Reden wir hier nicht von RAM, sondern von Festplattenspeicher.
Und wenn ich überlege, dass heute sogar Handys Teil schon 1TB haben (Galaxy S10 mit 1TB Speicher) sollte das mit den 4MB kein Problem sein

Hallo Brotfische,

ich bin es mal wieder *nerv nerv* mit einem neuen Include.
Inspiriert durch diesen Thread: Kennzeichen System / Polizei KFZ Akte

Es geht, wie der Titel es schon vermuten lässt um "Eindeutig zufällige Zahlen".

Aber was heißt das eigentlich?
Nun, wenn man z.B. die Funktion random(n) verwendet, dann erhält man zufällige Zahlen im Intervall von [0, n-1]. (Also zum Beispiel random(100) gibt uns die Zahlen von 0-99.)
So, verwenden wir diese Funktion jetzt noch einmal, dann erhalten wir mit Pech, die Selbe Zahl wie zuvor.
Das ist natürlich ärgerlich, wenn man jetzt Telefonnummern vergibt oder eindeutige Kennzeichen haben will.

Naive Lösung?
Eine naive Lösung wäre jetzt, eine Zahl zu generieren und dann zu speichern.
Anschließend alle Nachfolgenden generierten Zahlen durch die, die schon erstellt wurden zu iterieren und zu schauen, ob es bereits schon so eine Zahl gibt.
Falls dem so ist, eine neue generieren. Ansonsten hat man ja seine eindeutige Zahl gefunden.
Problem: Der Aufwand wird immer größer (je mehr Zahlen man generiert hat) und das will man gerade im Laufenden betrieb natürlich nicht. (O(n^x) n=Wie viele Zahlen es bereits gibt. x=Wie viele Versuche benötigt werden, eine zu finden, die noch nich existiert.)

Meine Lösung?
Es ist einfacher, einen Zahlen Pool abzubilden.
Sprich man will eine zufällige Zahl von 0-1000, dann speichert man einfach alle 1000 Zahlen in einer Datenbank.
Anschließend wird einfach eine zufällig ausgewählt und aus der Liste gelöscht (das heißt, je mehr verwendet werden, desto kleiner wird der Speicherplatz bedarf.)
Das bedeutet, egal wie viele Zahlen man hat, es ist immer gleich schnell (O(1)).

Das Gute hier ist noch, dass man Multi-Threaded querys nutzen kann (nur in MySQL, nicht SQLite), dann wird das eh ausgelagert alles und beeinflusst 0 den Server.

MySQL VS SQLite
Ich habe 2 Versionen geschrieben.
Definitiv zu präferieren wäre hier die MySQL Version (aber für die, die kein MySQL haben und das benötigen, habe ich eine SQLite Version geschrieben).

MySQL
Wie oben schon erwähnt, können wir hier Multi-Threaded Querys nutzen und das Asynchron gestalten.
Somit läuft hier alles sehr schnell und glatt.

SQLite
Hier belastet das die Server-Laufzeit. (Gerade der Server-Start, wenn die Pools inistialisiert werden, kann das (je nach Größe der Pools) bis zu einer Minute dauern.)
Die spätere Generierung dauert ~0.5ms. Also nicht soo viel, aber doch schon etwas, gerade wenn man hintereinander mehrere Zahlen generiert.

Beispiele
MySQL Version

enum //POOLS
{
	NUMBER_PLATE_POOL,
	SMALL_EXAMPLE_POOL
};

/*
	Die Parameter sehen so aus:
	CreateRandomPool(MySQL_Handle, POOL, MIN_VALUE, MAX_VALUE);
*/
CreateRandomPool(handle, NUMBER_PLATE_POOL, 0, 1000); //Erstellt einen Pool von 0-999
CreateRandomPool(handle, SMALL_EXAMPLE_POOL, 250, 500); //Erstellt einen Pool von 250-499

/*
	Die Parameter sehen so aus: (Wenn man playerid nicht nutzen will, kann man einfach 0 reinschreiben, ist nicht notwendig anzugeben)


	StartGenerateUniqueRandomNumber(playerid, MySQL_Handle, POOL, TOKEN);
*/
StartGenerateUniqueRandomNumber(playerid, handle, SMALL_EXAMPLE_POOL, 0);

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:
	            {
			printf("Zufällige Nummer %d wurde aus SMALL_EXAMPLE_POOL generiert.", status);
	            }
	        }
	    }
	}
	return 1;
}

SQLite Version

enum //POOLS
{
	NUMBER_PLATE_POOL,
	SMALL_EXAMPLE_POOL
};

/*
	Die Parameter sehen so aus:


	CreateRandomPool(POOL, MIN_VALUE, MAX_VALUE);
*/
CreateRandomPool(NUMBER_PLATE_POOL, 0, 1000); //Erstellt einen Pool von 0-999
CreateRandomPool(SMALL_EXAMPLE_POOL, 250, 500); //Erstellt einen Pool von 250-499

/*
	Die Parameter sehen so aus:


	UniqueRandom(POOL);
*/
new r = UniqueRandom(SMALL_EXAMPLE_POOL);
printf("Eindeutig zufällige Zahl (%d) wurde generiert.",r);

Was passiert, wenn der Pool leer ist?
Dann ist der status wie schon gezeigt -1.
Allerdings, wenn beim Server Restart immer noch das CreateRandomPool steht, wird dieser automatisch wieder aufgefüllt.

Download

Wer die MySQL Version verwenden will, da ist die Neuste MySQL Version (R41) nötig!

MySQL - Version - Links:

SQLite - Version - Links:

Wichtige Hinweise / Mögliche Errors

Schlusswort
Ich hoffe das Include ist für den ein oder anderen nützlich.

Bei Fragen oder Anregungen, stehe ich euch wie immer gerne zur Verfügung.

Viel Spaß weiterhin beim skripten

Mit freundlichen Grüßen
Euer Kalle

Zitat von iceberq

Bekomme immer noch denselben MySQL Fehler

Es handelt sich um ein Query, wo ein WHERE Statement steht.

Einfach:

if( strcmp( key , "BanReason" , true ) == 0 ) format(PlayerInfo[playerid][pBanReason], 64, ini_Get( Data )); //oder wie lang auch immer der Reason so sein kann
if( strcmp( key , "BanAdmin" , true ) == 0 ) format(PlayerInfo[playerid][pBanAdmin], MAX_PLAYER_NAME, ini_Get( Data ));

//Edit: Natürlich dini_get...

Zitat von BrightLeaN

Es gab doch dieses Timestamp?

new string[128], y, m, d, h, _m, s;
getdate(y, m, d), gettime(h, _m, s);
printf("%02d:%02d:%d %02d:%02dUhr", d, m, y, h,_m);

...da ist ein Komma vor der (

Zitat von BrightLeaN

Also wann die erstellt werden nachdem das Biz geladen wurde.

Nein, das ist ja egal, das Interior selbst existiert ja unabhängig von den Bizzen.

Zitat von BrightLeaN

Und wie soll ich die erstellen

Na ganz normal mit einer Schleife:

for(new i; i<sizeof(BizInts); i++)
{
    //Und hier einfach BizInts[i][IX],BizInts[i][IY],BizInts[i][IZ] verwenden :D
}

Zitat von BrightLeaN

Müsste das dann unter Bizladen hin?

Ich verstehe die Frage nicht lol.

BizInts ist eine globale Variable und muss ganz noch oben.

Zitat von BrightLeaN

if(bi==0) return SendClientMessage(playerid,-1,"Das Biz hat kein Interior");

Dann lösch diese Zeile.

Vermutlich, ist der 24/7 id 0.