I think they did so because they did not need this class anywhere else.
Without opening the singleton class the flow would look as following (assuming every method defined in metaclass from original code would be prefixed with self.
):
They could have defined the Identity
as
class LetterAvatar
class Identity
end
end
and then use the class in self.generate
method as follows:
class LetterAvatar
# code omitted
def self.generate
identity = LetterAvatar::Identity.from_username(username)
# code omitted
end
# other class level methods defined with `self.`
end
But why doing so if the Identity
class is actually used only (and need not to be accessed anywhere else) in the singleton class (in generate
)?
The solution is IMO very elegant, haven't seen anything like this before.