public class BuyerHelper
{
    private static final UserRepository userRepository;

    public static void createAndCheckBuyer(Cart cart)
    {
        int userId = cart.getUserId();
        User user = userRepository.getUserById(userId);

        Buyer buyer = BuyFactory.createFrom(user);
        BaseCheckResult checkResult 
 
= BuyerChecker.checkBuyerPermission(buyer);
        
        if (checkResult.isSuccess())
            cart.setBuyer(buyer);
        else
            throw new InvaildBuyerException();        
    }
}

很簡單的一段程式碼，存在以下問題：

1. 責任不清晰。

該方法既包含了驗證使用者是否是一個合法買家的功能，又實現了根據使用者建立買家的功能，並且還隱藏了一個很危險的功能，它還負責將這個建立好的買家設定到cart的屬性。而cart是一個外來引數，這意味著後續的程式碼中很有可能會依賴這個被附加到cart上的buyer屬性上，而這一點其實是比較隱晦的。

2. 建立非法物件。

本方法實現的功能應該是判斷一個使用者是否是一個有效的買家，理論上來說如果一個使用者不是有效買家的話，就不應該出現一個買家物件。但是，在這段程式碼中，確實先不管三七二十一先把Buyer物件創建出來，然後再對它進行驗證。我們應該盡力避免建立非法物件。