Loading TextDraw by tayloR

[_tayloR]

Poradi na namingu varijabli, u includeovima koristi static da se ne bi conflict sa ostalim skriptama.
Takodje ne trebaju ti 2 timera, mozes izbaciti onaj za destroy i sa malo matematike srediti sve u jednom.
Mozes i da izbjegnes 3 2d arraya koja sluze istoj svrsi

Kod [Izaberi] Proširi

PlayerText:LoadingType1[MAX_PLAYERS][7],
PlayerText:LoadingType2[MAX_PLAYERS][5],
PlayerText:LoadingType3[MAX_PLAYERS][7],

Povrijedjen sam 

Hvala brt na savjetima razmisljo sam o tome da stavim pod jedan 2d array ali reko hajd necu ako neko bude mijenjao TDove jer je Design za mene spansko selo, nece se poklapati krugovi na iste pozicije kao u ovom slucaju ali eto. Ne znam tacno kako si zamislio pod jedan tajmer moglo bi nekom varijablom i ponavljanjem tajmera ali kontam opet da ne ponavlja ga stalno i vrsi provjere. Tako sam ja zamislio mozda imas neko bolje resenje ali otom potom.

Mozda sutra ako me ne bude mrzilo da odradim to sve noviju verziju includea sa novim TD i mini optimizacijom.

[Maxwellâ„¢]

Galardo druze jako si iskompleksiran lik,takodje mi je zao sto nisi sposoban da uspes sa sampom.Ne zelim da te uvredim i nadam se da ovo ne shvatas kao uvredu,samo zelim da ti otvorim oci da izlecis komplekse,poz.

[_tayloR]

!!! UPDATED >> VERZIJA V2 !!!


>> Slike: https://imgur.com/a/yfVvXqQ
>> Code: https://pastebin.com/fep6T5tV


>> Izmjene:

- Novi TextDrawovi za sve tri verzije
- Uklonjena dva 2D Arraya
- Uklonjen jedan timer
- Bool _LoadingShow prebacen u integer
- Varijable dodane kao static

>> Credits - savjet i optimizacija: _xunder

[// xunder]

Kod [Izaberi] Proširi

if(playerid == INVALID_PLAYER_ID || second < 1 || type < 1 || type > 3 || _LoadingShow[playerid] > 0) return true;

Malo ko ce u playerid unijeti tacno INVALID_PLAYER_ID, veca mogucnost je da se desi npr da unese nekonektovanog igraca

Kod [Izaberi] Proširi

if(!IsPlayerConnected(playerid) || second < 1 || !(1 <= type <= 3) || _LoadingShow[playerid] > 0) return false;

Takodje bolje je da returnas false, razloga moze biti dosta da ne nabrajam.

Sto se tice promjena TD-a dobro bi bilo da Dots-e izdvojis u poseban array, tako ne moras brinitu o indexima u koje su spremljeni.

Kod [Izaberi] Proširi

static PlayerText: g_sLoadingDots[MAX_PLAYERS][3], PlayerText: g_sLoadingUI[MAX_PLAYERS][X];

Takodje mali trikovi:

Ovo sve

Kod [Izaberi] Proširi


if(freeze) TogglePlayerControllable(playerid, 0);
_LoadingCount[playerid] = 1; _LoadingShow[playerid] = second;
SetTimerEx("_Load", 1000, false, "ii", playerid, _LoadingCount[playerid]);
Mozes da napises ovako:

Kod [Izaberi] Proširi


if (freeze) TogglePlayerControllable(playerid, 0);
_LoadingShow[playerid] = second;
SetTimerEx("_Load", 1000, false, "ii", playerid, (_LoadingCount[playerid] = 1));


I npr:

Kod [Izaberi] Proširi


if((-- _LoadingShow[playerid]) > 0) {
....
new count = (++ _LoadingCount[playerid]);
if(count  > 3) _LoadingCount[playerid] = 1;
return SetTimerEx("_Load", 1000, false, "ii", playerid, count);
} else {
TogglePlayerControllable(playerid, 1);
_LoadingCount[playerid] = 0; _LoadingShow[playerid] = 0;
for(new i = 0; i < 10; i++) PlayerTextDrawHide(playerid,Loading[playerid][i]);
}


Evo verzije teze za ~5kb (i nije savrseno, radjeno 5 minuta): https://pastebin.com/FXHheRCE
Msm da je ovakav kod laksi za odrzavanje, a i imas 2 extra features (OnPlayerDisconnect kill timer, callback OnPlayerLoadingFinished)

Nadam se da ces pokupiti neke cake i iskoristiti ih dalje u radovima

[realplaya]

I mene boli briga za tvoj komentar i sto ti neces ostaviti komentar pohvale i sto trazis dlaku u jajetu. LP

Covek ti hoce pomoci, ne trazi dlaku u jajetu vec ti daje savete da ti kod bude bolji. Cemu to gubljenje?

[ᗪ卂乂]

dobra fora 

[№Grella]

I mene boli briga za tvoj komentar i sto ti neces ostaviti komentar pohvale i sto trazis dlaku u jajetu. LP

Samo ti je rekao ono sto je ispravnije, a ti sto se pronadjes uvredjen tvoja stvar

[_tayloR]

Covek ti hoce pomoci, ne trazi dlaku u jajetu vec ti daje savete da ti kod bude bolji. Cemu to gubljenje?

Idi kradi tudje skripte i daji sebi creditse pametniji si tad

dobra fora 

Hvala bruda

Samo ti je rekao ono sto je ispravnije, a ti sto se pronadjes uvredjen tvoja stvar

Kolumbo ne trebaju mi savjetnici i prevoditelji

[№Grella]

Bolesnikuuuuuuu

[Zivkovic]

[realplaya]

Idi kradi tudje skripte i daji sebi creditse pametniji si tad
Hvala bruda
Kolumbo ne trebaju mi savjetnici i prevoditelji

ne gubi se puno retarde mali, znas i sam da sam ja ono napravio...

I ne pise se "daji" nego "daj" nepismeni pederu..