I want to ask:

1. Whether the Role class violates Single Responsibility Principle in SOLID ? I think deleteAccount() is not belong to Role class but Role class is way to extend code in the future

2. There are deleteAccount() in Role and Account. That is ok ?

3. Admin and Moderate violates DRY principle ? Because the deleteAccount() was repeated. If yes, how to fix it ?

4. How to implement the deleteAccount() which can delete itself account ? Should I use strategy design pattern to do that ?

    class Account: def __init__(self, email, password, security_ans): self.status = "Active" self.role = Member() def deleteAccount(self, dest_email): dest_account = accounts.getByEmail(dest_email) #get account obj from accounts self.role.deleteAccount(self) 

This is my Role class:

class Role: def blockAccount(self, account): raise NotImplementedError("No Permission") class Admin(Role): def deleteAccount(self, account_obj): account_obj.setStatus('Closed') class Moderator(Role): def deleteAccount(self, account_obj): account_obj.setStatus('Closed') class Member(Role): def deleteAccount(self, account_obj): raise NotImplementedError("No Permission")