[SAMMELTHREAD] Kleine Scripting Fragen

reaL. · 1. Juli 2016

Hey wo liegt mein Fehler es wird iwie nie
IsPlayerInRangeOfPoint geprintet obwohl ich direkt vor einem Car stehe

Code

stock IsPlayerInRangeOfJobVehicle(playerid)
{
	for(new i = 0; i < sizeof(Jobc); i++)
	{
		new Float:jfx,Float:jfy,Float:jfz;
		GetVehiclePos(Jobc[i][v_ID],jfx,jfy,jfz);
		printf("GetVehiclePos");
		printf("%d",i);
	    if(IsPlayerInRangeOfPoint(playerid,5,jfx,jfy,jfz))
	    {
	        printf("IsPlayerInRangeOfPoint");
	        return i;
	    }
	}
	return 0;
}

Alles anzeigen

dnlk · 1. Juli 2016

Zitat von reaL.

Hey wo liegt mein Fehler es wird iwie nie
IsPlayerInRangeOfPoint geprintet obwohl ich direkt vor einem Car stehe

Code

stock IsPlayerInRangeOfJobVehicle(playerid)
{
	for(new i = 0; i < sizeof(Jobc); i++)
	{
		new Float:jfx,Float:jfy,Float:jfz;
		GetVehiclePos(Jobc[i][v_ID],jfx,jfy,jfz);
		printf("GetVehiclePos");
		printf("%d",i);
	    if(IsPlayerInRangeOfPoint(playerid,5,jfx,jfy,jfz))
	    {
	        printf("IsPlayerInRangeOfPoint");
	        return i;
	    }
	}
	return 0;
}

Alles anzeigen

Überprüf mal ob Jobc[ i ][v_ID] wirklich eine Fahrzeug ID ist.

reaL. · 1. Juli 2016

Hab ich mal gemacht

Code

ocmd:jlock(playerid)
{
	new i = IsPlayerInRangeOfJobVehicle(playerid);
	printf("übergabe %d",i);
	if(i == 0) return SendClientMessage(playerid,ROT,"Du bist nicht an einem Jobfahrzeug.");
	if(sInfo[playerid][pJob] != Jobc[i][Jobid]) return SendClientMessage(playerid,ROT,"Du hast kein Schlüssel für dieses Fahrzeug.");
	return SendClientMessage(playerid,ROT,"In der Nähe und dein Car");
}


stock IsPlayerInRangeOfJobVehicle(playerid)
{
	for(new i = 0; i < sizeof(Jobc); i++)
	{
	    printf("Carid %d",Jobc[i][v_ID]);
		new Float:jfx,Float:jfy,Float:jfz;
		GetVehiclePos(Jobc[i][v_ID],jfx,jfy,jfz);
		printf("GetVehiclePos");
		printf("%d",i);
	    if(IsPlayerInRangeOfPoint(playerid,5,jfx,jfy,jfz))
	    {
	        printf("IsPlayerInRangeOfPoint");
	        return i;
	    }
	}
	return 0;
}

Alles anzeigen

Code

