Привет) Такой вопрос. У меня есть EdgeInsets.all(20) и булевая flag.

Question

Привет) Такой вопрос. У меня есть EdgeInsets.all(20) и булевая flag.

Если флаг это тру, то справа должно быть 10 пикселей, а если фолз, то 20. Есть ли более короткая запись чем EdgeInsets.fromLTRB(20, 20, flag ? 10 : 20, 20)? А то как-то некрасиво

#dart #programming #russian

0

17.02.2022

16 ответов

11 просмотров

Kirill Bubochkin

EdgeInsets.all(20).copyWith(right: flag ? 10 : 20)

0

17.02.2022

Andrii Buravykov Автор вопроса

Kirill Bubochkin
EdgeInsets.all(20).copyWith(right: flag ? 10 : 20)

оо, пасибо

0

17.02.2022

Дмитрий Щербаков

Kirill Bubochkin
EdgeInsets.all(20).copyWith(right: flag ? 10 : 20)

но так же длинннеееее)

0

17.02.2022

Kirill Bubochkin

Дмитрий Щербаков
но так же длинннеееее)

вообще, да, но если делать именованную константу вместо magic number, то будет короче и (имхо) немного нагляднее – лучше видно, что right это особый случай, а всё остальное одинаковое

0

17.02.2022

Дмитрий Щербаков

Kirill Bubochkin
вообще, да, но если делать именованную константу в...

оверинжиниринг)

0

17.02.2022

Kirill Bubochkin

Дмитрий Щербаков
оверинжиниринг)

оверинжиниринг – это *ненужное* усложнение. Если это хоть немного улучшает читабельность, и никак не влияет на всё остальное, то нет, это не оверинжиниринг

0

17.02.2022

Kirill Bubochkin

Kirill Bubochkin
оверинжиниринг – это *ненужное* усложнение. Если э...

это скорее nitpicking. Понятно, что на codereview я бы PR не заворачивал, ни с первым вариантом, ни со вторым

0

17.02.2022

Adel' Gilyazeev

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

0

17.02.2022

Kirill Bubochkin

Adel' Gilyazeev
А если я выношу интерфейс дата класса отдельно, ко...

it depends

0

17.02.2022

Adel' Gilyazeev

Ничего не понял

0

17.02.2022

Kirill Bubochkin

Adel' Gilyazeev
Ничего не понял

не может быть общего ответа, в каждом конкретном случае будет по-разному. Ну и «оверинжиниринг» - это не бинарное состояние )

0

17.02.2022

Adel' Gilyazeev

Kirill Bubochkin
не может быть общего ответа, в каждом конкретном с...

Ну как бы прям тривиально: изначально имеем: class Data { const Data(this.v) final String v; } И вместо того чтобы отдельно в модулях сделать const moduleData = Data('A'); Я сделал интерфейс: abstract class IData { String get v; } И в каждом модули сделал class DataModuleA implements IData { @literal const DataModuleA(); String get v => 'A'; }

0

17.02.2022

Adel' Gilyazeev

Adel' Gilyazeev
Ну как бы прям тривиально: изначально имеем: clas...

А интерфейс бд съедает теперь IData, но выдаёт всегда просто Data

0

17.02.2022

Kirill Bubochkin

Adel' Gilyazeev
Ну как бы прям тривиально: изначально имеем: clas...

с какой целью?

0

17.02.2022