Jedinstven sistem pecanja v2

[rane]

Tip: Filterskripta
Slike ili Video:

[/i]
Opis i dodatne informacije:
Changelog:
- Od sada imate mogucnost kupovine, postavljanja i koristenja rostilja (/postavirostlj, /rostiljaj)
- Od sada mozete ispeci ribu na rostilju, ukoliko ne skinete ribu sa rostilja na vrijeme ona ce da se prepece
- Od sada mozete jesti ispecenu ribu
- Kod optimizovan i uređen
- Fixani sitni bugovi
Download link: U deskripciji videa
Scan link: pastebin je
Credits: Sage za optimizaciju

[rane]

Novi yt link, nisam stigao modify

[saviic]

nije lose, pohvale za trud 

[_tayloR]

Gledo sam preko fona malo i et savjet

Switchaj listitem
Ne provjeravaj dva puta response
Kod for petlje daj pocetnu vr varijabli
Cuvanje stavi u Sql
Moze manji string kod ribarpath             
if((RibarInfo[playerid][StapPecanje]-1) == 0)
kod pecanja moze ovako, testiraj nisam ssssiguran, da ne dodjeljujes gore vrijedost pa onda provjeravas

to osnovno sto sam ucio ako sta jos budem update cu text

[rane]

Kod for petlje daj pocetnu vr varijabli

Pocetna vrijednost je automatski 0 kao kod svake varijable?

Switchaj listitem

Na jednom dialogu je switchan na drugom sam zabb

Cuvanje stavi u Sql

Radim sa Y_ini vec dugi niz godina bas kao i vecina balkan samp korisnika. Jedva da 10% foruma zna mysql tkd to nije rentabilno jer niko od ovih ostalih ne bi koristio.

if((RibarInfo[playerid][StapPecanje]-1) == 0)
kod pecanja moze ovako, testiraj nisam ssssiguran, da ne dodjeljujes gore vrijedost pa onda provjeravas

Ne kapiram zasto bi prvo provjeravao pa onda dodjelio vrijednost kad mi prvo treba odraditi radnju pa onda obavijestiti ukoliko se stap potrgao.

Ovo ostalo manje vise.
Drago mi je da komentarisete i čekirate kod. Zahvalan sam na diskusiji.

[fortzz]

lijepo, sviđa mi se pravo

[Galardo]

varijable po principu lowerCamelCase, tipa playerMoney.
Varijable kao sto su Peca u tvom slucaju i slicne njoj, koje bi ti sluzile samo u slucaju nekih provjera, prebaci u boolean.

[rane]

varijable po principu lowerCamelCase, tipa playerMoney.
Varijable kao sto su Peca u tvom slucaju i slicne njoj, koje bi ti sluzile samo u slucaju nekih provjera, prebaci u boolean.

hvala na savjetu ali to sam ostavio tako jer u narednim verzijama ce te varijable imati vrijednost 1,2,3,4 i sl. zavisno od faza da ne bi kreirao nove varijable guram sto vise u jednu. Ne znam da li je to pametno ali sta cu.

[fejza]

bolji od proslog top...  8/10

[Zivkovic]

Nije losa fs mogo si malo bolje doradit ali i ovako je dobra

[JoPa.PwN]

Bogme kume, za razliku od proslog puta, ovo jebe 

[Sage]

Ponovo, velika zamerka što imenuješ variajble na srpskom. Niti jedan jezik nije previđen za to pa ni pawn.
(primera radi ni kinezi to ne rade)

...ali ako već želiš...
[pawn]#define CIJENASTAPA 2000
#define CIJENAMAMCA 150
#define CIJENAROSTILJA 4000
#define PRODAJNACIJENA 162[/pawn]
>>
Ne treba ti macro za ovako simple stvari, koristi const ( format nek bude gCenaStapa, gCenaMamca, gCenaRostilja, gProdajnaCena ).
( ako je moguće - koristi static const U FUNKCIJI, znači ne kao globalan. )

#define MAX_TRG 10 > isto može const