[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 0
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 1
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 2
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 3
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 4
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 5
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 6
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 7
[01/07/2016 16:57:24] Carid 1
[01/07/2016 16:57:24] GetVehiclePos
[01/07/2016 16:57:24] 8
[01/07/2016 16:57:24] übergabe 0

Alles anzeigen

Jeffry · 1. Juli 2016

Hast du mal versucht den Range etwas zu erhöhen, anstatt 5?

Ändere außerdem:
return 0;
zu:
return -1;
Sowie
if(i == 0) return SendClientMessage(playerid,ROT,"Du bist nicht an einem Jobfahrzeug.");
zu:
if(i == -1) return SendClientMessage(playerid,ROT,"Du bist nicht an einem Jobfahrzeug.");

Da es ja einen Index mit der 0 gibt.

reaL. · 1. Juli 2016

Ok
Habe den fehler beim erstellen gemacht
Funktioniert nun
Gibt es eine möglichkeit den Code besser zu schreiben ?

Jeffry · 1. Juli 2016

Zitat von reaL.

Gibt es eine möglichkeit den Code besser zu schreiben ?

new Float:jfx,Float:jfy,Float:jfz;
Schreibe das vor die Schleife, damit es nur einmal instanziiert wird.
Ansonsten passt es so.

JustMe.77 · 1. Juli 2016

Hallo, habe folgendes Problem. Egal ob der Spieler gespawnt ist oder nicht, die Nachricht das er nicht gespawnt ist erscheint immer.

CMD:duel(playerid, params[]) {
new weapid1, weapid2;

if(GetPlayerState(playerid) != PLAYER_STATE_SPAWNED)
{
SendClientMessage(playerid, COLOR_RED, "You need to spawn first!");
return 1;
}

if(Round[r_iniciando] == true) return SendClientMessage(playerid, COLOR_RED, "You can't use this command now!");
if(sscanf(params, "ii", weapid1, weapid2)) return SendClientMessage(playerid, COLOR_RED, "Use: /duel <gun id 1> <gun id 2>");
if(weapid1 < 22 || weapid1 > 34 || weapid2 < 22 || weapid2 > 34) return SendClientMessage(playerid, COLOR_RED, "Invalid weapon ID.");
if(InDuel[playerid] == true) return SendClientMessage(playerid, COLOR_RED, "You're already in a duel!");
if(Duel[playerid] != 998) return SendClientMessage(playerid, COLOR_RED, "You're already in a duel!");
if(CreatedDuel[playerid] == 1) return SendClientMessage(playerid, COLOR_RED, "You already created a duel, type /cancelduel to leave it.");

CreatedDuel[playerid] = 1;
Duel[playerid] = 999;

DuelWeapons[playerid][0] = weapid1;
DuelWeapons[playerid][1] = weapid2;

new dQuery[144];
format(dQuery, sizeof(dQuery), "{FF0000}(DUEL) {%06x}%s {FFFFFF}has created a duel (%s and %s) type {FF0000}/acceptduel %d", GetPlayerColor(playerid) >>> 8, GetName(playerid), GetWeaponNameEx(weapid1), GetWeaponNameEx(weapid2), playerid);
SendClientMessageToAll(COLOR_RED, dQuery);

return 1;
}

Was habe ich falsch gemacht?

dnlk · 1. Juli 2016

Zitat von [LnD]JustMe.77

Hallo, habe folgendes Problem. Egal ob der Spieler gespawnt ist oder nicht, die Nachricht das er nicht gespawnt ist erscheint immer.

CMD:duel(playerid, params[]) { new weapid1, weapid2; if(GetPlayerState(playerid) != PLAYER_STATE_SPAWNED) { SendClientMessage(playerid, COLOR_RED, "You need to spawn first!"); return 1; } if(Round[r_iniciando] == true) return SendClientMessage(playerid, COLOR_RED, "You can't use this command now!"); if(sscanf(params, "ii", weapid1, weapid2)) return SendClientMessage(playerid, COLOR_RED, "Use: /duel <gun id 1> <gun id 2>"); if(weapid1 < 22 || weapid1 > 34 || weapid2 < 22 || weapid2 > 34) return SendClientMessage(playerid, COLOR_RED, "Invalid weapon ID."); if(InDuel[playerid] == true) return SendClientMessage(playerid, COLOR_RED, "You're already in a duel!"); if(Duel[playerid] != 998) return SendClientMessage(playerid, COLOR_RED, "You're already in a duel!"); if(CreatedDuel[playerid] == 1) return SendClientMessage(playerid, COLOR_RED, "You already created a duel, type /cancelduel to leave it."); CreatedDuel[playerid] = 1; Duel[playerid] = 999; DuelWeapons[playerid][0] = weapid1; DuelWeapons[playerid][1] = weapid2; new dQuery[144]; format(dQuery, sizeof(dQuery), "{FF0000}(DUEL) {%06x}%s {FFFFFF}has created a duel (%s and %s) type {FF0000}/acceptduel %d", GetPlayerColor(playerid) >>> 8, GetName(playerid), GetWeaponNameEx(weapid1), GetWeaponNameEx(weapid2), playerid); SendClientMessageToAll(COLOR_RED, dQuery); return 1;}

Was habe ich falsch gemacht?

Wenn der Spieler gespawned ist , dann ist sein Status auf PLAYER_STATE_ONFOOT oder PLAYER_STATE_DRIVER bzw. PLAYER_STATE_PASSENGER
Keine Ahnung wofür PLAYER_STATE_SPAWNED existiert...

JustMe.77 · 1. Juli 2016

Alles klar, verwende dann PLAYER_STATE_ONFOOT, danke dir

dnlk · 1. Juli 2016

Zitat von [LnD]JustMe.77

Alles klar, verwende dann PLAYER_STATE_ONFOOT, danke dir

Mach es lieber so:

Code

if(GetPlayerState(playerid) < PLAYER_STATE_ONFOOT || GetPlayerState(playerid) > PLAYER_STATE_PASSENGER)
{
        SendClientMessage(playerid, COLOR_RED, "You need to spawn first!");
        return 1;
}

JustMe.77 · 1. Juli 2016

Ne, der Befehl soll auch nicht verfügbar sein während man Fahrer/Beifahrer ist, trotzdem danke

Edit:

Habe ein weiteres kleines Problem, hoffe ihr könnt mir behilflich sein. Mit folgendem Befehl gebe ich Spielern eine bestimmte Waffe, benutze ich anstatt den Waffen Namen die ID (beispielsweise 24), taucht folgendes in der ServerLog auf.

Spoiler anzeigen

CMD:givegun(playerid, params[]) {
new playerID, gun[24], cQuery[144];

if(pData[playerid][P_Admin] < 5) return
SendClientMessage(playerid, COLOR_RED, LV_Fail);

if(sscanf(params,"us[24]", playerID, gun)) return
SendClientMessage(playerid, COLOR_RED, "Use: /givegun <player> <weapon name>");

GivePlayerWeapon(playerID, GetWeaponIdByName(gun), 999);
new weapname = GetWeaponIdByName(gun);

if(IsSilent[playerid] == false) {
format(cQuery, sizeof(cQuery), "{%06x}%s gave you a weapon (%s)", GetPlayerColor(playerid) >>> 8, GetName(playerid), GetWeaponNameById(weapname));
SendClientMessage(playerID, -1, cQuery);
}
return 1;
}

Jeffry · 1. Juli 2016

GetWeaponIdByName gibt -1 zurück, daher geht es nicht.

Füge nach dem Auslesen hinzu:
if(weapname == -1) return SendClientMessage(playerid, 0xFF0000FF, "Error: Waffe existiert nicht.");

Setze außerdem das GivePlayerWeapon unter diese Nachricht.

#S. · 1. Juli 2016

Ist es möglich in der MySQL DB nur eine Spalte für eine einzige Zeile hinzuzufügen?

biztankeoil brauch logischerweise nur das Tankstellen Biz, die anderen nicht.

dnlk · 1. Juli 2016

Zitat von [LnD]JustMe.77

Ne, der Befehl soll auch nicht verfügbar sein während man Fahrer/Beifahrer ist, trotzdem danke

Edit:

Habe ein weiteres kleines Problem, hoffe ihr könnt mir behilflich sein. Mit folgendem Befehl gebe ich Spielern eine bestimmte Waffe, benutze ich anstatt den Waffen Namen die ID (beispielsweise 24), taucht folgendes in der ServerLog auf.

Spoiler anzeigen

sscanf warning: Format specifier does not match parameter count.
[debug] Run time error 4: "Array index out of bounds"
[debug] Accessing element at negative index -1
[debug] AMX backtrace:
[debug] #0 0009e8d8 in GetWeaponNameById (weaponid=-1) at C:\Users\David\Desktop\LnD\gamemodes\main.pwn:6446
[debug]
#1 0008c31c in public cmd_givegun (playerid=0, params[]=@00dc5d54 "0
24") at C:\Users\David\Desktop\LnD\gamemodes\main.pwn:4886
[debug] #2 native CallLocalFunction () from samp-server.exe
[debug]
#3 000238c0 in public OnPlayerCommandText (playerid=0,
cmdtext[]=@00dc5d1c "/givegun 0 24") at
C:\Users\David\Desktop\LnD\pawno\include\zcmd.inc:110

