确定复杂的安全性和权限

Question

我参与了一个项目，该项目有一个混乱的类，该类将按钮写入页面。该应用程序是一个文档管理器，有一个弹出按钮列表，例如下载、电子邮件和打印。根据用户的角色和文档的状态，会显示不同的按钮。

在其他 WTF 中，有这样的事情：

bool showEditButton = document.docTypeId == documentEnum.docType.text && 
( document.statusId == documentEnum.docStatus.Editable || (user.UserStatus == userEnum.Status.SuperUser) || ( user.UserID == document.CreatedByUserId ) )

等等，直到我无法弄清楚发生了什么。

我不知道这是否只是更深层次的架构缺陷的副作用，或者是否有一个好的方法来处理检查权限和状态值的混合。我是否应该将所有这些疯狂的条件放入一个方法中然后忘记它？但这并不利于下一个继承该项目的程序员。

Answer 1

在您的示例中，这只是一堆布尔逻辑，但可以通过 Compose 方法重构来清理可读性。如果您有一个接受文档和当前用户主体的类，那么您可能会得到类似的内容：

public class DocumentPermissions
{
    private Document document;
    private User user;

    public DocumentPermissions(Document doc, User currentUser)
    {
        document = doc;
        user = currentUser;
    }

    public bool ShouldShowEditButton()
    {
        if(!IsTextDocument())
        {
            return false;
        }
        return IsSuperUser() || IsDocumentOwner() ||  DocumentIsEditable();
    }

    private bool IsTextDocument()
    {
        return document.docTypeId == documentEnum.docType.text;
    }

    private bool IsSuperUser()
    {
        return user.UserStatus == userEnum.Status.SuperUser;
    }

    private bool IsDocumentOwner()
    {
        return user.UserID == document.CreatedByUserId ;
    }

    private bool DocumentIsEditable()
    {
        return document.statusId == documentEnum.docStatus.Editable ;
    }
}

显然这是很多代码，所以我希望您可以重用许多私有方法。

Answer 2

或者你可以使用：

bool showEditButton = (document.statusId == documentEnum.docStatus.Editable); //show if Editable..
showEditButton |= (user.UserStatus == userEnum.Status.SuperUser); //or a superuser or
showEditButton |= (user.UserID == document.CreatedByUserId); //the Creator
showEditButton &= (document.docTypeId == documentEnum.docType.text); //and a text Doc

虽然，我更喜欢瑞安的答案，但我会以另一种方式抛弃它，这种方式至少更具可读性，并为一些评论提供更好的位置。

Answer 3

您暂时拥有它；如果你被分配去重构它，那就重构它。如果您有其他更紧迫的问题，请处理它们，但如果可以的话，您应该花业余时间重构它（不要做得太好，他们可能会让您成为永久所有者）。关于您的其他问题，安全等，信息不足。

http://en.wikipedia.org/wiki/Refactoring

Answer 4

老实说，示例代码看起来还不错。我当然见过更糟糕的情况。

它非常易读，并且没有“魔术字符串”或“魔术数字”。我相信您可以找到更紧迫的清理机会。