[pawn]enum
{
DIALOG_PECANJEOPREMA,
DIALOG_KUPOVINAMAMCA,
DIALOG_PANEL,
DIALOG_PANEL2
}[/pawn]
Još jednom ću reći, SA-MP ima OČAJAN dizajn dialog sistema interno.
Igrač može da vidi SAMO JEDAN DIALOG I NITI JEDAN VIŠE IKADA, koristi y_dialogs ili nešto slično (easydialog).

[pawn]new Peca[MAX_PLAYERS],KreiraShop[MAX_PLAYERS] = -1,Rostilja[MAX_PLAYERS];
new PostavljaRostilj[MAX_PLAYERS]; //0 - nije postavljen, 1 - postavlja, 2 - postavljen[/pawn]
Ponovo, koristiš streamer i samim tim ove varijable ti NE TREBAJU zbog CUSTOM / EXTRA funkcija istog.

[pawn]new Text3D:TrgovinaLabel[MAX_TRG],TrgovinaPickup[MAX_TRG];
new Float:PickupPos[MAX_TRG][3],Kreirana[MAX_TRG],Trgovinaarea[MAX_TRG],TrgMapIcon[MAX_TRG],interierarea;
new Text3D:RostiljLabel[MAX_PLAYERS],RostiljObjekt[MAX_PLAYERS],RostiljArea[MAX_PLAYERS];[/pawn]
Isto, streamer EXTRA.

Moze manji string kod ribarpath

Zašto uopšte praviti string za te stvari ako koristiš YSI

I opet ima toliko grešaka u komandama i u samim funkcijama i ja to stvarno ne mogu ovako kasno da pišem

[Kanac]

pa lepoe sta reci

[Kanac]

Ponovo, velika zamerka što imenuješ variajble na srpskom. Niti jedan jezik nije previđen za to pa ni pawn.
(primera radi ni kinezi to ne rade)

...ali ako već želiš...
[pawn]#define CIJENASTAPA 2000
#define CIJENAMAMCA 150
#define CIJENAROSTILJA 4000
#define PRODAJNACIJENA 162[/pawn]
>>
Ne treba ti macro za ovako simple stvari, koristi const ( format nek bude gCenaStapa, gCenaMamca, gCenaRostilja, gProdajnaCena ).
( ako je moguće - koristi static const U FUNKCIJI, znači ne kao globalan. )

#define MAX_TRG 10 > isto može const

[pawn]enum
{
DIALOG_PECANJEOPREMA,
DIALOG_KUPOVINAMAMCA,
DIALOG_PANEL,
DIALOG_PANEL2
}[/pawn]
Još jednom ću reći, SA-MP ima OČAJAN dizajn dialog sistema interno.
Igrač može da vidi SAMO JEDAN DIALOG I NITI JEDAN VIŠE IKADA, koristi y_dialogs ili nešto slično (easydialog).

[pawn]new Peca[MAX_PLAYERS],KreiraShop[MAX_PLAYERS] = -1,Rostilja[MAX_PLAYERS];
new PostavljaRostilj[MAX_PLAYERS]; //0 - nije postavljen, 1 - postavlja, 2 - postavljen[/pawn]
Ponovo, koristiš streamer i samim tim ove varijable ti NE TREBAJU zbog CUSTOM / EXTRA funkcija istog.

[pawn]new Text3D:TrgovinaLabel[MAX_TRG],TrgovinaPickup[MAX_TRG];
new Float:PickupPos[MAX_TRG][3],Kreirana[MAX_TRG],Trgovinaarea[MAX_TRG],TrgMapIcon[MAX_TRG],interierarea;
new Text3D:RostiljLabel[MAX_PLAYERS],RostiljObjekt[MAX_PLAYERS],RostiljArea[MAX_PLAYERS];[/pawn]
Isto, streamer EXTRA.
Zašto uopšte praviti string za te stvari ako koristiš YSI

I opet ima toliko grešaka u komandama i u samim funkcijama i ja to stvarno ne mogu ovako kasno da pišem

jel te stvarno nije mrzelo sve ovo da ispises 

[fajnydzieciak]

jel te stvarno nije mrzelo sve ovo da ispises 

iiih, ja sam pisao da skrolujes 2 stranice xd i opet me niko nije jebao ni 2%...