CMD:givegun(playerid, params[]) { new playerID, gun[24], cQuery[144]; if(pData[playerid][P_Admin] < 5) return SendClientMessage(playerid, COLOR_RED, LV_Fail); if(sscanf(params,"us[24]", playerID, gun)) return SendClientMessage(playerid, COLOR_RED, "Use: /givegun <player> <weapon name>"); GivePlayerWeapon(playerID, GetWeaponIdByName(gun), 999); new weapname = GetWeaponIdByName(gun); if(IsSilent[playerid] == false) { format(cQuery, sizeof(cQuery), "{%06x}%s gave you a weapon (%s)", GetPlayerColor(playerid) >>> 8, GetName(playerid), GetWeaponNameById(weapname)); SendClientMessage(playerID, -1, cQuery); } return 1;}

Alles anzeigen

Sofern du sscanf2 benutzt kannst du k<weapon> benutzen, macht's einfacher

Code

CMD:givegun(playerid, params[]) {
    new playerID,weaponid,cQuery[144];
    if(pData[playerid][P_Admin] < 5) return SendClientMessage(playerid, COLOR_RED, LV_Fail);
    if(sscanf(params,"uk<weapon>",playerID,weaponid)) return SendClientMessage(playerid,COLOR_RED,"Use: /givegun <player> <weapon name>");

    GivePlayerWeapon(playerID,weaponid,999);


    if(IsSilent[playerid] == false) {
        format(cQuery, sizeof(cQuery),"{%06x}%s gave you a weapon (%s)",GetPlayerColor(playerid) >>> 8,GetName(playerid),GetWeaponNameById(weaponid));
        SendClientMessage(playerID,-1,cQuery);
    }
    return 1;
}

