Pull to refresh

Comments 7

Пропустим прелюдия, и не будем объяснять зачем это нужно

Эх, пропустили самое интересно


А зачем нужен _index? Везде только изменение его, но нигде нет чтения

getFromKey и getFromIndex лучше переименовать в getByKey и getByIndex.

removeIndex и removeKey не отражают сути, лучше тоже removeByIndex и removeByKey

А максимальный индекс у вас всегда будет равен размеру вашего контейнера?

Почему при добавлении нового значения вы всегда увеливаете index на единицу, даже в случае если такое значение уже есть?

Проще уж тогда сделать ещё одну мапу индексов на ключи, если в таковой есть нужда и искать по индексу вот так key_map[index_map[index]] и для этого не нужно делать ещё одну структуру данных, которая противоречит первому принципу в solid

О, опять Денис решил порадовать нас своим велосипедом на колесах с кирпичами. "документацию читать не хочу, там все непонятно" - снова, да?

Словарь к которому можно обращаться по индексу.

Словарь, значит? Но перегрузка скобок - это слишком сложно оказалось? А про UserDict надо в документации читать, а это трудно, понимаю.

Вот это вот что за дичь такая?

# Исключаем дублирование в списке `_index_list`

if self._date.get(k, None):

То есть значение с None у нас уже не считается значением?

А имя "_date" - это отсылка к тому, что будут храниться только даты?

И хотелось бы, правда, узнать сценарий, в котором будет реально нужен такой класс.

Ну доколе это издевательство тут будет публиковаться? Казалось бы - почти каждая статья в глубоком минусе, но нет...

Молодец, агрументы по факту

Sign up to leave a comment.

Articles