hxnvr / pricelist Goto Github PK

1. Если единственное, чего Вы хотите от классов - хранить данные, то предпочтительным способом реализации является создание data class'ов. К "плюшкам" такого подхода относится то, что за Вас переопределяются стандартные методы типа equals;

2. Не вполне понятно, почему Вы решили использовать список в качестве структуры, хранящей набор уникальных продуктов. Идея с инкапсуляцией данных продукта в отдельный класс - хорошая, но если мы не поддерживаем дублирование данных, применение такого контейнера, как список - явно не оптимальное по быстродействию решение, в тех же методах clear Вы пробегаете по всем элементам списка в фильтре;

3. Пользователь Ваших классов должен как-то понимать, какой результат он получил, вызывая тот или иной метод - пожалуйста, посмотрите, как это решено для стандартных коллекций.

1. В Котлине, так же, как и в Java, принято разбивать код на пакеты. Why so?;

2. В именовании классов в Kotlin-е, так же, как в Java, принято следовать нотации UpperCamelCase.

При работе с системами контроля версий некоторые файлы и директории, такие, как, например, генерируемые Идеей workspace.xml, tasks.xml, *.iws (список легко найти), содержащие информацию о пользовательских настройках системы, принято игнорировать. Для указания информации об игнорируемых типах данных создаётся файл .gitgnore. В IntelliJ Idea доступен плагин, с помощью которого такой файл можно сгенерировать на основе стандартных шаблонов.