Alles anzeigen

Jeffry · 1. Juli 2016

Zitat von #S.

Ist es möglich in der MySQL DB nur eine Spalte für eine einzige Zeile hinzuzufügen?

Nein, das geht nicht. Setze die, die es nicht haben, auf einen Default Wert von z.B. -1.

#S. · 1. Juli 2016

Alles klar, danke schön.
Wie kann ich es lösen, dass das Script sich automatisch alle anderen Werte erschließt?
z.B. habe ich eine Preisliste, die von 1.10 bis 2.10 gehen soll. Ich müsste ja jetzt im Array dann jede Zahl einzelnt schreiben, also quasi 1.11,1.12 usw. Gibt es da irgendeinen speziellen Befehl für?
1.10 ... 2.10 o.Ä.? Oder lässt sich das Problem irgendwie besser lösen. Ich möchte auch nachher aus dem Array random Preise ziehen, wie das geht weiß ich, allerdings vielleicht als Info nebenbei.

Jeffry · 1. Juli 2016

new Float:preis = float(random(101) + 110) / 100.0;

Das wäre dann ein Preis von 1,10€ bis 2,10€, der möglich wäre.

#S. · 1. Juli 2016

Da streikt mein Compiler. Er stürzt einfach ab.

Jeffry · 1. Juli 2016

Sicher, dass du es richtig gemacht hast?
Bei mir kompiliert der angegebene Code nämlich ohne Probleme.

Hast du es eventuell außerhalb eines Callbacks stehen? Dann würde der Absturz passieren. Es muss dort stehen, wo der Preis ausgegeben wird.

#S. · 1. Juli 2016

Aaah, alles klar. Danke dir.

//Edit:
Irgendwas passt da nicht, kriege immer extrem hohe Zahlen wie z.B. 1071393014. Habe einfach nur deine Funktion übernommen und hab mir das Ergebnis printen lassen. Ich überschreibe den Wert nirgends.