Repository navigation
Auto add builder factories for nested built_collection classes #1254
Description
When using BuiltMap nesting, is it not automatically created?
abstract class Calendar implements Built<Calendar, CalendarBuilder> {
...
@BuiltValueField(wireName: 'calendar')
BuiltMap<String, BuiltMap<String, BuiltList<DayDTO>>>? get calendar;
...
}
automatically generated below
..addBuilderFactory(
const FullType(BuiltMap, const [
const FullType(String),
const FullType(BuiltMap, const [
const FullType(String),
const FullType(
BuiltList, const [const FullType(DayDTO)])
])
]),
() => new MapBuilder<String,
BuiltMap<String, BuiltList<DayDTO>>>())
however, the below needs to be added manually.
..addBuilderFactory(
const FullType(BuiltMap, const [
const FullType(String),
const FullType(
BuiltList, const [const FullType(DayDTO)])
]),
() => new MapBuilder<String, BuiltList<DayDTO>>())
..addBuilderFactory(
const FullType(
BuiltList, const [const FullType(DayDTO)]),
() => new ListBuilder<DayDTO>())
It won't work unless you add it manually in the code above. So you have to add it manually, but it works.
Is there a way to automatically generate it?
Reacted by Pierre Martin and Timm Preetz
Activity
JosephNK commentedon Jun 3, 2023
I added the following code for auto-generation.
Is it correct to use it like this?
davidmorgan commentedon Jun 5, 2023
Yes, it's necessary to add builder factories manually in some cases.
It's true that you could add fields as in your second post so the builder factories get generated for you, but then you will end up with a very weird class :) ... maybe you could put all such fields in a single class that you don't use, as a more convenient way to add builder factories.
Sorry that this part of
built_valueis so awkward, there is a long standing feature request to improve it but I haven't had a chance to tackle it![-]When using BuiltMap nesting, is it not automatically created?[/-][+]Auto add builder factories for nested built_collection classes[/+]pierremrtn commentedon Sep 1, 2024
I was considering using built_value for a project, but this is a big deal to me. The amount of boilerplate can be huge and hard to write.
tp commentedon Oct 19, 2024
@pierremrtn I understand that concern. For a current project (which internally uses
built_valuewith theDioOpenAPI generator) the value gained form this package much outweighs the extra effort / headache caused by this.For the case where we now want to deal with e.g.
BuiltMap<String, BuiltList<Foo>>I added a helper method:With that we only have to call
addMapOfList<String, X>()for each case on theSerializersBuilderon init. So after the initial setup this is now easy to use, should this ever come up again.Maybe this helps anyone else to work around this problem quickly until it's supported via code generation.