2

Столкнувшись с задачкой преобразовать список объектов в Map, решил ее следующим образом:

class OrganizationFields {
  late Map<String, OrgField> items = <String, OrgField>{};

  OrganizationFields({ List<OrgField>? fields })
    : this.items = Map.fromIterable([...?fields], key: (itm) => itm.name, value: (itm) => itm);
}

Но не нравится то, что тут на каждой внутренней итерации fromIterable будет выполняться по два вызова. Это выглядит не очень "красиво", и предполагаю что в Dart возможны более скоростные решения. Тем более что в другом месте проекта предполагается очень частое пересоздание практически аналогичных Map другой сущности, и хотелось бы научиться это оптимизировать. Входные данные всегда списками, и это нельзя изменить, к сожалению.
Поэтому, вопрос: есть ли способы сделать то же самое (так же создать Map из List'а объектов с использованием свойства в качестве ключа) более шустрым кодом без использования множества вызовов?


Тот же код в песочнице (с рандомными элементами).


p.s.:
В проекте использую Flutter+GetX, если вдруг это имеет значение.
Это мой самый первый опыт с дартом (в основном на JS пишу), и если делаю что-то неправильно - буду признателен за критику в комментах.

8
  • «Входные данные всегда списками, и это нельзя изменить, к сожалению.» - Списки прилетают с сервера, джейсоном. Как резервный вариант я рассматриваю ручное переопределение fromJson, но схем данных в апишке слишком много, некоторые сильно ветвистые, и писать десериализаторы вручную - это ад и израиль (сейчас разбор данных выполняется через генерацию кода, пакетом json_serializable).
    – yar85
    30 июн 2021 в 2:13
  • 1
    В любом случае это будет проход циклом по списку что через Map.fromIterable, что через for... Если вы так хотите заморочиться с оптимизация и красивом кодом, я бы предложил выкинуть GetX из проекта, так как это не красиво (со временем как снежный ком будет) и еще очень плохо по производительности (если что, это один большой синглтон)...
    – MiT
    30 июн 2021 в 17:25
  • 1
    P.S. items = { for (var itm in fields) v.name: itm }; но это тоже самое и читать сложнее
    – MiT
    30 июн 2021 в 17:26
  • 1
    P.S.S. с первым вопросом вас!)
    – MiT
    30 июн 2021 в 17:31
  • @MiT, понимаю что прохода по списку избежать конечно нельзя, и он беспокоит меня гораздо меньше чем 2n вызовов при каждом инстанцировании (вопрос не просто так акцентирован на вызовах). «выкинуть GetX из проекта, так как это не красиво (со временем как снежный ком будет)» - о, вот это весьма интересно и тревожно!)) Насчет производительности предостережение учел, исследую этот вопрос. Но не понял второе: от чего оно будет расти как снежный ком, то есть на что следует смотреть чтобы заметить это?
    – yar85
    30 июн 2021 в 17:45

1 ответ 1

2
+200

Есть альтернатива Map.fromIterable без callback, это for/for-in:

items = { for (int i = 0; i < fields.length; i++) fields[i].name: fields[i] };
// или
items = { for (var itm in fields) v.name: itm };

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.