[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 28. Oktober 2016

Zitat von Jeffry

DestroyVehicle sollte ebenfalls gehookt werden.

Stimmt, das hab ich vergessen...

C

/* SA:MP Include GetVehicleNumberPlate
 *
 * (c) by Kaliber, 2016
 *
 */
#if !defined _samp_included
	#include <a_samp>
#endif
#if defined _samp_plate_ex
	#endinput
#endif
#define _samp_plate_ex
native __SetVehicleNumberPlate(vehicleid, numberplate[]) = SetVehicleNumberPlate;
native __DestroyVehicle(vehicleid) = DestroyVehicle;
stock _DestroyVehicle(vehicleid)
{
	if(existproperty(.value=50+vehicleid)) deleteproperty(.value=50+vehicleid);
	return __DestroyVehicle(vehicleid);
}
#if defined _ALS_DestroyVehicle
	#undef DestroyVehicle
#else
	#define _ALS_DestroyVehicle
#endif
#define DestroyVehicle _DestroyVehicle
stock _SetVehicleNumberPlate(vehicleid, numberplate[])
{
	if(strlen(numberplate) > 32) return 0; //Um Schaden zu vermeiden!
	setproperty(.value=50+vehicleid,.string=numberplate);
	return __SetVehicleNumberPlate(vehicleid, numberplate);
}
#if defined _ALS_SetVehicleNumberPlate
	#undef SetVehicleNumberPlate
#else
	#define _ALS_SetVehicleNumberPlate
#endif
#define SetVehicleNumberPlate _SetVehicleNumberPlate
/*
	native GetVehicleNumberPlate(vehicleid);
*/
stock GetVehicleNumberPlate(vehicleid)
{
	new plate[32];
	getproperty(.value=50+vehicleid,.string=plate);
	strunpack(plate,plate,sizeof(plate));
	return plate;
}

Alles anzeigen

Ne, das mit dem Standard setzen find ich bescheuert..weil ist ja eh immer XYZ1234 oder sowas..und man kann ja einfach prüfen ob die Funktion was leeres wiedergibt, dann setzt man halt das Standard Ding..ansonsten das Nummernschild, das wesentlich effizienter

Jeffry · 28. Oktober 2016

Zitat von Kaliber

Ne, das mit dem Standard setzen find ich bescheuert..weil ist ja eh immer XYZ1234 oder sowas..und man kann ja einfach prüfen ob die Funktion was leeres wiedergibt, dann setzt man halt das Standard Ding..ansonsten das Nummernschild, das wesentlich effizienter

Das macht aber nicht immer Sinn.
Angenommen in einem Dialog möchte ich alle Fahrzeuge mit deren Nummernschild anzeigen, habe aber nie SetVehicleNumberPlate genutzt. Dann gibt der Dialog nichts aus, oder, wenn man dort dann das Nummernschild setzt, ändert es sich weg vom Standard (wäre reiner Zufall das gleiche Schild zu bekommen).

CreateVehicle (etc) zu hooken und dort ein random Kennzeichen zu genererieren macht den Code nicht wirklich uneffizienter. Ob man das jetzt selbst im eigenen Code macht, oder es in der Include steht, spielt keine Rolle, nur dass letzteres weniger Aufwand für den User ist und ersterem gegenüber effizienter ist, da ein Funktionsaufruf weniger notwendig wird.

Um immer richtige Ergebnisse zu bekommen gehört das meiner Meinung nach dazu.

Kaliber · 28. Oktober 2016

Zitat von Jeffry

Das macht aber nicht immer Sinn.
Angenommen in einem Dialog möchte ich alle Fahrzeuge mit deren Nummernschild anzeigen, habe aber nie SetVehicleNumberPlate genutzt. Dann gibt der Dialog nichts aus, oder, wenn man dort dann das Nummernschild setzt, ändert es sich weg vom Standard (wäre reiner Zufall das gleiche Schild zu bekommen).

C

new string[512],plate[32];
for(new i=GetVehiclePoolSize(); i!=0; i--)
{
    format(plate,32,GetVehicleNumberPlate(i));
    format(string,sizeof(string),"%sID: %d | Plate: %s",string,i,((plate[0]==EOS)?("XYZ1234"):(plate)));
}
ShowPlayerDialog(..., string, ...);

So meinte ich das...
und da dieses Nummernschild anfangs immer das selbe ist..brauchste nichts hooken..

Jeffry · 28. Oktober 2016

Zitat von Kaliber

So meinte ich das...

Ich habe es grade mal überprüft. Das Standard-Plate ist tatsächlich immer das gleiche, und zwar: XYZSR998

Dann kannste es aber noch viel einfacher machen, und zwar beim CreateVehicle (etc) das Property einfach auf den Wert setzen.
Das ist wesentlich geschickter, als die Prüfung auf einen leeren String überall einzubauen.

Oder optional dennoch das Random-Plate mit dort einbauen (per define ein-/ausschaltbar), damit im eingeschalteten Fall nicht alle Fahrzeuge das gleiche Nummernschild haben.

Gut, muss beides nicht sein, aber den Standardwert zu setzen würde sicher nicht schaden.

Kaliber · 28. Oktober 2016

Zitat von Jeffry

aber den Standardwert zu setzen würde sicher nicht schaden.

Oki, hier dann:

C

/* SA:MP Include GetVehicleNumberPlate
 *
 * (c) by Kaliber, 2016
 *
 */
#if !defined _samp_included
	#include <a_samp>
#endif
#if defined _samp_plate_ex
	#endinput
#endif
#define _samp_plate_ex
native __SetVehicleNumberPlate(vehicleid, numberplate[]) = SetVehicleNumberPlate;
native __DestroyVehicle(vehicleid) = DestroyVehicle;
stock _DestroyVehicle(vehicleid)
{
	if(existproperty(.value=50+vehicleid)) deleteproperty(.value=50+vehicleid);
	return __DestroyVehicle(vehicleid);
}
#if defined _ALS_DestroyVehicle
	#undef DestroyVehicle
#else
	#define _ALS_DestroyVehicle
#endif
#define DestroyVehicle _DestroyVehicle
stock _SetVehicleNumberPlate(vehicleid, numberplate[])
{
	if(strlen(numberplate) > 32) return 0; //Um Schaden zu vermeiden!
	setproperty(.value=50+vehicleid,.string=numberplate);
	return __SetVehicleNumberPlate(vehicleid, numberplate);
}
#if defined _ALS_SetVehicleNumberPlate
	#undef SetVehicleNumberPlate
#else
	#define _ALS_SetVehicleNumberPlate
#endif
#define SetVehicleNumberPlate _SetVehicleNumberPlate
/*
	native GetVehicleNumberPlate(vehicleid);
*/
stock GetVehicleNumberPlate(vehicleid)
{
	new plate[32];
        if(existproperty(.value=50+vehicleid)) 
        {
	    getproperty(.value=50+vehicleid,.string=plate);
	    strunpack(plate,plate,sizeof(plate));
        }
        else plate="XYZSR998";
	return plate;
}

Alles anzeigen

Soo jetzt müsste alles gut sein

Jeffry · 28. Oktober 2016

Zitat von Kaliber

Soo jetzt müsste alles gut sein

Stimmt, so geht's auch, spart die Hooks wenn mans ohne random's macht.

Jetzt passt es

TimeLine · 28. Oktober 2016

er soll auch die account und die scheine auslesen die in einer anderen tabelle sind... kann mir da einer helfen ?

SELECT * FROM accounts, accounts_scheine WHERE userID;
//error
[SQL]SELECT * FROM accounts, accounts_scheine WHERE userID;
[Err] 1052 - Column 'userID' in where clause is ambiguous

Jeffry · 28. Oktober 2016

Nutze zwei Select Queries nacheinander im Code, einmal für die erste Tabelle und einmal für die zweite Tabelle.

Alternativ kannst du mit einem JOIN arbeiten, das kommt drauf an welche Daten in den Tabellen stehen, ob das Sinn macht.

TimeLine · 28. Oktober 2016

beim ucp sollen auch die scheine angezeigt werden von dem user mit der userid

Marschl · 29. Oktober 2016

Guten Morgen

Wie mache ich am besten eine TimeBan funktion?

Also dass ich später /tban [Spielername/ID] [Zeit] [Grund]

Wenn mir jemand helfen will kann er mich auch per PN kontaktieren

Canna · 29. Oktober 2016

indem du den aktuellen timestamp + die gebannte zeit in sekunden angibst und in der datenbank speicherst. Beim connevten/login fragst du einfach ab ob der aktuelle timestamp höher als der aus der Datenbank ist, wenn nicht ist er noch gebannt.

Ganz grob.

Marschl · 29. Oktober 2016

Zitat von Canna

indem du den aktuellen timestamp + die gebannte zeit in sekunden angibst und in der datenbank speicherst. Beim connevten/login fragst du einfach ab ob der aktuelle timestamp höher als der aus der Datenbank ist, wenn nicht ist er noch gebannt.

Ganz grob.

Könntest du mir da ggf. helfen?

//edit

wo ist mein Fehler?

stock FraktionName(frakid)
{
new var = frakid;
new frakname[256];
switch(var)
{
case 0:{frakname = "Bürger";}
case 1:{frakname = "San Andreas Police Department";}
}
return 1;
}

AirM4X · 29. Oktober 2016

Anstatt return 1 einfach return frakname.

Marschl · 29. Oktober 2016

Würde die Ladefunktion vom Spieler und er Fraktion so funktionieren?

Code

stock SpielerLaden(playerid)
{
    cache_get_value_name_int(0, "ID", sI[playerid][pID]);
	cache_get_value_name(0, "Passwort", sI[playerid][pPasswort], 65);
	cache_get_value_name_int(0, "Geschlecht", sI[playerid][pGeschlecht]);
	cache_get_value_name(0, "Alter", sI[playerid][pAlter], 11);
	cache_get_value_name(0, "EMail", sI[playerid][pEMail], 128);
	cache_get_value_name_int(0, "Level", sI[playerid][pLevel]);
	cache_get_value_name_int(0, "UpPoints", sI[playerid][pUpPoints]);
	cache_get_value_name_int(0, "Geld", sI[playerid][pGeld]);
	cache_get_value_name_int(0, "BankKonto", sI[playerid][pBankKonto]);
	cache_get_value_name_int(0, "BankGeld", sI[playerid][pBankGeld]);
	cache_get_value_name_int(0, "Fraktion", sI[playerid][pFraktion]);
	cache_get_value_name_int(0, "FraktionRang", sI[playerid][pFraktionRang]);
	cache_get_value_name_int(0, "Admin", sI[playerid][pAdmin]);
	cache_get_value_name_int(0, "Skin", sI[playerid][pSkin]);


	sI[playerid][pLoggedIn]  = true;
	return 1;
}


stock FraktionLaden()
{
    for(new i; i<MAX_FRAKTION; i++)
    {
        cache_get_value_name_int(0, "ID", fI[i][fID]);
    }
	return 1;
}

Alles anzeigen

TimeLine · 29. Oktober 2016

kann mir jemand dabei noch helfen -.-

Zitat von TimeLine

beim ucp sollen auch die scheine angezeigt werden von dem user mit der userid

Zitat von Jeffry

Nutze zwei Select Queries nacheinander im Code, einmal für die erste Tabelle und einmal für die zweite Tabelle.

Alternativ kannst du mit einem JOIN arbeiten, das kommt drauf an welche Daten in den Tabellen stehen, ob das Sinn macht.

Zitat von TimeLine

er soll auch die account und die scheine auslesen die in einer anderen tabelle sind... kann mir da einer helfen ?

SELECT * FROM accounts, accounts_scheine WHERE userID;

//error

[SQL]SELECT * FROM accounts, accounts_scheine WHERE userID;[Err] 1052 - Column 'userID' in where clause is ambiguous

Jeffry · 29. Oktober 2016

Hast du denn bereits einen Code, oder anhand von was kann dir geholfen werden?
Die Erklärung habe ich dir ja gegeben. brauchst du da noch weitere Hilfe, oder was benötigst du?

EDIT:
@TimeLine Super!!

TimeLine · 29. Oktober 2016

Zitat von Jeffry

Hast du denn bereits einen Code, oder anhand von was kann dir geholfen werden?
Die Erklärung habe ich dir ja gegeben. brauchst du da noch weitere Hilfe, oder was benötigst du?

danke Jeffry habs schon hinbekommen

BlackDown000003 · 29. Oktober 2016

wie kann ich machen das das

switch(Spieler[playerid][pAdmin])
{
case 1: format(string,sizeof(string),"Meldung: %s geht in den Probe Supporterdienst.",SpielerName(playerid));
case 2: format(string,sizeof(string),"Meldung: %s geht in den Supportdienst.",SpielerName(playerid));
case 3: format(string,sizeof(string),"Meldung: %s geht in den Support Leitungsdienst.",SpielerName(playerid));
case 4: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
case 5: format(string,sizeof(string),"Meldung: %s geht in den Super Admindienst.",SpielerName(playerid));
case 6: format(string,sizeof(string),"Meldung: %s geht in den CommunityManager Dienst.",SpielerName(playerid));
case 7: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
case 8: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
}

zu jedem spieler der auf den server is im chat steht

also das da halt im chat steht das der jenige in den admin dienst gegangen is und auch zb als Supporter in den dienst gegangen is

Jeffry · 29. Oktober 2016

Unter den switch:
SendClientMessageToAll(0xFFFFFFFF, string);

BlackDown000003 · 29. Oktober 2016

also so dann

switch(Spieler[playerid][pAdmin])
{
case 1: format(string,sizeof(string),"Meldung: %s geht in den Probe Supporterdienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 2: format(string,sizeof(string),"Meldung: %s geht in den Supportdienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 3: format(string,sizeof(string),"Meldung: %s geht in den Support Leitungsdienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 4: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 5: format(string,sizeof(string),"Meldung: %s geht in den Super Admindienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 6: format(string,sizeof(string),"Meldung: %s geht in den CommunityManager Dienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 7: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
case 8: format(string,sizeof(string),"Meldung: %s geht in den Admindienst.",SpielerName(playerid));
SendClientMessageToAll(0xFFFFFFFF, string);
}