[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 15. März 2017

Zitat von Sub Royal

Habe ich da was falsch gemacht?

Also 2 Dinge:

1. 0 zu returnen ist nicht gut, da auch das 1. Element im Array mit 0 indexiert ist.

2. mit l und r gibst du die Array Positionen an und nicht die Elemente.

Also deine müsste z.B. so aussehen:

binsearch(MoneyPickups,pickupid,0,sizeof(MoneyPickups)-1);

xPatrick · 15. März 2017

Hallo, kann mir jemand verraten warum der 2 Sting nicht an gesendet wird? Im Spiel ist nur eine Leerzeile im Chat dann

ocmd:ooc(playerid,params[])
{
new string1[128],string2[128];
if(!IsPlayerGM(playerid,2)) return 1;
if (OOChat == 1)
{
format(string1,sizeof(string1), "%s %s hat den OOC Chat geschlossen!", GetGMRang(playerid),getPlayerName(playerid));
SendClientMessageToAll(FARBE_ROT,string2);
OOChat = 0;
}
else
{
OOChat = 1;
format(string2,sizeof(string2), "%s %s hat den OOC Chat geöffnen (/o )!", GetGMRang(playerid),getPlayerName(playerid));
SendClientMessageToAll(FARBE_ROT,string2);
}
return 1;
}

Kaliber · 15. März 2017

Zitat von xPatrick

Hallo, kann mir jemand verraten warum der 2 Sting nicht an gesendet wird? Im Spiel ist nur eine Leerzeile im Chat dann

Hier, probier mal folgendes:

C

ocmd:ooc(playerid,params[])
{
    if(!IsPlayerGM(playerid,2)) return 1;
    new string[128];
    format(string,sizeof(string),"%s %s hat den OOC Chat %s!", GetGMRang(playerid),getPlayerName(playerid),((OOChat)?("geschlossen"):("geöffnet (/o )")));
    SendClientMessageToAll(FARBE_ROT,string);
    OOChat = !OOChat;
    return 1;
}

Da sparste dir einiges

Sub Royal · 15. März 2017

1A !! Danke es funktioniert jetzt einwandfrei
An die Sache mit dem return 0 habe ich garnicht gedacht

Dachdecker · 15. März 2017

Zitat von Jeffry

new str[16], money = GetPlayerMoney(playerid);format(str, sizeof(str), "$%d", money);if(money >= 0){ for(new i = strlen(str)-3; i > 1; i-=3) strins(str, ".", i);}else{ for(new i = strlen(str)-3; i > 2; i-=3) strins(str, ".", i);}PlayerTextDrawSetString(playerid, pMoneyText[playerid], str);

sa-mp-044.png passt net ganz...

xPatrick · 15. März 2017

Zitat von Kaliber

Hier, probier mal folgendes:

C

ocmd:ooc(playerid,params[])
{
    if(!IsPlayerGM(playerid,2)) return 1;
    new string[128];
    format(string,sizeof(string),"%s %s hat den OOC Chat %s!", GetGMRang(playerid),getPlayerName(playerid),((OOChat)?("geschlossen"):("geöffnet (/o )")));
    SendClientMessageToAll(FARBE_ROT,string);
    OOChat = !OOChat;
    return 1;
}

Da sparste dir einiges

Hat funktioniert, danke.
Ich wusste gar nicht das man einen string so noch einmal teilen kann

Jeffry · 15. März 2017

Zitat von Dachdecker

passt net ganz...

Du hast das Dollar-Zeichen hinten hin gemacht. Das ist von der Schreibweise her nicht richtig.
Daher auch der Punkt an der falschen Stelle.

Wenn du es zwingend so haben willst, dann so:
new str[16], money = GetPlayerMoney(playerid);
format(str, sizeof(str), "%d$", money);
if(money >= 0)
{
for(new i = strlen(str)-4; i > 0; i-=3) strins(str, ".", i);
}
else
{
for(new i = strlen(str)-4; i > 1; i-=3) strins(str, ".", i);
}
PlayerTextDrawSetString(playerid, pMoneyText[playerid], str);

Sub Royal · 15. März 2017

Ich muss leider nochmal stoeren

C

stock GetPickupType(pickupid, &index)
{
	new ares = binarysearch(ActorPickups,pickupid,0,sizeof(ActorPickups)-1);
	if(ActorPickups[ares] == pickupid && ares > -1) return index=ares,ACTOR_TYPE;	
	new mres = binarysearch(MoneyPickups,pickupid,0,sizeof(MoneyPickups)-1);
	if(MoneyPickups[mres] == pickupid && mres > -1) return index=mres,MONEY_TYPE;
	return INVALID_PICKUP_TYPE;
}

Wenn ich das so schreibe, dann funktioniert das ganze nur fuer die Actor Pickups, das heisst beim Geld einsammeln kriegt der Spieler kein Geld

C

stock GetPickupType(pickupid, &index)
{
	new mres = binarysearch(MoneyPickups,pickupid,0,sizeof(MoneyPickups)-1);
	if(MoneyPickups[mres] == pickupid && mres > -1) return index=mres,MONEY_TYPE;
	new ares = binarysearch(ActorPickups,pickupid,0,sizeof(ActorPickups)-1);
	if(ActorPickups[ares] == pickupid && ares > -1) return index=ares,ACTOR_TYPE;	
	return INVALID_PICKUP_TYPE;
}

Wenn ich das so rum mache ist das genau umgedreht Money Pickups funktionieren, das heisst der Spieler kriegt Geld, jedoch funktionieren die ActorPickups nicht
Kann ich das irgendwie mit nem if-else Gebilde loesen ?

Kaliber · 15. März 2017

Nein, das bedeutet dass du in beiden Arrays die selben IDs hast xD

Irgendwas stimmt da mit deiner Zuweisung nicht

Aber wie gesagt, eigentlich musst du Money pickups nicht extra definieren nur alle anderen

Jeffry · 15. März 2017

if(MoneyPickups[mres] == pickupid && mres > -1)
zu:
if(mres > -1 && MoneyPickups[mres] == pickupid)

Sonst wird das Array gesprengt, wenn mres -1 ist.

Gleiches bei der anderen Abrage mit ares.

Sub Royal · 15. März 2017

Zitat von Kaliber

Nein, das bedeutet dass du in beiden Arrays die selben IDs hast xD

Irgendwas stimmt da mit deiner Zuweisung nicht

Aber wie gesagt, eigentlich musst du Money pickups nicht extra definieren nur alle anderen

Ja das probiere ich mal
Mit der Zuweisugn hmm, habe da nix gefunden
Habe mal die Inhalte von dem kleineren Array im grossen gesucht, und das einzige wo es immer ein match gibt, ist bei Pickupid 0, die ist in beiden drin, keine Ahnung warum
Alle anderen sind nicht in den jeweils anderem Array enthalten

Jeffry · 15. März 2017

Zitat von Sub Royal

Habe mal die Inhalte von dem kleineren Array im grossen gesucht, und das einzige wo es immer ein match gibt, ist bei Pickupid 0, die ist in beiden drin, keine Ahnung warum

Eventuell leere Indizes.
Um Fehler mit der ID 0 zu vermeiden entweder alle Array-Indizes auf -1 setzen, oder ganz oben in OnGameModeInit ein Pickup im Nirgendwo erstellen, das hat dann die ID 0 und wird nie aufgerufen.

Zitat von Sub Royal

Alle anderen sind nicht in den jeweils anderem Array enthalten

Siehe Post #24.196.

Sub Royal · 15. März 2017

Zitat von Jeffry

Eventuell leere Indizes.Um Fehler mit der ID 0 zu vermeiden entweder alle Array-Indizes auf -1 setzen, oder ganz oben in OnGameModeInit ein Pickup im Nirgendwo erstellen, das hat dann die ID 0 und wird nie aufgerufen.

Siehe Post #24.196.

Danke dir, das hat mir schonmal geholfen die doppelten IDs zu verhindern
Habe jedoch immernoch das Problem wenn:

Den Teil aus Post #24.196 habe ich auch eingebaut.

Ich das hier habe

C

stock GetPickupType(pickupid, &index)
    {
    	new mres = binarysearch(MoneyPickups,pickupid,0,sizeof(MoneyPickups)-1);
    	if(MoneyPickups[mres] == pickupid && mres > -1) return index=mres,MONEY_TYPE;
    	new ares = binarysearch(ActorPickups,pickupid,0,sizeof(ActorPickups)-1);
    	if(ActorPickups[ares] == pickupid && ares > -1) return index=ares,ACTOR_TYPE;	
    	return INVALID_PICKUP_TYPE;
    }

Und dann ein ActorPickup einsammle, dann wird INVALID_PICKUP_TYPE returnt, weshalb weiss ich nicht.
Wenn ich ein MoneyPickup einsammle ist alles gut, dann funktioniert alles so wie es soll.
Ist alles umgedreht, also der Teil mit den ActorPickups oben, funktionieren die ActorPickups super, sammle ich jedoch ein MoneyPickup ein wird INVALID_PICKUP_TYPE returnt.

So als wuerde es mir scheinen dass nach dem ersten if statement der direkt dannach zu return INVALID_PICKUP_TYPE geht.

Aktuell benutze ich den Teil von Kaliber der prima funkt mit den beiden for Schleifen, mit der Suche waere aber die Kroenung

C

stock GetPickupType(pickupid, &index)
{
    for(new i; i<sizeof(money_pickups); i++)
    {
        if(money_pickups[i] == pickupid) return index=i,MONEY_TYPE;
    }
    for(new i; i<sizeof(actor_pickups); i++)
    {
        if(actor_pickups[i] == pickupid) return index=i,ACTOR_TYPE;
    }
    return INVALID_PICKUP_TYPE;
}

Alles anzeigen

Sind Queues eigentlich moeglich in PAWN?

Dachdecker · 16. März 2017

Zitat von Jeffry

Du hast das Dollar-Zeichen hinten hin gemacht. Das ist von der Schreibweise her nicht richtig.Daher auch der Punkt an der falschen Stelle.

Wenn du es zwingend so haben willst, dann so:
new str[16], money = GetPlayerMoney(playerid);format(str, sizeof(str), "%d$", money);if(money >= 0){ for(new i = strlen(str)-4; i > 0; i-=3) strins(str, ".", i);}else{ for(new i = strlen(str)-4; i > 1; i-=3) strins(str, ".", i);}PlayerTextDrawSetString(playerid, pMoneyText[playerid], str);

vielen Dank hat geklappt

Sub Royal · 16. März 2017

habe den Fehler dank dem englischen SAMP Forum gefunden

C

stock GetPickupType(pickupid, &index)
{
	new ares = binarysearch(ActorPickups,pickupid,0,sizeof(ActorPickups)-1);
	if(ares > -1 && MoneyPickups[ares] == pickupid) return index=ares,ACTOR_TYPE;
	new mres = binarysearch(MoneyPickups,pickupid,0,sizeof(MoneyPickups)-1);
	if(mres > -1 && MoneyPickups[mres] == pickupid) return index=mres,MONEY_TYPE;
	return INVALID_PICKUP_TYPE;
}

Habe faelschlicherweise auch unten MoneyPickups drin statt ActorPickups jetzt gehts einwandfrei

Dr. Frauenarzt · 16. März 2017

CMD:load(playerid, params[])
{
new string[128], int, vehicleid = GetPlayerVehicleID(playerid);
if(sscanf(params, "i", int))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {33AA33}/load [Menge]");
if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "** Du befindest dich in keinem Lieferwagen.");
if(!(Spieler[playerid][pFraktion] == 1))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter.");
//if(GetVehicleModel(vehicleid) == 428) return SendClientMessage(playerid, COLOR_RED,"Du bist in keinem Transporter.");
if(!IsPlayerInRangeOfPoint(playerid, 5.0, 2185.9954,-2262.5286,13.5721))
{
if(int < 1 || int > 50000)return SendClientMessage(playerid, COLOR_GREY, "Du kannst nur maximal 50000 Produkte aufladen.");
new money = int*4;
if(GetMoney(playerid) < money)return SendClientMessage(playerid, COLOR_GREY, ERROR_MONEY);
if((int+Vehicle[vehicleid][vehProds]) > 50000)return SendClientMessage(playerid, COLOR_GREY, "** Du kannst nur maximal 50000 Produkte aufladen.");
Vehicle[vehicleid][vehProds] += int;
GiveMoney(playerid, -money);
SendClientMessage(playerid, COLOR_GREEN,"Ladung: Materialien");
format(string, sizeof(string), "Menge: %d/50000", Vehicle[vehicleid][vehProds], money);
SendClientMessage(playerid, COLOR_GREEN, string);
}
return 1;
}
CMD:unload(playerid, params[])
{
new string[128], int, vehicleid = GetPlayerVehicleID(playerid);
if(sscanf(params, "i", int))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {33AA33}/unload [Menge]");
if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "** Du befindest dich in keinem Lieferwagen.");
if(!(Spieler[playerid][pFraktion] == 1))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter.");
//if(GetVehicleModel(vehicleid) == 428) return SendClientMessage(playerid, COLOR_RED,"Du bist in keinem Transporter.");
if(IsPlayerInRangeOfPoint(playerid, 5.0, 1563.6859,-1694.5925,5.8906))
{
if(Frak[1][frakEisen] > 50000)return SendClientMessage(playerid, COLOR_GREY, "** Das Lager ist voll.");
Frak[1][frakEisen] += (int*50);
Vehicle[vehicleid][vehProds] -= int;
format(string, sizeof(string), "{B4B5B7}Verbrecher mit /arrest einsperren\n\n{21DD00}Waffenkammer (LSPD)\n%d/250000\nMaterialien", Frak[1][frakEisen]);
Update3DTextLabelText(lspd3d, COLOR_LSPDSTOR, string);
SendClientMessage(playerid, COLOR_GREEN,"Gehalt: 300000$");
format(string, sizeof(string), "Materialien: %d/50000", Vehicle[vehicleid][vehProds]);
SendClientMessage(playerid, COLOR_GREEN, string);
Spieler[playerid][pBank] += 300000;
format(string, sizeof(string), "* Staatskasse hat dir 300000$ überwiesen. Betreff: Materialien abgeliefert (%d)", Vehicle[vehicleid][vehProds]);
SendClientMessage(playerid, COLOR_GREEN, string);
}
return 1;
}

