🚀 go-pugleaf

> 1. Spacje wokół przypisania zmiennych, Zmienna := Wartość;
Zawsze to stosuje, dla mnie jest bardziej czytelne.
> 2. Spacje po przecinkach, dwukropkach itd.
Jak powyżej.
> 
> 3. Nazwy parametrów metod oznaczamy np. małą literą a z przodu, np:
>    function TMyClass.Process(aParam : TSomeType);
Kwestia gustu, ale ok.

> 4. Nie mieszj języka polskiego i angielskiego, czyli nie twórz potworków
> typu GetWartosc(), SendWiadomosc(), stosuj ALBO polski ALBO angielski
OK
> 5. Używaj camel case do nazw zmiennych, metod, parametrów czyli np.
> DecrementThreadCount a nie pokraka typu: Decrementthereadcount (tu masz
> jeszcze literówkę!)
Przeoczenie, zazwyczaj tak robię

> 6. Skoro musisz hardcodować SQL w kod to zrób to za pomocą consta lub
> resourcestringa a nie pakuj go w środek kodu:
> 
>   const
>    cSQL = 'select * from tabela '  +
>           'where kolumna = ':kolumna';
>   begin
>    lQuery := TQuery.Create;
>    lQuery.SQL := cSQL;
A tutaj się z tobą nie zgodzę, ponieważ nie mam kodu SQL w miejscu w 
którym go potrzebuję. Czyli muszę podglądać co jest w danej chwili 
załadowane do SQl, a Ja chcę go widzieć i mieć pewność co mam.

Dodatkowo jaki jest sens tworzenia const w takim wypadku?
W czym poprawia ono przejrzystość kodu?
Jakie inne możliwości są przekształcenia kodu SQL aby nie było 
"hardcodowania SQL"?

W czym resourcestring (string w zasobach?)jest lepszy od mojego 
rozwiązania? ResourceHacker i mamy wszystkie SQL-e na dłoni.

> 7. Pola klas powinny mieć prefiks f (wszystkie! i do tego nie raz F a raz
> f, tylko tak samo!)
Kwestia gustu nie jest to wymóg formalny ale OK