[SAMMELTHREAD] Kleine Scripting Fragen

Hallo,

wollte mal fragen wie man das ohne disen komischen cmdtext idx ect schreiben kann den Befhl sieht so hässlich aus
"Und bugt um"

ocmd:setskin(playerid,params[])
{
	if(IsPlayerConnected(playerid))
 	{
 	    new tmp[126], cmdtext[126], idx, string[128];
		tmp = strtok(cmdtext, idx);
		if(!strlen(tmp))
		{
			SendClientMessage(playerid, COLOR_GRAD2, "{247BA6}* Benutze {009933}/setskin [SpielerID/Name] [Skin Model]");
			return 1;
		}
		new playa;
		new health;
		playa = ReturnUser(tmp);
		tmp = strtok(cmdtext, idx);
		health = strvalEx(tmp);
		if(PlayerInfo[playerid][pAdmin] >= 4)
		{
  			if(IsPlayerConnected(playa))
	    	{
      			if(playa != INVALID_PLAYER_ID)
	        	{
          			new name[20];
					new victim[20];
					GetPlayerName(playerid, name, sizeof(name));
					GetPlayerName(playa, victim, sizeof(victim));
					PlayerInfo[playa][pModel] = health;
					PlayerInfo[playa][pChar] = health;
					SetPlayerSkin(playa, PlayerInfo[playa][pChar]);
					format(string, 256, "%s hat den Skin vom %s zu %d geändert.",name, victim, health);
					ABroadCast(COLOR_RED,string,1347);
				}
			}
		}
		else
		{
			SendClientMessage(playerid, COLOR_GRAD1, "   Du darfst diesen CMD nicht nutzen,schäm dich!");
		}
	}
	return 1;
}

Stichwort: "sscanf".

Zitat von malE

Weil du den Motor dann ausschaltest!
Du startest den Motor automatisch, sobald du in ein Fahrzeug einsteigst, welches du in deinem Array (VehOhneMotor[]) definiert hast (siehe public OnPlayerStateChange(...)). Ich nehme mal an, dass du die ID des Fahrrads auch im Array definiert hast.

Hier ist alles dazu.

Eigentlich soll Ja gar nicht passieren wenn man auf ein Motorlosen Fahrzeug sitzt. Soweit geht es ja wenn ich Aufsteige kann ich normal fahren. Wenn ich /motor eingebe SOLLTE nichts passieren, jedoch geht der Motor aus.Wenn ich dann Absteige und wieder Aufsteige kann ich wieder normal fahren ohne das ich /machen müsste

So?

ocmd:motor(playerid,params[])
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return 1;
new vID=GetPlayerVehicleID(playerid);
if(!HatFahrzeugEinMotor(GetVehicleModel(vID)))return SendClientMessage(playerid, -1, "Dieses Fahrzeug hat kein Motor.");
new tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective;
GetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
if(tmp_motor==1)
{
tmp_motor =0;
}
else
{
tmp_motor=1;
}
SetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
return 1;
}

stock HatFahrzeugEinMotor(vehicleid)
{
for(new i=0; i < sizeof(VehOhneMotor); i++) {
if(vehicleid != VehOhneMotor[i]) continue;
return false;
}
return true;
}

Zitat von malE

Stichwort: "sscanf".

ocmd:setskin(playerid, params[])
{
	new menge, pID, string[128];
    if(PlayerInfo[playerid][pAdmin] < 1337) return SendClientMessage(playerid, COLOR_RED,"Du hast nicht genügend Rechte um diesen Befehl auszuführen.");
	if(sscanf(params, "us[30]i", pID,menge))
	{
	    SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {009933}/setskin [SpielerID/Name] [Skin Model]");
		return 1;
	}
	if(menge<1||menge>311)return SendClientMessage(playerid, COLOR_GREY, "Die Menge muss eine Zahl zwischen 1 und 350 betragen!");
	format(string, sizeof(string), "%s hat dein Skin auf %d gesetzt.", GetName(playerid), menge);
	SendClientMessage(pID, COLOR_LIGHTBLUE, string);
	format(string, sizeof(string), "Du hast den Skin von %s auf %d gesetzt.", GetName(pID), menge);
	SendClientMessage(playerid, COLOR_LIGHTBLUE, string);
	PlayerInfo[pID][pChar] = menge;
	SetPlayerSkin(pID, menge);
	return 1;
}

Habe ich jetzt gemacht wen ich /setskin 0 5 eingebe gibt er mir wider {247BA6}* Benutze {009933}/setskin [SpielerID/Name] [Skin Model] das raus warum?

Es reicht "ui" in sscanf brauchst das s[30] nicht.

Zitat von Kaliber

Es reicht "ui" in sscanf brauchst das s[30] nicht.