Hallo, ich habe ein kleines problem wen ich die mats unten abgebe in der Garage (LSPD)
sieht das so aus obwohl es nur 50.000 Stück waren und 7 Stück drinne waren.

Sub Royal · 16. März 2017

Gibt es die Moeglichkeit die Streamdistance von Pickups einzustellen, bei CreatePickup? Oder geht das bei Streamern?

Pr0x · 16. März 2017

Zitat von Dr. Frauenarzt

Code

CMD:load(playerid, params[])
{
	new string[128], int, vehicleid = GetPlayerVehicleID(playerid);
	if(sscanf(params, "i", int))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {33AA33}/load [Menge]");
	if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "** Du befindest dich in keinem Lieferwagen.");
	if(!(Spieler[playerid][pFraktion] == 1))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter.");
	//if(GetVehicleModel(vehicleid) == 428) return SendClientMessage(playerid, COLOR_RED,"Du bist in keinem Transporter.");
	if(!IsPlayerInRangeOfPoint(playerid, 5.0, 2185.9954,-2262.5286,13.5721))
    {
        if(int < 1 || int > 50000)return SendClientMessage(playerid, COLOR_GREY, "Du kannst nur maximal 50000 Produkte aufladen.");
        new money = int*4;
        if(GetMoney(playerid) < money)return SendClientMessage(playerid, COLOR_GREY, ERROR_MONEY);
        if((int+Vehicle[vehicleid][vehProds]) > 50000)return SendClientMessage(playerid, COLOR_GREY, "** Du kannst nur maximal 50000 Produkte aufladen.");
		Vehicle[vehicleid][vehProds] += int;
		GiveMoney(playerid, -money);
		SendClientMessage(playerid, COLOR_GREEN,"Ladung: Materialien");
 		format(string, sizeof(string), "Menge: %d/50000", Vehicle[vehicleid][vehProds], money);
		SendClientMessage(playerid, COLOR_GREEN, string);
	}
	return 1;
}


