F18

ini read, write¶

http://redmine.bring.out.ba/issues/25245#note-37

mađarsku notaciju izbaciti, koristimo ruby "like" notaciju imenovanja varijabli¶

podsjećam ovo je mađarska notacija:

cSection
cDatabase
cUserName

mjenjamo to na

section
database
user_name

koristimo tmp_ i slične prefixe za kontekst varijabli. Ne koristimo "c", "l" za određivanje tipa varijable.

u postojećim funkcijama se ostavlja zatečeno stanje, ali se to primjenjuje u novim:

primjer ini_write, ini_read

default parametri¶

kada ne postoji config, init parametri moraju biti logični

npr. database_port = 5432, a ne database_port = 0

upoznavanje funkcija - priprema, harbour playground¶

ako imamo neki bug, pravimo jednostavne izolovane testove u harbour_playground-u

#25258

https://github.com/knowhow/harbour_playground/blob/master/hash/hello.prg

Na ovaj način praktično pripremamo code snippet-e i dokumentaciju funkcija koji su od koristi za čitav tim.

Bez toga svaki developer otkriva toplu vodu. vsasa je potrošio puno vremena na hash upoznavanju, da bih i ja to isto vrijeme morao potrošiti. Da je napravljen playground/hash, sigurno bi moje učenje bilo puno brže.

uvođenje novih funkcija - review¶

kada se naprave sve potrebne pripreme, onda pristupamo pripremi novih funkcija.

Veoma je bitno da te funkcije budu jasno napisane i jednostavne za korištenje:

da ovaj ini_read ne valja

vidi se kada pogledamo kako se koristi:

ponavljanje ini parametra

s obzirom da imamo dvije lokacije predefinisane lokacije, u konačnoj verziji se prosljeđuje logička varijabla koja kaže da li se snima u globalni ili lokalni ini baze.

Zato je za generičke funkcije kakva je ova (common/) najbolje napraviti prototip u harbour_playground.

Ako to traži puno vremena može se napraviti prototip rješenja u f18 i primjer korištenje koji se kasnije komentariše i predstavlja dobru dokumentaciju korištenja.

dokumentacija funkcije i načina korištenja¶

kod novih funkcija bitno je da se na ticket ne stavlja samo commit koji je za drugog čitaoca često beskorisna informacija. Bitno je staviti linkove na funkcije opisati njihovu svrhu i staviti primjere korištenja opet putem linkova.

slično kako sam ja pisao komentare na ovaj ticket

uvođenje novih funkcija - funkcionalni review¶

iako sam već dosta rekao, funkcionalni review je bitna stavka kod dizajniranja novih funkcija.

Kako će se funkcija koristiti najčešće ?

Na šta treba pripaziti da bi se postigle dobre performanse ?

Podsjetiću: ja sam kod ini_read/write uočio da se, shodno logici rada hb_ini funkcije, vrši puno puta iščitavanje i pisanje kompletnog ini fajla.

To funkcionalno svojstvo je odredilo konačni dizajn f18 read/write funkcija.

dummy funkcije u common/dummy.prg¶

sve funkcije koje nisu implementirane trebaju ići u common/dummy.prg da znamo šta smo to samo "krpili" da bi build "progulio"

Zašto F18 programmer's tips ?¶

Zato što smo već sada u F18 utrošili puno vremena.

Trebamo jasno raščlaniti improvizacije koje su rezultat kompromisa usljed nedostatka vremena i ono što se zove kvalitetno programiranje.

Ne smijemo dopustiti da se sa F18, barem kod novog koda, stvara nova "kodna džungla"

F18 jeste i biće puno "legacy" koda, ali za očekivati je da će se shodno potrebama taj kod postupno čistiti barem tamo gdje postoje potrebe za stalnim promjenama.

Na kraju krajeva, očekujemo da će F18 za mnoge korisnike biti dugoročno rješenje.

produktivno korištenje vim-a¶

zajedničkim radom uočio sam da vsasa stalno ulazi/izlazi iz vim-a za svaku korekciju i radi grep.

pokazao sam mu ":vimgrep" koja radi unutar grupe fajlova ali je on i kasnije prema svojim navikama nastavio raditi izađi/grep/uđi.

sa tom praksom za ovakav commit developeru bi bio potreban čitav dan.

programerski editor se treba produktivnije koristiti. Na to treba utrošiti vrijeme. U T0_toolbox ima vim_usage.md.

