refactor: collection-factory create method

[recepkefelii]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

The original code used a switch statement to determine which type of AbstractCollection instance to return. The new code uses a conditional if statement to perform the same task, which makes the code more concise and readable. Additionally, the instantiation of the SchematicRunner object was moved outside of the conditional statement, so that it only needs to be created once, improving the performance of the code.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

[micalevisk]

so that it only needs to be created once, improving the performance of the code.

the old version creates it once too as the 'case' is evaluated only one time.

But yours is more clean to me.

[recepkefelii]

so that it only needs to be created once, improving the performance of the code.

the old version creates it once too as the 'case' is evaluated only one time.

But yours is more clean to me.

thanks