Dann verstehe ich das nicht warum das bei

ocmd:gov(playerid, params[])
{
	if(!IsPlayerConnected(playerid))return 1;
	new entry[96], string[128];
	if(!(PlayerInfo[playerid][pMember] == 1 || PlayerInfo[playerid][pLeader] == 1 || PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2 || PlayerInfo[playerid][pMember] == 11 || PlayerInfo[playerid][pLeader] == 11))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter!");
	if(PlayerInfo[playerid][pRank] < 5)return SendClientMessage(playerid, COLOR_GREY, "Für diesen Befehl musst du den Co-Leader Rang besitzen.");
	if(sscanf(params, "ui", entry))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {009933}/gov [Nachricht]");
	format(string, sizeof(string), "%s: %s", PlayerInfo[playerid][pRank], GetName(playerid), entry);
	SendClientMessageToAll(COLOR_DARKBLUE, string);
	return 1;
}

Nicht geht? Gibt er mir das auch wider zurück * Benutze /gov [Nachricht]

Zitat von Dr. Frauenarzt

Dann verstehe ich das nicht warum das bei

Code

ocmd:gov(playerid, params[])
{
	if(!IsPlayerConnected(playerid))return 1;
	new entry[96], string[128];
	if(!(PlayerInfo[playerid][pMember] == 1 || PlayerInfo[playerid][pLeader] == 1 || PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2 || PlayerInfo[playerid][pMember] == 11 || PlayerInfo[playerid][pLeader] == 11))return SendClientMessage(playerid, COLOR_GREY, "Du bist kein Beamter!");
	if(PlayerInfo[playerid][pRank] < 5)return SendClientMessage(playerid, COLOR_GREY, "Für diesen Befehl musst du den Co-Leader Rang besitzen.");
	if(sscanf(params, "ui", entry))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {009933}/gov [Nachricht]");
	format(string, sizeof(string), "%s: %s", PlayerInfo[playerid][pRank], GetName(playerid), entry);
	SendClientMessageToAll(COLOR_DARKBLUE, string);
	return 1;
}

Alles anzeigen

Nicht geht? Gibt er mir das auch wider zurück * Benutze /gov [Nachricht]

if(sscanf(params, "s[96]", entry))

Das u steht für user (username/userid) und das i steht für integer
In diesem fall brauchst du aber einen string, also s

C:\Users\Medion\Desktop\German Next Era Life - v2.5\gamemodes\GNE.pwn(44523) : error 028: invalid subscript (not an array or too many subscripts): "gOOC"
C:\Users\Medion\Desktop\German Next Era Life - v2.5\gamemodes\GNE.pwn(44523) : warning 215: expression has no effect
C:\Users\Medion\Desktop\German Next Era Life - v2.5\gamemodes\GNE.pwn(44523) : error 001: expected token: ";", but found "]"
C:\Users\Medion\Desktop\German Next Era Life - v2.5\gamemodes\GNE.pwn(44523) : error 029: invalid expression, assumed zero
C:\Users\Medion\Desktop\German Next Era Life - v2.5\gamemodes\GNE.pwn(44523) : fatal error 107: too many error messages on one line

ocmd:ooc(playerid, params[])
{
	new string[128], text[96];
	if(PlayerInfo[playerid][pLevel] < 2)return SendClientMessage(playerid, COLOR_GREY, "Du musst mindestens Level 2 sein.");
	if(sscanf(params, "s[96]", text))return SendClientMessage(playerid, COLOR_GREY, "{247BA6}* Benutze {009933}/(/o)oc [Nachricht]");
	if(PlayerInfo[playerid][pAdmin] > 0)
	{
		for(new i=0;i<GetMaxPlayers();i++)
		{
			if(pOOC[i] == 1)
			{
				format(string, sizeof(string), "(( %s: %s ))", GetName(playerid), text);
				SendClientMessage(i, COLOR_BLUE, string);
			}
		}
	}
	else
	{
		for(new i=0;i<GetMaxPlayers();i++)
		{
			if(gOOC[i] == 1) //44523
			{
				format(string, sizeof(string), "(( %s: %s ))", GetName(playerid), text);
				SendClientMessage(i, COLOR_WHITE, string);
			}
		}
	}
	return 1;
}

Hey,

Ich versuche gerade ein öffentlichen Chat zu machen aber scheinbar mag er nicht ;/

Wie hast du g00C deklariert?

Zitat von Jeffry

Wie hast du g00C deklariert?

new gOOC = 1;

new gOOC[MAX_PLAYERS] = 1;

new gOOC[MAX_PLAYERS] = {1, ...};

Das hinter dem Gleichzeichen steht für alle Indizes. Die drei Punkte sind hier korrekt, das bedeutet, dass es auf alle Indizes angewendet wird.

