PHP closing tag

[bOkIcA]

Eto ja sam od tih koji u svojim projektima u CI-u izostavljam php closing tag i stavljam '// EOF'. Korektno mi deluje i sa i bez, nista cudno tu ne vidim.

PHP kôd:

<?php

/**
 * example
 */



// EOF

Nemam nista protiv kada se tim dogovara o standardima ali mi nije korektno kada se oni bez obrazlozenja i logike namecu samo zato sto se to nekom ko namece svidja ili je navikao.

[holodoc]

Originalno napisao bluesman

@degojs: nismo se razumeli, ok je to za alate, nego treba napraviti dogovor o svemu pre početka i onda nema potrebe za nekim super-alatima kao što je opisao holodoc. Koji je to timski rad ako svako radi šta hoće i kako hoće?

Code "beautifieri" stvarno nisu nikakavi "super" alati već poprilično korisne stvarčice koje se danas isporučuju uz svaki kvalitetniji IDE. PHPEd recimo dolazi sa podrškom za Polystyle (doduše u trial varijanti) dok Eclipse i NetBeans imaju svoja ugrađena rešenja. Postoji i dobar razlog za to. Naime, koliko god tim bio uštelovan i kakve god konvencije kodiranja da se usvoje na početku, svaki koder ima neku svoju specifičnost koje se teško odriče. Ja recimo ne mogu da smislim kod u kome nov blok koda počinje u vitičastoj zagradi u sledećem redu i te prakse se držim od kako sam ukucao prvu vitičastu zagradu u svom kodu pre sada već nešto više od 20 godina. I ne verujem da će me iko ikada naterati da drugačije pišem kod. Ako se to nekome ne sviđa što se tiče mene u redu je. Napisaću kod po mome a pred predavanje ću ga provući kroz Polystyle što je operacija koja traje sekundu-dve.

Da stvar bude gora specifičnosti se posebno osećaju u timskom radu sa strancima. Nemci su recimo bez daljnjeg najspecifičniji u pisanju koda (kada je PHP u pitanju dosta njih koristi neku vrstu mutanta između PEARa i nekih krajnje čudnih sintaksičkih "sentenci") dok recimo ima slučajeva da kod bude toliko loš da ništa osim refactoringa ne preostaje.

Zato code "beautifieri" nisu alatke koje treba potcenjivati, posebno Polystyle. Neki ih ne vole ali nepobitna je činjenica da jednim potezom može da se postigne potpuna doslednost jednom istom načinu formatiranja koda tj. izbegla karta u jednom pravcu za Golgotu kada nalogodavac posla zakuca na vrata

Originalno napisao bluesman

Kažeš "Otvoriš neki fajl, udariš kombinaciju tastera i to je to." I onda sačuvaš, pa onda neko drugi uradi to isto, pa neko treći to isto i na kraju dođete do revizije 14 a niko ništa nije menjao nego je samo hteo da "baci pogled".

Većina klijenata za CVS i SVN danas ima opciju pregleda sadržaja revizija direktno iz repozitorijuma tako da ne vidim zbog čega bi neko radio check-in za materijal koji može direktno da se pogleda u repozitorijumu. SVN recimo podržava ovu opciju i bez nekih specijalnih klijenata (za one koji su više privrženi konzoli) a odlična alternativa korisnicima na Widnowsu je TortoiseSVN.

Originalno napisao bluesman

Ako ćemo iskreno, današnji editori sve manje donose noviteta vezaih za samo kodiranje, a sve više glavnih features su vezani za kozmetiku, odnosno "beautifying the code" i onda editori rade 40% sporije da bi mogli da progutaju svačije coding styles i kako kaže holodoc: "istrenira" da zapazi najbitnije šablone formatiranja izvornog koda i napravi profil a nakon završene ispravke sve se vrati na staro sa sačuvanim profilom... Super što može ali to je bacanje resursa i vremena koje može da se preči običnim "old-school" dogovorom. )

