What's preferred/right way to use component in module helper

[joomdonation]

In getList method of some of our modules, we need to use component model to get data. However, currently, we are doing it in different ways in different modules:

1. Create model object in the module entry point and pass it to the method as a variable. Example https://github.com/joomla/joomla-cms/blob/4.0-dev/modules/mod_banners/mod_banners.php#L21-L22
2. Create the model inside getList method, example https://github.com/joomla/joomla-cms/blob/4.0-dev/modules/mod_articles_popular/src/Helper/ArticlesPopularHelper.php#L40

We should choose an option and do it in the same way in all of our modules instead of doing it in different ways like how we are doing at the moment

To me, I prefer the second option because we usually have $app variable as a parameter in the method already, so creating model instead the method makes the module entry point a bit cleaner.