public Optional<User> findUser(String userId) {
    User user = userRepository.findById(userId);
    if (user == null) {
        return null;
    }
    return Optional.of(user);
}

public String userName(String userId) {
    return findUser(userId).get().name();
}

@Reviewer: `Optional` を返す契約なのに `null` を返しているため、呼び出し側はOptionalとnullの両方を考慮する必要があります。未存在は `Optional.empty()` に統一してください。

public Optional<User> findUser(String userId) {
    return Optional.ofNullable(userRepository.findById(userId));
}

@Reviewer: 戻り値が `Optional` である以上、メソッドは常にOptionalインスタンスを返す契約にしてください。未存在は `Optional.empty()`、異常は例外やエラー型で分けてください。

@Reviewer: `findUser(...).get()` により未存在時の分岐が呼び出し側で表現されていません。`orElseThrow` で例外契約を明示するか、未存在時の代替処理をコードに出してください。

User user = findUser(userId)
    .orElseThrow(() -> new UserNotFoundException(userId));

@Reviewer: 複数件検索の戻り値が `Optional>` になっていますが、0件は正常結果に見えます。未取得と0件を分ける要件がないなら、空リストを返す契約にしてください。

public Optional<User> findUser(String userId) {
    return userRepository.findById(userId);
}

public User requireUser(String userId) {
    return findUser(userId)
        .orElseThrow(() -> new UserNotFoundException(userId));
}

public List<User> searchUsers(UserSearchCondition condition) {
    return userRepository.search(condition);
}

public UserProfile getProfile(String userId) {
    User user = requireUser(userId);
    return profileMapper.toProfile(user);
}

public Optional<User> findUser(String userId) {
    try {
        return userRepository.findById(userId);
    } catch (SQLException e) {
        return Optional.empty();
    }
}

@Reviewer: DBエラーを `Optional.empty()` に変換しており、未存在と障害が区別できません。存在しないことが正常な場合だけemptyにし、取得失敗は例外として文脈を残してください。

@Reviewer: DTOフィールドが `Optional` になっていますが、シリアライズやBean validationとの境界が分かりにくくなります。外部入出力ではnullable fieldとして扱い、ドメイン境界でOptional戻り値に変換する方が読みやすいか確認してください。