The operations themselves are fine but the code style gets in the way of readability. A simple 4-line function instead of the 6-7 methods allows for the same f

The operations themselves are fine but the code style gets in the way of readability. A simple 4-line function instead of the 6-7 methods allows for the same functionality, the same level of encapsulation and for the same testability, without the “jumping around”. Here's the original text I inputted into ChatGPT that generated the code above: def create_user(email) user = User.create(email) user.roles << Roles.find_by(name: 'admin') user.permissions << Permissions.find_by(name: 'invite') Templates.find_by(name: 'invite').call(user: user) end Pretty much anyone writing this kind of class would be able to understand the same code if it were written in a 4-line function in one glance. This here, however severely obscures the intent, the code flow and the operations.