为什么这个坏代码？

Question

我在这里看到一个关于承认你写过的最糟糕的代码的问题,我不太确定,因为我不知道为什么这是错误的代码.

public string GetUsername (string userName)
{
    User user = DbLookup.GetUser(userName);
    return user.Username;
}

是因为它假定username存在并且不检查null？或者还有更多吗？

/sf/ask/9167581/#191969

Answer 1

因为它返回的是用户作为输入发送给方法的相同内容...用户名

Answer 2

它不会返回提供的相同字符串.它从数据库返回用户名,用户可能存在也可能不存在 - 因此它可以返回null.考虑到它的作用,方法名称可能不正确.有人在原帖中提到它应该是CheckIfUsernameExistsAndReturn sorta方法名.