[M4lutki]

#programowanie #c #codereview

Mireczki programiści, zgłaszam się do Was z ogromną prośbą. Mianowicie chciałbym by ktoś doświadczony przeglądnął mój krótki kod i zwrócił uwagę co poprawić. Nie chodzi mi o optymalizacje (aczkolwiek jeśli komuś się chce... :P), a bardziej o składnie, wcięcia i praktyczne porady co poprawić, na co zwracać uwagę :)

Tutaj fragment mojej mini-biblioteki do obsługi kolejki komunikatów.

Klient wysyła ciąg 'char'ów do serwera, a ten zamienia male litery na duże i zwraca do kolejki tak by klient mógł sobie odebrać. Wszystko elegancko działa nawet dla kilku klientów naraz :)

http://pastebin.com/tMBBTcCq

[m.....t]

@M4lutki: codereview.stackexchange.com

[m.....t]

@matiit: No i jest też tag: #codereview

[M4lutki]

@matiit: Dzięki wielkie, nawet nie miałem pojęcia o istnieniu takiej strony i tagu :)

[Szab]

@M4lutki: G---o, wszystko do poprawy.

[morsik]

@M4lutki: Ale c-----e wcięcia.

[M4lutki]

@morsik: To mi popraw tak jak powinny być i mi pokaż a nie że c-----e :P

[morsik]

@M4lutki: No masz biedaku :(

https://dpaste.de/rfFE

[M4lutki]

@morsik: K---a nigdy nie lubiłem tych wcięć od while.

for(bla;bla;bla)

[morsik]

@M4lutki: Tylko, że jest czytelne i wiadomo dokładnie gdzię kończą i zaczynają klamry. Ale to tam kij, jak wolisz to pisz

if (d--a) {

} else