Projekat

Općenito

Profil

Akcije

Podrška #25375

Zatvoren

krčenje FMK džungle

Dodano od Ernad Husremović prije više od 13 godina. Izmjenjeno prije više od 13 godina.

Status:
Zatvoreno
Prioritet:
Normalan
Odgovorna osoba:
Početak:
23.11.2011
Završetak:
% završeno:

0%

Procjena vremena:

Povezani tiketi 2 (0 otvoreno2 zatvorenih)

korelira sa F18 - Podrška #25271: F18, programmer's tipsZatvorenoErnad Husremović16.11.2011

Akcije
korelira sa F18 - Podrška #25411: glavni meni izbaciti iz main funkcijeZatvorenoErnad Husremović24.11.2011

Akcije
Akcije #1

Izmjenjeno od Ernad Husremović prije više od 13 godina

kilometarske prg fajlove dijeliti na logične cijeline.

sifk_sifv: https://github.com/knowhow/F18_knowhow/commit/1369823136c48583026d099b4cba586d771d7cac

prebacivati stvari koje pripadaju drugoj logičnoj cjelini

barkod: https://github.com/knowhow/F18_knowhow/commit/fa4184a27c747c0575622ed1b2cbc0122b8793af

Akcije #2

Izmjenjeno od Ernad Husremović prije više od 13 godina

jednostavno organizacija source code-a treba nam omogućiti lakše snalaženje

izbjegavati kilometarske source fajlove - razbijati ih

Akcije #3

Izmjenjeno od Ernad Husremović prije više od 13 godina

  • Odgovorna osoba promijenjeno iz Saša Vranić u Ernad Husremović

privatne i nedeklarisane varijable su naši neprijatelji

zato mjenjam konvenciju imenovanja, deklarišem varijable

https://github.com/knowhow/F18_knowhow/commit/5764dd83772ab5d4391e4b844926082429d93c07#L3R21

Akcije #4

Izmjenjeno od Ernad Husremović prije više od 13 godina

stisnuto razmaketi

iz nekoh sadomazo razloga mirsad i ja smo ovako stiskali varijable

if(a,b,c) => if(a, b, c)

to je teško razumljivo. čistiti to:

https://github.com/knowhow/F18_knowhow/commit/8587733de084c7b5135fcbc6adf118e58a711bc0

Akcije #5

Izmjenjeno od Ernad Husremović prije više od 13 godina

a+b+c

=>

a + b + c
Akcije #7

Izmjenjeno od Ernad Husremović prije više od 13 godina

codes_browse.prg

privatne varijable, nepregledni if-ovi grozno grozno.

koliko-toliko sam formatirao kod iako je to sve ali ne pregledno.

Akcije #8

Izmjenjeno od Ernad Husremović prije više od 13 godina

f18 takođe može postati džungla

puna varijabli koje iskaču na sve strane ...

poradio na tome:

https://github.com/knowhow/F18_knowhow/commit/3eab8e3a6069ad096feaa3f476754bda55e4c8de

Akcije #9

Izmjenjeno od Ernad Husremović prije više od 13 godina

hash matrica je vaš friend

umjesto milion parametara koristim hash __server_params

Akcije #10

Izmjenjeno od Ernad Husremović prije više od 13 godina

kada mi se pojavi novi sličan parametar, samo proširujem postojeću hash matricu - server search_path

Akcije #11

Izmjenjeno od Ernad Husremović prije više od 13 godina

  • Status promijenjeno iz Dodijeljeno u Zatvoreno
Akcije

Također dostupno kao Atom PDF