testnode
NODE TEST
-
dependencies:
- express — Это серверный фреймворк, который необходим для работы вашего веб-сайта в продакшене, поэтому он должен находиться в разделе dependencies.
-
devDependencies:
- parcel — Это инструмент, который используется только во время разработки и сборки проекта. Он не нужен в продакшене для выполнения приложения, следовательно, его следует поместить в devDependencies.
- svelte — Это библиотека для создания пользовательского интерфейса, но так как она также участвует в процессе сборки и используются только в процессе разработки, её тоже лучше поместить в devDependencies.
-
EventEmitter.call(this); // <=IE10 fix (unable to call parent constructor)
Здесь используется вызов конструктора родительского класса только для IE10, что может привести к проблемам, если класс будет переиспользован в других местах кода. Лучше всегда вызывать
в конструкторе, чтобы гарантировать, что родитель инициализирован корректно.super()-
В методах getResource и других методах, использующих
, не проверяется, существует ли запрашиваемый ресурс. Это может привести к тому, что функция вернетutils.getPath
, но не сгенерирует никакое предупреждение или ошибку.undefined -
Логика обработки параметров:
Некоторые параметры, такие как
, устанавливаются в нескольких местах, что может привести к путанице. Лучше рассмотреть возможность приведения параметров к единому виду в начале метода (например, в addResource).keySeparator -
Методы без проверки на наличие данных:
В методах вроде
сначала проверяется наличие ресурсного пакета с помощьюremoveResourceBundle
, но затем не проверяется на наличие данных в массиве или объекте. Например, в методеhasResourceBundle
стоит убедиться, чтоremoveResourceBundle
существует, перед тем как к нему обращаться.this.data[lng] -
Отсутствие комментариев и типов для параметров и возвращаемых значений методов делает код менее читабельным.
-
npm i
npm test
node index.js try/test.txt
-
-
-
https://github.com/mihailgok/telegramwebapps - это просто самый популярный мой репо. Там JS + Python.