Сегодня я раскопал свой код на typescript и он... ну такой себе.

В нем ноль комментариев, что как плюс т.к. они не вводят нас в заблуждение, так и минус, потому что код сам не 100% очевидный.

Сейчас для меня код выглядит так, как будто я либо не думал над неймингом, либо не смог над ним подумать. Он вроде консистентный, но не говорящий. Как будто у меня была идея в голове и эти названия в неё вписывались. Еще мне хотелось написать меньше строк, поэтому цепочки методов все в строку. Строковые и числовые литералы не вынесены в константы. Т.е. этот код как минимум не пересматривался после написания рабочей версии.

Часть функций похожа на копипаст откуда-то, т.к. они оформлены по-разному. Какие-то как function statement (filterTag), какие-то как анонимные function expresssion с хранением в переменной (setOpenInNewLeaf). Смысла в этом не видно, т.к. во всем коде коллбэком является только handleOpenRandomNote.

Подытоживая можно сказать, что код писался с попыткой в структуризацию и копипастом. Не переписывался для упрощения понимания, не рефакторился для уменьшения потенциальных ошибок. Почти нет обработки негативных сценариев. Очень позитивно я был настроен, когда это писал =) Прикладываю исходник на гитхабе https://github.com/ShockThunder/improved-random-note/blob/master/src/utilities.ts

#программирование #code #код #говнокод #refactor #IT #typescript