CMD:unload(playerid, params[])
{
	new string[128], int, vehicleid = GetPlayerVehicleID(playerid);
	if(sscanf(params, "i", int))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {33AA33}/unload [Menge]");
	if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "** Du befindest dich in keinem Lieferwagen.");
	if(!(Spieler[playerid][pFraktion] == 1))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter.");
	//if(GetVehicleModel(vehicleid) == 428) return SendClientMessage(playerid, COLOR_RED,"Du bist in keinem Transporter.");
	if(IsPlayerInRangeOfPoint(playerid, 5.0, 1563.6859,-1694.5925,5.8906))
    {
		if(Frak[1][frakEisen] > 50000)return SendClientMessage(playerid, COLOR_GREY, "** Das Lager ist voll.");
		Frak[1][frakEisen] += (int*50);
		Vehicle[vehicleid][vehProds] -= int;
		format(string, sizeof(string), "{B4B5B7}Verbrecher mit /arrest einsperren\n\n{21DD00}Waffenkammer (LSPD)\n%d/250000\nMaterialien", Frak[1][frakEisen]);
		Update3DTextLabelText(lspd3d, COLOR_LSPDSTOR, string);
		SendClientMessage(playerid, COLOR_GREEN,"Gehalt: 300000$");
		format(string, sizeof(string), "Materialien: %d/50000", Vehicle[vehicleid][vehProds]);
		SendClientMessage(playerid, COLOR_GREEN, string);
		Spieler[playerid][pBank] += 300000;
		format(string, sizeof(string), "* Staatskasse hat dir 300000$ überwiesen. Betreff: Materialien abgeliefert (%d)", Vehicle[vehicleid][vehProds]);
		SendClientMessage(playerid, COLOR_GREEN, string);
	}
	return 1;
}

Alles anzeigen

Hallo, ich habe ein kleines problem wen ich die mats unten abgebe in der Garage (LSPD)
sieht das so aus obwohl es nur 50.000 Stück waren und 7 Stück drinne waren.

Du nimmst beim unload den int Wert x 50 und 50.000 x 50 sind nunmal 2.500.000

Dr. Frauenarzt · 16. März 2017

Zitat von Pr0x

Du nimmst beim unload den int Wert x 50 und 50.000 x 50 sind nunmal 2.500.000

also reicht es auf Frak[1][frakEisen] += (int*1); zu setzen?

JustMe.77 · 16. März 2017

Zitat von Sub Royal

Gibt es die Moeglichkeit die Streamdistance von Pickups einzustellen, bei CreatePickup? Oder geht das bei Streamern?

Ohne ist das nicht wirklich möglich, am besten einfach den Streamer verwenden, dann ist das problemlos möglich