[Peppe951]

_All3Xz_, il 18 agosto 2010 - 18:50 ha detto:

IsPlayerInClan(playerid,clan[]);
non l'ho testato ma dovrebbe andare...
stock IsPlayerInClan(playerid,clan[])
{
new nome[24];
GetPlayerName(playerid,nome,sizeof(nome));
return strfind(nome,clan,true)==0 ? 1:0;
}

Qui l'operatore ternario è inutile :/
Avresti potuto fare semplicemente:

stock IsPlayerInClan(playerid,clan[])
{
    new nome[24];
    GetPlayerName(playerid,nome,sizeof(nome));
    return (!strfind(nome,clan,true));
}

[_All3Xz_]

si ma dopo ritorna 0 se trova la tag... cioè dopo è anche 'difficile' da usare

[Peppe951]

Veramente ritorna 1 se trova la tag e 0 se non la trova :/

[Peppinux aka Peppe_Stasu]

@Skiaffo: Grazie anche se era una cosa facilissima non ci ero arrivato

[Skiaffo]

Gli operatori sono inutili returna comunque 1 se trova e 0 se non trova

[Lama]

Skiaffo, il 19 agosto 2010 - 12:42 ha detto:

Gli operatori sono inutili returna comunque 1 se trova e 0 se non trova

returna 0 o più se lo trova, e -1 se non lo trova

[Peppe951]

Lama, il 19 agosto 2010 - 12:44 ha detto:

Skiaffo, il 19 agosto 2010 - 12:42 ha detto:

Gli operatori sono inutili returna comunque 1 se trova e 0 se non trova

returna 0 o più se lo trova, e -1 se non lo trova

Come ho fatto io no.

[Skiaffo]

Lama, il 19 agosto 2010 - 12:44 ha detto:

Skiaffo, il 19 agosto 2010 - 12:42 ha detto:

Gli operatori sono inutili returna comunque 1 se trova e 0 se non trova

returna 0 o più se lo trova, e -1 se non lo trova

In generale sì, returna l'offset del primo carattere che trova corrispondente (in questo caso 0 quindi) e -1 se non trova, ma non nel caso del codice di peppe

[Peppinux aka Peppe_Stasu]

GivePlayerWeapons(playerid, weaponid1 = 0, ammo1 = 0, weaponid2 = 0, ammo2 = 0, weaponid3 = 0, ammo3 = 0, weaponid4 = 0, ammo4 = 0, weaponid5 = 0, ammo5 = 0, weaponid6 = 0, ammo6 = 0, weaponid7 = 0, ammo7 = 0, weaponid8 = 0, ammo8 = 0, weaponid9 = 0, ammo9 = 0, weaponid10 = 0, ammo10 = 0, weaponid11 = 0, ammo11 = 0, weaponid12 = 0, ammo12 = 0)//By peppinux aka Peppe_Stasu
{
	GivePlayerWeapon(playerid,weaponid1,ammo1);
	GivePlayerWeapon(playerid,weaponid2,ammo2);
	GivePlayerWeapon(playerid,weaponid3,ammo3);
	GivePlayerWeapon(playerid,weaponid4,ammo4);
	GivePlayerWeapon(playerid,weaponid5,ammo5);
	GivePlayerWeapon(playerid,weaponid6,ammo6);
	GivePlayerWeapon(playerid,weaponid7,ammo7);
	GivePlayerWeapon(playerid,weaponid8,ammo8);
	GivePlayerWeapon(playerid,weaponid9,ammo9);
	GivePlayerWeapon(playerid,weaponid10,ammo10);
	GivePlayerWeapon(playerid,weaponid11,ammo11);
	GivePlayerWeapon(playerid,weaponid12,ammo12);
}

La funzione dice tutto... Se non la capite dite pure xD

Messaggio modificato da peppinux aka Peppe_Stasu il 23 agosto 2010 - 10:44

[Peppe951]

peppinux aka Peppe_Stasu, il 23 agosto 2010 - 11:44 ha detto:

GivePlayerWeapons(playerid, weaponid1 = 0, ammo1 = 0, weaponid2 = 0, ammo2 = 0, weaponid3 = 0, ammo3 = 0, weaponid4 = 0, ammo4 = 0, weaponid5 = 0, ammo5 = 0, weaponid6 = 0, ammo6 = 0, weaponid7 = 0, ammo7 = 0, weaponid8 = 0, ammo8 = 0, weaponid9 = 0, ammo9 = 0, weaponid10 = 0, ammo10 = 0, weaponid11 = 0, ammo11 = 0, weaponid12 = 0, ammo12 = 0)//By peppinux aka Peppe_Stasu
{
	GivePlayerWeapon(playerid,weaponid1,ammo1);
	GivePlayerWeapon(playerid,weaponid2,ammo2);
	GivePlayerWeapon(playerid,weaponid3,ammo3);
	GivePlayerWeapon(playerid,weaponid4,ammo4);
	GivePlayerWeapon(playerid,weaponid5,ammo5);
	GivePlayerWeapon(playerid,weaponid6,ammo6);
	GivePlayerWeapon(playerid,weaponid7,ammo7);
	GivePlayerWeapon(playerid,weaponid8,ammo8);
	GivePlayerWeapon(playerid,weaponid9,ammo9);
	GivePlayerWeapon(playerid,weaponid10,ammo10);
	GivePlayerWeapon(playerid,weaponid11,ammo11);
	GivePlayerWeapon(playerid,weaponid12,ammo12);
}

La funzione dice tutto... Se non la capite dite pure xD

Così è molto inefficiente, esegue 13 volte la funzione GivePlayerWeapon anche se si vogliono settare meno armi .-.

Io l'avrei fatta così:

GivePlayerWeapons(playerid, ...)
{
	new
		start = 1,
		num = numargs();
	for(new i = start; i < num; i++)
	{
		GivePlayerWeapon(playerid, getarg(i), getarg(i++));
	}
	return 1;
}

[Peppinux aka Peppe_Stasu]

xD Il tuo script infatti è molto complicato e non ci ho capito molto comunque hai ragione ma io l'ho saputo fare in quel modo e l'ho pubblicato xD...

[Stage6**]

_All3Xz_, il 13 agosto 2010 - 19:45 ha detto:

FORALLE (nientaltro che una ridefinizione del ciclo for,creata solo perchè più veloce e più breve da scrivere)
#define foralle(%1,%2) for(new %1=0; %1<%2; %1++)

TEST:

new primofori = GetTickCount();
 for(new i=0; i<MAX_PLAYERS;i++)
 {
		print("\n");
	}
	new primoforf = GetTickCount();
	new secondofori = GetTickCount();
	foralle(i,MAX_PLAYERS)
	{
 print("\n");
	}
	new secondoforf = GetTickCount();
	printf("\n\n\n\n\n");
	printf(" RISULTATI ");
	printf(" CICLO NORMALE %dms",primoforf-primofori);
	printf(" CICLO ALLE %dms",secondoforf-secondofori);

testato sul mio pc:
CICLO NORMALE 95ms
CICLO ALLE 25ms

Che nabbata, lo script inizia il timer, esegue il for normale e il timer non stacca, esegue il for tuo e segna i risultati del tuo che è partito dopo e logicamente e dimeno, quindi per avere il risultato preciso si dovrebbe fare:

new primofori = GetTickCount();
for(new i=0; i<MAX_PLAYERS;i++)
{
print("\n");
}
new primoforf = GetTickCount();
printf(" CICLO NORMALE %dms",primoforf-primofori);
new secondofori = GetTickCount();
foralle(i,MAX_PLAYERS)
{
print("\n");
}
new secondoforf = GetTickCount();
printf(" CICLO ALLE %dms",secondoforf-secondofori);

[_All3Xz_]

Vabbè, l'ho creato per abbreviare. Tutte le volte scrivere quella formula è lungo. Comunque "nabbata" te la puoi risparmiare.

[Peppe951]

_All3Xz_, il 24 agosto 2010 - 23:34 ha detto:

Vabbè, l'ho creato per abbreviare. Tutte le volte scrivere quella formula è lungo. Comunque "nabbata" te la puoi risparmiare.

Molto meglio usare il foreach, che ha un codice più breve e più veloce o.ò

Peppe951, il 08 marzo 2010 - 19:32 ha detto:

In cima:

#define SendClientMessage(%1,%2,%3,%4);  format(string, 128, %3, %4); SendClientMessage(%1, %2, string);
#define SendClientMessageToAll(%1,%2,%3);  format(string, 128, %2, %3); SendClientMessageToAll(%1, string);

Quando scrivete
SendClientMessage(playerid, colore, "testo");
oppure
SendClientMessageToAll(colore, "testo");
non cambia niente

però con questo define potete saltare il passaggio in cui scrivete
format(string, eccetera);
per inviare messaggi che cambiano in base alle variabili e quindi potete scrivere ad esempio
new string[128];
SendClientMessage(playerid, 0xFFFFFFFF, "OMG %d %s %d", 1, "asd", 2);
Che diciamo che vi fa risparmiare tempo ed è più semplice

ATTENZIONE: prima di questa funzione bisogna creare una stringa di nome string con
new string[128];
se l'avete già creata in quel punto non ricreatela

Funzione migliorata

new 
    SFM_tmpString[128];

#define SendClientMessage(%1,%2,%3,%4) \
format(SFM_tmpString, 128, %3, %4); \
SendClientMessage(%1, %2, SFM_tmpString)

#define SendClientMessageToAll(%1,%2,%3) \
format(SFM_tmpString, 128, %2, %3); \
SendClientMessageToAll(%1, SFM_tmpString)

Ora non c'è bisogno di creare una stringa prima di usare questa funzione, ne viene creata una globale automaticamente.

Testato e funzionante

[.ReVo^]

Peppe anche con la tua vecchia versione si poteva fare infatti io definivo la variabile in cima in modo da non definirla sempre .

[Peppe951]

.ReVo^, il 26 agosto 2010 - 11:31 ha detto:

Peppe anche con la tua vecchia versione si poteva fare infatti io definivo la variabile in cima in modo da non definirla sempre .

Si lo so ma così è già fatto per chi non saprebbe che si può definire in cima xD

Messaggio modificato da Peppe951 il 26 agosto 2010 - 10:56

[LaxC]

Ho provato a fare /colori con case, cioè setta i colori ad un player molto semplice, dovrebbe fungere.

if(strcmp(cmd, "/colori", true) == 0)
{
if (!strlen(cmdtext[8])) return SendClientMessage(playerid,0xEFEFF7AA,"[Error]Digita: /colore [1-7]");
switch (cmdtext[8])
{
case '1': SendClientMessage(playerid, 0xFFFFFFFF, "[HINT:]/rosso /blu /giallo /verde /bianco /nero");
case '2': SetPlayerColor(playerid, 0xFF0000FF);
case '3': SetPlayerColor(playerid, 0x33AA33AA);
case '4': SetPlayerColor(playerid, 0x0000FFFF);
case '5': SetPlayerColor(playerid, 0xFFFF00FF);
case '6': SetPlayerColor(playerid, 0x000000FF);
case '7': SetPlayerColor(playerid, 0xFFFFFFFF);
default: SendClientMessage(playerid,0xEFEFF7AA,"[Error]Digita: /colore [1-7]");
}
return 1;

Messaggio modificato da LaxC il 28 agosto 2010 - 23:17

[_All3Xz_]

SearchAdvertise by me, cerca in una stringa se uno spamma un ip di un server

ServerAdvertise(string[])
{
	new lol = 0;
        for(new i=0; i<=strlen(string);i++)
	{
	    if(string[i] == '.') lol++;
		if(lol == 3 && string[i] == ':')
		{
		    print("TROVATO ADVERTISE");
		    print(string);
		    return 1;
		}
	}
	return 0;
}

semplice uso:

OnPlayerText(playerid,text[])
{
if(ServerAdvertise(text))
{
return 0;
}
return 1;
}

Testato e funziona.

[Peppinux aka Peppe_Stasu]

Utile ma ti consiglio di aggiungere anche se la stringa è numerica perchè se un players scrive:"Ciao. Sonouno scripter. Ecco cosa ho scriptato: Nome Gamemode 1 eccetera" viene riconosciuto xD

[.ReVo^]

Dopo 3 punti controlla la : e lo blocca quindi non dovrebbe buggare.
if(lol == 3 && string[i] == ':')