Zitat von malE

So?
ocmd:motor(playerid,params[]){ if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return 1; new vID=GetPlayerVehicleID(playerid); if(!HatFahrzeugEinMotor(GetVehicleModel(vID)))return SendClientMessage(playerid, -1, "Dieses Fahrzeug hat kein Motor."); new tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective; GetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective); if(tmp_motor==1) { tmp_motor =0; } else { tmp_motor=1; } SetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective); return 1;}stock HatFahrzeugEinMotor(vehicleid){ for(new i=0; i < sizeof(VehOhneMotor); i++) { if(vehicleid != VehOhneMotor[i]) continue; return false; } return true;}

Hat geklappt danke !

case DIALOG_MUSIC2:
		{
  			if(response)
  			{
     			switch(listitem)
     			{
        			case 0: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14200/;stream"); //GameTextForPlayer(playerid, "~g~Hands up", 4000, 3);
        			case 1: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14000/;stream"); //GameTextForPlayer(playerid, "~~Main - Charts", 4000, 3);
        			case 2: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14100/;stream"); //GameTextForPlayer(playerid, "~g~Rock", 4000, 3);
        			case 3: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14120/;stream"); //GameTextForPlayer(playerid, "~g~Disco Fox", 4000, 3);
    				case 4: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14060/;stream"); //GameTextForPlayer(playerid, "~g~Club", 4000, 3);
        			case 5: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14080/;stream"); //GameTextForPlayer(playerid, "~g~Harder", 4000, 3);
        			case 6: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14180/;stream"); //GameTextForPlayer(playerid, "~g~Black", 4000, 3);
        			case 7: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14020/;stream"); //GameTextForPlayer(playerid, "~g~Eurodance", 4000, 3);
        		}
    		}
		}

Hallo wen ich GameTextForPlayer darein mache kriege ich 26 ERRORS warum ?

Hey Leute,

Welchen Befehl brauch ich um ein einfachen /do zu machen?

Also das Wenn man den Motor startet, Automatisch für alle spieler im Fahrzeug oder in direkter nähe eine Nachricht bekommen z.B. PLAYER X dreht am Zündschloss und startet den Motor.

SendClientMessageToAll ? Mit Radius?

Würde der Befehl so stimmen Bis jetzt?

ocmd:motor(playerid,params[])
{
SetTimerEx("Motorstarten",1000,false,"i",playerid);
}