Slažem se da neka razvojna okruženja imaju gomilu funkcionalnosti koje realno nisu potrebna krajnjim korisnicima ali definitivno se ne slažem da nije potrebno imati alatke u okviru editora koje omogućavaju "održavanje higijene" koda kao što sam to već objasnio u prethodnom delu ovoga posta. Vodeći se tom logikom ispašće da su i sintaksno obeležavanje koda, indentacija i sličen stvari takođe luksuz što je po meni opet potpuna krajnost Da ne budem pogrešno shvaćen, kao i svako ko ima nameru da živi od svojih projekata, volim brze i pre svega upotrebljiva razvojna okruženja tako da na Windows baziranim platformama kada je PHP u pitanju moj izbor pada na PHPEd. Na Linuxu nažalost postoji samo jedna dostojna zamena za koju može da se kaže da je upotrebljiva a to je po meni PDT.

Uostalom koga ne mrzi može da pročita jednu od mojih skorašnjih tekstova na temu razvojnih PHP okruženja (link). Tekst je malo stariji ali generalno nema nekih preteranih razlika u odnosu na nove verzije.

[bluesman]

"Potcenjivati"? "Nije potrebno"? Ti si nešto pogrešno razumeo u mom postu. Potrebno je, i to baš zbog ljudi koji imaju takav stav kao ti Evo i zašto tako mislim: ako već dođe do toga da moraš da uptrebljavaš te alate i pored svih dogovora, onda džabe i dogovor - neka radi ko šta hoće i kako hoće. A ako niko ne može nekoga naterati da promeni navike - onda je taj solista i takav stav nije za bilo kakav timski rad. A ako nema timskog rada, i sve radiš sam - onda ti ti alati nisu ni potrebni jer je sav kod tvoj i nema "konflikta". Dakle, paradoks

[holodoc]

Ne bavim se paradoksima jer su previše rekurzivni

Što se tiče pitanja "dogovora"... Jedno je kada kompanija ima takvu poslovnu politiku da svakom junior developeru dodeli mentora koji će biti zadužen za uvođenje "junoše" u sitna crevca i uklapanje u svoje izolovane razvojne timove a potpuno drugo je realnost u kojoj se dešava da na projektu sarađuje nekoliko kompanija, neretko iz različitih država, gde svaka ima svoja ustaljena pravila i načine rada. Šta onda? Poslodavac kaže da posao mora da se odradi a njegov razvojni tim kuka kako će verovatno biti problema jer postoje neke neusklađenosti oko formatiranja koda

Još gore ako se radi na projektu gde se više postojećih različtih projekata uklapa u jednu celinu. Ako nije potrebno "prčkati" po postojećem kodu i njegova enkapsulacija "drži vodu" onda u redu, ne treba poprvljati ono što nije pokvareno. Međutim, ako treba zavrnuti rukave teško da ima profesionalca koji nije osetio čari refactoringa i "ulepšivača koda"

[eraser]

"You’re not here to write code; you’re here to ship products!"

[mbabuskov]

Originalno napisao bluesman

Znači preporuka Code Igniter je da se ne zatvara PHP tag? Meni to zvuči prilično čudno

Razlog je CI celo izvrsavanje koda stavlja u ob_start/ob_end_flush, kreira prvo celu stranicu i tek na kraju salje headere i sadrzaj. E sad, ako bi se nedge potkrao taj enter iza ?> na kraju fajla, pucale bi neke stvari. Dakle, razlog je interni mehanizam kako framework radi (bar tako kazu u CI dokumentaciji).

E sad, zasto imati Enter na kraju fajla je vec drugo pitanje. Neki editori ga dodaju automatski, a neki version control sistemi po defaultu javljaju warning za to. Bas iz razloga sto je tekstualni fajl koji nema enter na kraju sumnjiv, kao da nije kompletan.

Licno, meni se svidja preporuka CodeIgnitera da se koristi // EOF ili neka slicna oznaka, pa sam poceo da koristim to i kada radim van CIa. Nekako mi kod izgleda lepse u editoru