Zaključano

PoÄetniÄke greÅ¡ke & Saveti za poÄetnike [ i neke "ozbiljne" "skriptere" ]

Započeo Makenzi

• pre 7 godina

2,053

pregleda

10

postova

Ova tema je zaključana. Samo moderatori i administratori mogu odgovarati.

Makenzi

Hustler

Postovi: 86

Teme: 4

Pridružio se: Nov 2018

pre 7 godina

#1

-= PoÄetniÄke greÅ¡ke & Saveti za poÄetnike. =-

Pozdrav svima,
u ovom tutorialu Ä‡u vas provesti kroz neke najÄeÅ¡Ä‡e greÅ¡ke koje na Äesto nailazim kada piÅ¡em u pdf-u "Razgovor u vezi PAWN - Skriptanja".

1. Brinete za bezopasne warninge, a one najbitnije propuÅ¡tate misleÄ‡i " ma to su samo warninzi, moÅ¾e da radi i sa njima, ne bi compile mode da neÅ¡to neÄ‡e da radi... ".
Na primer, ne brinete za warning

warning 225: unreachable code

Å¡to znaÄi da jedan deo skripte ispod nekog returna (primer) uopÅ¡te nije u funkciji i da tom na tom mestu ne treba return, a brinete se za warning

warning 217: loose indentation

koji je skroz bezopasan ( loose indentation - neki deo koda nije "uredan" tj. nije poravnat, npr 2 tab-a odvojen a ostalo su 1 tab odvojeni ).
2. Ponavljate istu funkciju kroz celu skriptu, a mogli ste samo na jednom mestu da uradite tu funckiju Å¡to automatski Å¡tedi vreme i optimizuje sam kod.
Nepravilno:

new g_Anti_CMD_Spam; //Sa ovim cemo proveravati komande tj. anti spam komandi...

public OnPlayerConnect(playerid)
{
	g_Anti_CMD_Spam = -1;
	return 1;
}

CMD:cmd1(playerid, params[])
{
	if(gettime() < g_Anti_CMD_Spam) return SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
	{
		g_Anti_CMD_Spam = gettime() + 3; //Povecava ukupno vreme (int) za 3 tj. 3 sekunde.
	}
	return 1;
}

CMD:cmd2(playerid, params[])
{
	if(gettime() < g_Anti_CMD_Spam) return SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
	{
		g_Anti_CMD_Spam = gettime() + 3; //Povecava ukupno vreme (int) za 3 tj. 3 sekunde.
	}
	return 1;
}

CMD:cmd3(playerid, params[])
{
	if(gettime() < g_Anti_CMD_Spam) return SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
	{
		g_Anti_CMD_Spam = gettime() + 3; //Povecava ukupno vreme (int) za 3 tj. 3 sekunde.
	}
	return 1;
}

CMD:cmd4(playerid, params[])
{
	if(gettime() < g_Anti_CMD_Spam) return SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
	{
		g_Anti_CMD_Spam = gettime() + 3; //Povecava ukupno vreme (int) za 3 tj. 3 sekunde.
	}
	return 1;
}
//...
CMD:cmd500(playerid, params[])
{
	if(gettime() < g_Anti_CMD_Spam) return SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
	{
		g_Anti_CMD_Spam = gettime() + 3; //Povecava ukupno vreme (int) za 3 tj. 3 sekunde.
	}
	return 1;
}

Jace, brze, bolje:

new g_Anti_CMD_Spam; //Sa ovim cemo proveravati komande tj. anti spam komandi...

public OnPlayerConnect(playerid) //Poziva se kada se igrac konektuje na server.
{
	g_Anti_CMD_Spam = -1;
	return 1;
}

public OnPlayerCommandReceived(playerid, cmd[], params[], flags) //Pawn.CMD
{
    if(gettime() < g_Anti_CMD_Spam)
    {
    	SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
        return 0;
    }
    return 1;
}