public Motorstarten(playerid)
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return 1;
new vID=GetPlayerVehicleID(playerid),tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective;
if(!HatVehEinMotor(GetVehicleModel(vID)))return 1;
GetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
if(tmp_motor==1)
{
tmp_motor =0;
format(string,sizeof(string), "((%s: stoppt den Motor und zieht die Handbremse an.))", getPlayerName(playerid;
for(new i=0; i<MAX_PLAYERS; i++)
{
SendClientMessage(i, FARBE_FUNK, string);
}
}
else
{
tmp_motor=1;
format(string,sizeof(string), "((%s: dreht den Zündschlüssel um und startet den Motor.))", getPlayerName(playerid;
for(new i=0; i<MAX_PLAYERS; i++)
{
SendClientMessage(i, FARBE_FUNK, string);
}
}
SetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
return 1;

}

Zitat von Dr. Frauenarzt

Code

case DIALOG_MUSIC2:
		{
  			if(response)
  			{
     			switch(listitem)
     			{
        			case 0: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14200/;stream"); //GameTextForPlayer(playerid, "~g~Hands up", 4000, 3);
        			case 1: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14000/;stream"); //GameTextForPlayer(playerid, "~~Main - Charts", 4000, 3);
        			case 2: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14100/;stream"); //GameTextForPlayer(playerid, "~g~Rock", 4000, 3);
        			case 3: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14120/;stream"); //GameTextForPlayer(playerid, "~g~Disco Fox", 4000, 3);
    				case 4: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14060/;stream"); //GameTextForPlayer(playerid, "~g~Club", 4000, 3);
        			case 5: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14080/;stream"); //GameTextForPlayer(playerid, "~g~Harder", 4000, 3);
        			case 6: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14180/;stream"); //GameTextForPlayer(playerid, "~g~Black", 4000, 3);
        			case 7: PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14020/;stream"); //GameTextForPlayer(playerid, "~g~Eurodance", 4000, 3);
        		}
    		}
		}

Alles anzeigen

Hallo wen ich GameTextForPlayer darein mache kriege ich 26 ERRORS warum ?

case 0:
{
PlayAudioStreamForPlayer(playerid, "http://s.sunset-radio.com:14200/;stream");
GameTextForPlayer(playerid, "~g~Hands up", 4000, 3);
}

Bei einer funktion kann man { & } weg lassen, aber bei mehreren brauchst du die dann wieder, vergiss das nicht.

Zitat von xPatrick

Hey Leute,

Welchen Befehl brauch ich um ein einfachen /do zu machen?

Also das Wenn man den Motor startet, Automatisch für alle spieler im Fahrzeug oder in direkter nähe eine Nachricht bekommen z.B. PLAYER X dreht am Zündschloss und startet den Motor.

SendClientMessageToAll ? Mit Radius?

Würde der Befehl so stimmen Bis jetzt?

ocmd:motor(playerid,params[]){SetTimerEx("Motorstarten",1000,false,"i",playerid);}public Motorstarten(playerid){if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return 1;new vID=GetPlayerVehicleID(playerid),tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective;if(!HatVehEinMotor(GetVehicleModel(vID)))return 1;GetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);if(tmp_motor==1){tmp_motor =0;format(string,sizeof(string), "((%s: stoppt den Motor und zieht die Handbremse an.))", getPlayerName(playerid;for(new i=0; i<MAX_PLAYERS; i++){SendClientMessage(i, FARBE_FUNK, string);}}else{tmp_motor=1;format(string,sizeof(string), "((%s: dreht den Zündschlüssel um und startet den Motor.))", getPlayerName(playerid;for(new i=0; i<MAX_PLAYERS; i++){SendClientMessage(i, FARBE_FUNK, string);}}SetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);return 1;}

Alles anzeigen

Es macht mehr Sinn eine Funktion dafür zu erstellen, zumal dadurch auch viele unnötige Zeilen gespart werden.
Außerdem hatte dein Code eins zwei Fehler, die ich zusätzlich behoben habe.

ocmd:motor(playerid,params[])
{
SetTimerEx("Motorstarten",1000,false,"i",playerid);
return 1;
}

public Motorstarten(playerid)
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return 1;
new vID=GetPlayerVehicleID(playerid);
if(!HatVehEinMotor(GetVehicleModel(vID)))return 1;

new tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective;
GetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
if(tmp_motor==1)
{
tmp_motor =0;
format(string,sizeof(string), "((%s: stoppt den Motor und zieht die Handbremse an.))", getPlayerName(playerid));
SendClientMessageInRangeOfPoint(playerid, FARBE_FUNK, string, 15.0);
}
else
{
tmp_motor=1;
format(string,sizeof(string), "((%s: dreht den Zündschlüssel um und startet den Motor.))", getPlayerName(playerid));
SendClientMessageInRangeOfPoint(playerid, FARBE_FUNK, string, 15.0);
}
SetVehicleParamsEx(vID,tmp_motor,tmp_licht,tmp_alarm,tmp_tuer,tmp_mhaube,tmp_kraum,tmp_objective);
return 1;
}

stock SendClientMessageInRangeOfPoint(playerid, color, msg[], Float:range)
{
new Float:X, Float:Y, Float:Z;
GetPlayerPos(playerid, X, Y, Z);

for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++) {
if(!IsPlayerConnected(i)) continue;
if(!IsPlayerInRangeOfPoint(i, range, X, Y, Z)) continue;
SendClientMessage(i, color, msg);
}
return 1;
}

Hey,

Ich habe eine frage ich wollte kennzeichen machen.

z.b er heißt Chris sollte kommen CH 147999 als kennzeichen z.b nicht Ch 147999 das die Anfangsbuchstaben groß sind

new playerName[MAX_PLAYER_NAME+1], nPlate[32], [24+1];
GetPlayerName(playerid, playerName, sizeof(playerName));
strmid(cutName, playerName, 0, 1, sizeof(cutName));
format(nPlate, sizeof(nPlate), "%s %d", cutName, PrivateVehicle[playerid][slot][vehID]);

Zitat von Dr. Frauenarzt

Hey,

Ich habe eine frage ich wollte kennzeichen machen.

z.b er heißt Chris sollte kommen CH 147999 als kennzeichen z.b nicht Ch 147999 das die Anfangsbuchstaben groß sind

new playerName[MAX_PLAYER_NAME+1], nPlate[32], [24+1];GetPlayerName(playerid, playerName, sizeof(playerName));strmid(cutName, playerName, 0, 1, sizeof(cutName));format(nPlate, sizeof(nPlate), "%s %d", cutName, PrivateVehicle[playerid][slot][vehID]);

Alles anzeigen

Probier es mal so:

new playerName[MAX_PLAYER_NAME+1], nPlate[32];
GetPlayerName(playerid, playerName, sizeof(playerName));
format(nPlate, sizeof(nPlate), "%s%s %d", toupper(playerName[0]), toupper(playerName[1]), PrivateVehicle[playerid][slot][vehID]);