🚀 go-pugleaf

W dniu poniedziałek, 7 sierpnia 2017 20:17:39 UTC+2 użytkownik (fir) napisał:
> W dniu poniedziałek, 7 sierpnia 2017 19:27:50 UTC+2 użytkownik (fir) napisał:
> > wyglad kodu (postac, forma) jest tutaj drugorzędny w stosunku do idei (ktora jest 'przednia' jak dla mnie)
> > 
> > nad forma tak naprawde (jak i nad szczegolami kodu) nie rozmyslalem zbytnio - (nie majac czasu testujac prototypy) ale jak ktos ma jakies rzeczowe uwagi konstruktywne to moze i chetnie przeczytam
> > 
> > z ewidentnych beldow ktore rzucaja mi sie w oczy to np nazwy Compare i Quicksort (powinno byc oczywiscie CompareChunks i Quicksortchunks), bledow formy ktore mozna by tu wymienic jest zapewne duzo wiecej ale jak mowie nie mialem czasu na takie detale (a i dzis ciegle nie chce mi sie nad tym przysiąść)
> 
> ew mozna zastanowic sie jakie operacje na plikach tekstowych/teklstach w ramie moglyby byc uzyteczne a ja moglbym sie zastanowic jakby to wygladalo w implementacji 
> 
> jak widac takie operacje jak wczytanie pliku tekstowego posortowanie lini dodanie czegos do nich i zapisanie na dysk wykonuje sie tutaj latwo - kilkoma linijkami kosu, prostu bez zadnych wyrazen regularnych i tego typu badziewia 9mimoze biblioteczka jest mala 1000 linijek c)

nie wiem, co jest badziewnego w wyrażeniach regularnych.
Twoich kilka linijek w C to z grubsza jedna linijka w perlu albo awk.

myślę też, że zrobiłbyś wielką przysługę wszystkim czytającym Twój kod,
zakładając sobie konto na githubie.

funkcję IsWhiteSpace powinieneś raczej zdefiniować używając literałów
znakowych, zamiast kodów ASCII, np. c == ' ' || c == '\t' itd.
(pomijając kwestię, że prawdopodobnie już jest zdefiniowana np.
w <ctype.h>)

szczerze mówiąc, osobiście wolę korzystać z gotowych i ogólnych
rozwiązań, niż wynajdywać koło na nowo