Wątek przeniesiony 2019-08-11 03:11 z przez cerrato.

Mógłby ktoś ocenić jakie mam złe praktyki?

Odpowiedz Nowy wątek
2019-08-11 02:40
0

Hejka, programuję od pół roku i nie mam skąd brać tzw. code review mógłby ktoś powiedzieć np. jakie mam złe praktyki w kodzie?
Nie chodzi mi ,że cos nie działa tylko generalnie czego nie powinienem na podstawie np. tego, to jest ostatnie co zrobiłem.
Tutaj link do projektu, który chcę do zreviewiowania https://drive.google.com/driv[...]jsGj--S3d-yu8SnS6kvFfmKMNeu80
ew. https://github.com/SurmaAa/TrTranslate bo chodzi mi o ten projekt, nie o inne

edytowany 1x, ostatnio: MarcinekJedziemy, 2019-08-11 02:42

Pozostało 580 znaków

2019-08-11 03:49
1
  1. W tym punkcie nie tyle złe praktyki co niewykorzystanie możliwości języka.
        sendTranslationBack: sendTranslationBack,
        close: close,
        COUNTRY_LIST_EN : COUNTRY_LIST_EN,

w ES6 możesz to zapisać skrótowo, np.

{ foo, bar}

zamiast

{ foo: foo, bar: bar }
  1. https://github.com/SurmaAa/TrTranslate/blob/master/server.js No i to mi coś nie gra:
    app.get(`${configuration.file.server.req.indexWWW}`, (req, res) => res.sendFile(`${__dirname}${configuration.file.server.res.indexWWW}`));

    czemu te linijki są takie długie?
    tu też masz długie linijki: https://github.com/SurmaAa/Tr[...]/public/js/list-control.js#L7
    protip: jak wywołujesz funkcję, to nie musisz się zmieścić w jednej linii, możesz np. każdy parametr w innej:

app.get(
     `${configuration.file.server.req.indexWWW}`, 
     (req, res) => res.sendFile(`${__dirname}${configuration.file.server.res.indexWWW}`))
); 
  1. Poza tym w pliku server.js robisz ileś bardzo podobnych wywołań app.get do każdego pliku z osobna. Ogólnie przypuszczam, że w jakiś sposób możnaby to prościej zapisać albo na poziomie Expressa (dawno już z Expressa nie korzystałem, ale tam było coś chyba static files https://expressjs.com/en/starter/static-files.html O ile to to, z czego chcesz faktycznie skorzystać. Poza tym nawet poza Expressem można by to skrócić. Np. przelecieć w pętli (albo forEach) przez wszystkie (albo wybrane) klucze zmiennej configuration.file.server.req i w każdej iteracji wywołać app.get z odpowiednimi parametrami.

((0b10*0b11*(0b10**0b101-0b10)**0b10+0b110)**0b10+(100-1)**0b10+0x10-1).toString(0b10**0b101+0b100);
edytowany 1x, ostatnio: LukeJL, 2019-08-11 03:50

Pozostało 580 znaków

2019-08-11 16:35
0

dzieki, długie bo chciałem mieć stringi w jednym pliku i czytelnie zeby po przeczytaniu configuratio.costam.costam bylo wiedziec co to jest a nie tam zagladac

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0