CMD:cmd1(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd2(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd3(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd4(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}
//...
CMD:cmd500(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

Ovo nije korisno samo u ovom sluÄaju, na primer neprofesionalni skripter Xunder proverava da li je igrac ulogovan na svakoj cmd a mogao je jednostavno staviti pod isti ovaj callback... ( razumem da zelite neke cmd za koje ne treba provera, zato postoji cmd[] da proverite koja se cmd izvrsava ako je potrebno u callbacku.
3. RaÄunaj!, Stani! Ne beÅ¾i, i ja obiÄno odma pobegnem Äim nanjuÅ¡im matematiku ali sada je drugaÄije!
ÄŒesta greÅ¡ka koju sam primetio je da kreirate 256 stringove za SendClientMessage, maksimalna duÅ¾ina poruke za SendClientMessage je 144 tako da koristite veliÄinu stringa 145 kada formatirate poruku!
I nemojte za boju u SendClientMessage stavljati -1 pa onda posle u poruci boju, veÄ‡ tamo stavite Hex boju jer se i ta boja koja na poÄetku stavite u poruci raÄuna kao string i automatski imate manje karaktera na raspolaganju!
4. Nemojte koristiti arraye tamo gde ne morate, evo zaÅ¡to:

new Float:pos;
GetPlayerPos(playerid, pos, pos, pos);

Compile speed test ( na 10 arraya ): 2444,2448,2473

new Float:x, Float:y, Float:z;
GetPlayerPos(playerid, x, y , z);

Compile speed test ( na 10 Non-Arraya ): 972,975,963
ZnaÄi duplo je brÅ¾e!
5. Nemojte koristiti for loop da prodjete kroz sve konektovane igraÄe nego koristite YSI y_iterate!
Ako pak ne Å¾elite da koristite y_iterate (foreach), ovo je najefikasniji loop da proÄ‘ete kroz sve konektovane igraÄe:

for (new slots = GetMaxPlayers(), i; i < slots; i++)
{
    if (!IsPlayerConnected(i))
        continue;
    //kod koji se izvrÅ¡ava za konektovane igraÄe
}

6. Ukoliko igraÄu Å¾elite poslati poruku i da ga kikujete, pozivanje kika morate odloÅ¾iti ili igraÄ neÄ‡e videti poruku, mislim da je to do delay-a izmeÄ‘u servera i klijenta ( ping + joÅ¡ neÅ¡to malo zbog skripte, ne znam kako da napiÅ¡em to ).
7. KoriÅ¡tenje streamer-a kada ne morate, na primer imate malo objekata ili pak podloge, ako nemate podlogu u CreateObjectu, velika je Å¡ansa da Ä‡e vozila prolaziti kroz nju jer Ä‡e ona biti uniÅ¡tena Äim igraÄ izaÄ‘e iz stream distance COD objekta.
8. Nemojte koristiti PAWNO ! Sada oÄekujem mnogo osuda, razumem da svako radi kako hoÄ‡e, no ovo je samo savet i objasnit Ä‡u vam detaljno sve (liÄno iskustvo)...
Na primer, imate pawno koji je veoma spor program i nema tzv. "Auto-Complete" f-ju, sada Ä‡ete misliti da sam lenj da kucam ceo kod? Ne, VI ste lenji...
U PAWNO je logiÄno da koristite skraÄ‡enice tipa SCM i SCMTA jer morate sve da piÅ¡ete dok na primer u Sublime Text 2/3 editoru ukucate SCM i automatski Ä‡e Vam se pojaviti SendClientMessage i samo stisnete enter i eto, cela f-ja...
ZaÅ¡to to?
KoriÅ¡tenje skraÄ‡enica za boje, skraÄ‡enice tipa SCM/SCMTA/SPD i tako dalje znatno usporavaju vreme compile-a dok u ST2/3 nema tih skraÄ‡enica a isto dobijete...
9. Ovo se suprostavlja svemu ovome Å¡to sam pisao jer ovo Ä‡e jako malo "usporiti" ( ispravite me ako greÅ¡im ) vreme kompajla moda, treba da koristite macroe tipa

#define ERROR(%0,%1)    SCMF(%0, 0xFF0000FF, "ERROR | {FFFFFF}"%1)

ZaÅ¡to? Postoji mnogo razloga, prvi je da je brÅ¾e nego da formatirate celu poruku ispoÄetka, drugo ne morate non-stop da piÅ¡ete iste boje, i treÄ‡e Å¡to lako moÅ¾ete editovati PREFIX iz npr "BOJA"[MOD:RP] Poruka u "BOJA"| "BOJA"MOD:RP "BOJA"| - Poruka
10. Koristite enum-e za definisanje dialoga.
Ne znam da li je ovo brÅ¾e, predpostavljam da jeste ( ne mora da znaÄi ), ali mnogo je olakÅ¡avajuÄ‡e, vi u jedan enum moÅ¾ete spakovati sve dialoge koje imate u modu a ne za svaki dialog da imate poseban BROJ npr #define DIALOG (356)...
Primer:

enum
{
    DIALOG_REGISTER,
    DIALOG_LOGIN
    //...
};

* Ova lista nije poreÄ‘ana po redu ( od najÄeÅ¡Ä‡ih ka onim retkim i sliÄno tome! ).
* Ukoliko ste uoÄili greÅ¡ku, ispravite mi ili pak poÅ¡aljite VaÅ¡e miÅ¡ljenje.
* Ova lista Ä‡e blagovremeno biti updateovana od strane mene uz VaÅ¡u pomoÄ‡ ( ili ako Å¡ta zanimljivo naÄ‘em )...!

Purdinho

OG Legend

Postovi: 3,313

Teme: 92

Pridružio se: Jun 2016

pre 7 godina

#2

Nisam sve citao jer to znam,ali sam popratio nesto hah 😛 Oke je sve tutorial 😛 samo sto ja malo drugacije radim unutar enuma dialoge,ali gotovo je isto , sve u svemu dobro je 😛

bunta

Kingpin

Postovi: 1,146

Teme: 38

Pridružio se: Feb 2010

pre 7 godina

#3

Ovo treba staviti na sticky

Roby Mapper

OG Legend

Postovi: 3,533

Teme: 129

Pridružio se: Jul 2016

pre 7 godina

#4

Lepo objasnjeno pohvale

// xunder

Godfather

Postovi: 2,044

Teme: 55

Pridružio se: Sep 2013

pre 7 godina

#5

A ko si ti Bgt?
Poznajemo se? haha Niko nema niti ce imati SA modove posle mog dolaska pa i ne moze da prica o nacinu pisanja koda.
Svakako to sto pises tu glupost da sam neprofesionalan, djela govore a ne rijeci.

Ae da vidimo kako si ti profesionalan?

Makenzi wrote on December 5, 2018, 9:08 pm:

Jace, brze, bolje:

new g_Anti_CMD_Spam; //Sa ovim cemo proveravati komande tj. anti spam komandi...

public OnPlayerConnect(playerid) //Poziva se kada se igrac konektuje na server.
{
	g_Anti_CMD_Spam = -1;
	return 1;
}

public OnPlayerCommandReceived(playerid, cmd[], params[], flags) //Pawn.CMD
{
    if(gettime() < g_Anti_CMD_Spam)
    {
    	SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
        return 0;
    }
    return 1;
}

CMD:cmd1(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd2(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd3(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

CMD:cmd4(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}
//...
CMD:cmd500(playerid, params[])
{
	g_Anti_CMD_Spam = gettime() + 3; //Potpuno isto kao gore samo smo proveru prebacili na callback koji se poziva svaki put kada igrac izvrsi komandu...
	return 1;
}

Ponavljas ovo u svakoj komandi ?

g_Anti_CMD_Spam = gettime() + 3;

Covjek napravio callback 'OnPlayerCommandPerformed', zar ti nije lakse da tu stavis ovo tvoje silno ponavljanje koda?

Makenzi wrote on December 5, 2018, 9:08 pm:

5. Nemojte koristiti for loop da prodjete kroz sve konektovane igraÄe nego koristite YSI y_iterate!
Ako pak ne Å¾elite da koristite y_iterate (foreach), ovo je najefikasniji loop da proÄ‘ete kroz sve konektovane igraÄe:

for (new slots = GetMaxPlayers(), i; i < slots; i++)
{
    if (!IsPlayerConnected(i))
        continue;
    //kod koji se izvrÅ¡ava za konektovane igraÄe
}

Ajde kada se vec pravis tako pametan mogao bi napisati i brzu petlju. GetMaxPlayes ? stvarno haha


for(new i = GetPlayerPoolSize(); i >= 0; i --) if (IsPlayerConnected(i)) {
//Kod
}

Da koriste enum za dialoge? To ti smatras greskom? Vau
Da koriste makro definicije kako bi ispisivali poruke?
Ne znas sta pricas druze


#define ERROR(%0,%1) SCM(%0, boja, "Error "%1)
[/code
Jesi mozda testirao sta ce biti sa ovim divnim kodom ako uradimo ovo

new buff;
format(buff, sizeof buff, "Ovo je random error %d", 12);
ERROR(playerid, buff);

Ocigledno da nisi tako da ovo tvoje palamudjenje u prazno ne pije vodu a kamo li da se moze nazvati "GRESKAMA" pocetnickim

Nzm sta imas biti toliko ljubomoran na mene, i ja sam covjek kao ti jedino sto ti ovde palamudis po random forumu za skriptere a ja radim na najboljoj gaming zajednici na Balkanu. ( nije velika razlika ha? 🙄 )

DeitY

Shot Caller

Postovi: 111

Teme: 4

Pridružio se: Aug 2011

pre 7 godina

#6

hahahah uterali ti xunder realno 😄

"na primer neprofesionalni skripter Xunder" ljubomora too much? od svih ljudi na Bsamp bas xundera nasao, eh sta ti je zivot, bas se vidi ljubomora da te izjeda ono "decay" fazon 😄

youngdeal

OG Legend

Postovi: 3,748

Teme: 73

Pridružio se: Jan 2012

pre 7 godina

#7

// xunder wrote on December 25, 2018, 4:05 pm:

A ko si ti Bgt?
Poznajemo se? haha Niko nema niti ce imati SA modove posle mog dolaska pa i ne moze da prica o nacinu pisanja koda.
Svakako to sto pises tu glupost da sam neprofesionalan, djela govore a ne rijeci.

Ae da vidimo kako si ti profesionalan?Ponavljas ovo u svakoj komandi ?

g_Anti_CMD_Spam = gettime() + 3;

Covjek napravio callback 'OnPlayerCommandPerformed', zar ti nije lakse da tu stavis ovo tvoje silno ponavljanje koda?

Ajde kada se vec pravis tako pametan mogao bi napisati i brzu petlju. GetMaxPlayes ? stvarno haha


for(new i = GetPlayerPoolSize(); i >= 0; i --) if (IsPlayerConnected(i)) {
//Kod
}

Da koriste enum za dialoge? To ti smatras greskom? Vau
Da koriste makro definicije kako bi ispisivali poruke?
Ne znas sta pricas druze


#define ERROR(%0,%1) SCM(%0, boja, "Error "%1)
[/code
Jesi mozda testirao sta ce biti sa ovim divnim kodom ako uradimo ovo

new buff;
format(buff, sizeof buff, "Ovo je random error %d", 12);
ERROR(playerid, buff);

Ocigledno da nisi tako da ovo tvoje palamudjenje u prazno ne pije vodu a kamo li da se moze nazvati "GRESKAMA" pocetnickim

Nzm sta imas biti toliko ljubomoran na mene, i ja sam covjek kao ti jedino sto ti ovde palamudis po random forumu za skriptere a ja radim na najboljoj gaming zajednici na Balkanu. ( nije velika razlika ha? 🙄 )

tooo + tacnije GetPlayerPoolSize() se ne mora ni koristit mislim jeste posluzna ali se i koristi isplayerconnected jer izbacuje 0 iako ima konektovan igrac 1 ( a i iako nema) "The highest playerid currently in use on the server or 0 if there are no connected players." , moze se napraviti npr po mom misljenju ovako

new Prijavljeni;
 // OnGameModeInit
 Prijavljeni = 0;
 // Kad se prijavi npr
 Prijavljeni++;
// Kad se disconnecta i ak je prijavljen
 if(Prijavljen == 1)
{
  Prijavljeni--;
}

i onda samo

 for(new i = 0; i < Prijavljeni; i++)
    {
		SendClientMessage(i,-1,"Test 123");
    }

dao sam samo primjer i taj primjer sam testao (radi),nemam nista protiv 😄
P.S. Edit sam post jer slucajno sam TAB stisnuo i postao pola teksta

youngdeal

OG Legend

Postovi: 3,748

Teme: 73

Pridružio se: Jan 2012

pre 7 godina

#8

i onda samo provjeru kod timera stavit npr

if(Prijavljeni == 0)
{
  print(" Timer za sve igrace nema funkcije trenutno, 0 prijavljenih igraca... ");
  return 1;
}

te moze se i sa GetPlayerPoolSize npr


forward ImaIgraca();
public ImaIgraca()
{
 new koliko = GetPlayerPoolSize();
 if(koliko==0&&!IsPlayerConnected(koliko))
 {
	return 0;
 }
  return 1;
}

if(!ImaIgraca())
{
  print(" Timer za sve igrace nema funkcije trenutno, 0 online igraca... ");
  return 1;
}
for(new i= 0 ,j = GetPlayerPoolSize(); i<= j; i++)
  	{
  		if(!Prijavljen) continue;
  		
	}

samo sam primjer dao,i moze se na vise nacina 😄
P.S. sry za dp

Terza Lazarevic

Underboss

Postovi: 731

Teme: 29

Pridružio se: Nov 2016

pre 7 godina

#9

Nemate vi veze sa optimizacijom i ovim budalastinama, izuzetak Rade koga ne poznajem mnogo. Tjt..

omcho

Enforcer

Postovi: 345

Teme: 8

Pridružio se: Oct 2017

pre 7 godina

#10

Hmm procitaj komentare


new g_Anti_CMD_Spam;

public OnPlayerConnect(playerid)
{
g_Anti_CMD_Spam = -1;
return 1;
}

public OnPlayerCommandReceived(playerid, cmd[], params[], flags)
{
    if(gettime() < g_Anti_CMD_Spam)
    {
    SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
        return 0;
    }
    return 1;
}

CMD:cmd1(playerid, params[])
{
g_Anti_CMD_Spam = gettime() + 3;
return 1;
// Zasto na komandi da dodjes ti opet ces morati da dodajes na svakoj komandi zar nije lakse pod ovim callback-om?

public OnPlayerCommandReceived(playerid, cmd[], params[], flags)
{
    if(gettime() < g_Anti_CMD_Spam)
    {
    SendClientMessage(playerid, -1, " Komande mozete koristiti svake 3 sekunde!");
        return 0;
    }
    g_Anti_CMD_Spam = gettime() + 3;
    return 1;
}

A ovo za Xundera i SA nemate sta da pricate ni za njega ni za Deity-a mogu da vas (nas) uce oko ovog pawn-a, nijma server radi 24/7 sa 1000 igraca dok vecina balkana ne moze ni 500 da podrzi kako treba tako da ono nemas sta da kazes za SA 🙂

EDIT: Evo i on ti je ukazo na ovoj gresci taj isti lik kojeg zoves neprofesionalanim

Morate biti prijavljeni da biste odgovorili na ovu temu.

Prijava

PoÄetniÄke greÅ¡ke & Saveti za poÄetnike [ i neke "ozbiljne" "skriptere" ]

PoÄetniÄke greÅ¡ke & Saveti za poÄetnike [ i neke "ozbiljne" "skriptere" ]