harbour source code sa string konstantama i našim dikariticima treba biti utf-8¶

http://redmine.bring.out.ba/issues/25296

Lokalne varijable treba da imaju prefix "_"¶

umjesto:

function xzx()
 local aArr := {}
 local cFirma := "" 

treba:

function xzx()
 local _array := {}
 local _firma := "" 

poveži se na mjesto gdje smo to odlučili

Kako se koriste tabele p_xyx() (funkcije za otvaranje šifrarnika)¶

#25312

Uzet ću primjer funkcije p_konto(), otvaranje šifrarnika konta

1) Jednostavno otvaranje tabele radi browse-a, menij šifrarnika¶

Da bi jednostavno otvorili šifrarnik konta radi browsanja podataka jednostavno pozivamo:

p_konto()

2) Povezivanje sa nekom tabelom na uslovima, recimo uslovima izvještaja¶

Da bi napravili link sa tabelom konta koristimo ovako

@ 20, 2 SAY "Odaberi konto:" GET _id_konto VALID p_konto( @_id_konto )

Ovo će nam ako smo unjeli ispravnu šifru vratiti tabelu konta pozicioniranu na tom zapisu, ako ne, onda će otvoriti šifrarnik konta, te nakon odabira sa ENTER-om vraća nam u varijablu _id_konto vrijednost polja "ID" tabele konto.

3) Linkanje sa tabelom sa ispisom vrijednosti polja "NAZ"¶

Da bi na uslovima izvještaja odabrali konto i nakon odabira upisali njegov naziv pored GET polja koristimo ovakvu sintaksu

@ 20, 20 SAY "Odaberi konto:" GET _id_konto VALID p_konto( @_id_konto, 20, 2 )

dakle, sada koristimo i dx i dy varijable u p_konto() funkciji

Kako odrediti dimenzije tabele šifrarnika ?¶

One se određuju u samoj funkciji npr, p_konto() kod poziva funkcije postojisif()

vim¶

#25348

podaci postgresql -> dbf¶

obavezna konverzija: hb_Utf8ToStr

borba protiv džungle¶

#25375

test, komentar uz commit¶

http://redmine.bring.out.ba/issues/25368#note-4

mrtav kod treba brisati a ne komentarisati. zato imamo source code management !¶

http://redmine.bring.out.ba/issues/25395#note-7

getter/setter u harbour-u¶

harbour/xbase je de-facto moćan jezik

hash umjesto gomile parametara¶

neka_funkcija(p1, p2, p3, p4, p5)

zar nije ljepše neka_funkcija(params)

gdje je params hash matrica

poziv je jednostavan

Svaki artifakt svoj changelog i verzija¶

#25611

F18 harbour podrška našim slovima¶

http://redmine.bring.out.ba/issues/25296#note-30

dbf only get/update_rec¶

dbf_get_rec, dbf_update_rec

f18_menu use case¶

f18_menu use case

sa f18_menu se rješavamo nepotrebnih privatnih varijabli, kod je "čišći"

/* 
 * This file is part of the bring.out knowhow ERP, a free and open source 
 * Enterprise Resource Planning software suite,
 * Copyright (c) 1994-2011 by bring.out doo Sarajevo.
 * It is licensed to you under the Common Public Attribution License
 * version 1.0, the full text of which (including FMK specific Exhibits)
 * is available in the file LICENSE_CPAL_bring.out_knowhow.md located at the 
 * root directory of this source code archive.
 * By using this software, you agree to be bound by its terms.
 */

vsasa ove materijale pažljivo proći. Ako ima nejasnoća pitaj.

Sve dok ove prakse ne postanu dio developerske rutine, obavezno se vraćati na njih.

Neka te u tome ne sprječava utisak da te to usporava. Bolje da te usporava čitanje nego da usporavaš i mene i sebe sa mojim ukazima da pratiš ove procedure.

Test case-ovi retroaktivno¶

u slučaju svakog ozbiljnijeg upgrade-a ili debug-a core (common) funkcija procedura je da se uspostavi baterija testcase-a ako je ranije nije bilo.

naravno ima smisla proceduru uspostave test case-ova raditi i na aplikacijskim funkcijama. ali u ovom slučaju su test caseovi nad core funkcijama obavezni

F18 karakterna polja - hb_StrToUtf8 nepotrebno. outf.txt treba biti cp852 enkodiran radi PTXT-a¶

http://redmine.bring.out.ba/issues/25296#note-34