Ej wiecie co...

Question

Witajcie, jako mist chciałbym się pochwalić, że udało mi się spełnić jedno marzenie – może nie jedno, a więcej, ale chcę przedstawić wam moją pracę. Wiem, że pewnie mi powiecie dużo o bezpieczeństwie kodu czy temu, że nie ma większej dokumentacji, jednak postanowiłem się tym w końcu pochwalić.

Język ma jeszcze dużo błędów czy nieskończonych rzeczy.

 Mój język programowania SNS

 

Plik codf.sns zawiera przykładowy kod z użyciem wskaźników. 

Dziekuje.

Comments

Mialbym nadzieje na opinie panelo.

---

Zacznijmy od tego że się nie kompiluje:

void T_shift_chars(char *buffer, int S, int L) {
    int dc = strlen(bufor);

I nie wrzuciłeś main.c :P

---

Przepraszam juz to naprawie za pare minut

---

@adrian17, Zmieniałem pare nazw na lepsze z stąd ten bład juz poprawiłem i dodalem plik main.c

Answer 1

Nie wnikałem jakoś w detale, ale ogólne review jeśli chcesz:

- naucz się gita :P

- nie będę się czepiał temu że masz zmienne globalne albo temu że nie walidujesz poprawności... czegokolwiek, jak piszesz to sam dla siebie to whatever :)

- dużo kodu takiej postaci.

(get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -8) ? ' ' :
  (get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -4) ? 'd' :
    (get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -2) ? 'w' : 'b');

- również kod typu

bool x = costam == costam ? true : false;
// to równie dobrze możesz napisać
bool x = costam == costam;

- dużo kodu gdzie deklarujesz globalną zmienną i funkcję `init` gdzie deklarujesz stałą i kopiujesz tą stałą do globalnej zmiennej. Czemu to nie mogła być po prostu globalna stała bez żadnej funkcji?

- kompletny brak spójności czy nazywasz rzeczy jakos_tak, Jakos_tak, JakosTak czy Jakos_Tak.

- za dużo skrótów - rozumiem że nie chcesz pisać dokumentacji, ale jak nazywasz zmienną `ss`, to nikt nie ma pojęcia co to robi, w tym Ty za kilka miesięcy :)

- masz dużo fajnych enumów, po czym nagle rzucasz gołymi liczbami :c `New_Token_init(7, string, 0);`

Comments

Dziekuje za odpowiedz i rady napewno je zapamietam. A co do twojej uwagi w stosunku do:

(get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -8) ? ' ' : 
  (get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -4) ? 'd' : 
    (get_variable_offset(temp2->left->left->Text, tree->right->right->Text) == -2) ? 'w' : 'b'); 

 

Faktycznie jest to nieczytelne ani nie jest wydajne(choc mozliwe ze gcc może to optymalizować). Zdaje sobie z tego sprawe aczkolwiek to dość duży projekt i czesto pisze tak bo mi wygodniej podczas pracy nad kodem. Oczywiscie planuje to zmienić.

 

W najblizszej chwili gdy bede mogl chetnie poprawie owe rzeczy.

Miłego dnia zycze