You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently there's a single GltfImportconstructor that takes 4 optional parameters.
Problems with that:
Already quite overloaded, this will be even more so once more parameters get added.
It's not possible to change the behavior of the logger parameter to create a default ConsoleLogger if null is provided, as that takes away the possibility of not setting a logger.
Solutions
Create explicit overloads with no optional paramters at all.
Implement a Factory/Builder like pattern with chained setup methods.
Solution 1 might lead to many overloads, potentially conflicting with the original one.
Solution 2 is potentially cleaner.
The text was updated successfully, but these errors were encountered:
Problem
Currently there's a single
GltfImport
constructor that takes 4 optional parameters.Problems with that:
logger
parameter to create a defaultConsoleLogger
ifnull
is provided, as that takes away the possibility of not setting a logger.Solutions
Solution 1 might lead to many overloads, potentially conflicting with the original one.
Solution 2 is potentially cleaner.
The text was updated successfully, but these errors were encountered: