# 現在のブランチを確認
git branch --show-current

# mainブランチの場合は必ず新しいブランチを作成
# ブランチ名形式: refactor/[content]-[issue-number]
# 例: refactor/reduce-duplication-789

# mainブランチでないことを確認してから作業開始

// Before: 重複コード
public void methodA() {
    User user = getCurrentUser();
    if (user == null) throw new UnauthorizedException();
    // ...
}

public void methodB() {
    User user = getCurrentUser();
    if (user == null) throw new UnauthorizedException();
    // ...
}

// After: ユーティリティメソッド抽出
private User getAuthenticatedUser() {
    User user = getCurrentUser();
    if (user == null) throw new UnauthorizedException();
    return user;
}

public void methodA() {
    User user = getAuthenticatedUser();
    // ...
}

public void methodB() {
    User user = getAuthenticatedUser();
    // ...
}

// Before: 複雑な条件分岐
if (user != null && user.getRole() != null &&
    (user.getRole().equals("ADMIN") || user.getRole().equals("MANAGER"))) {
    // ...
}

// After: メソッド抽出
private boolean isAdminOrManager(User user) {
    return user != null && user.getRole() != null &&
           (user.getRole().equals("ADMIN") || user.getRole().equals("MANAGER"));
}

if (isAdminOrManager(user)) {
    // ...
}

// Before: N+1問題
public List<UserDto> getUsers() {
    List<User> users = userMapper.selectAll();
    return users.stream()
        .map(user -> {
            List<Post> posts = postMapper.selectByUserId(user.getId()); // N回実行
            return UserDto.from(user, posts);
        })
        .collect(Collectors.toList());
}

// After: JOINまたはIN句で一括取得
public List<UserDto> getUsers() {
    List<User> users = userMapper.selectAll();
    List<UUID> userIds = users.stream().map(User::getId).collect(Collectors.toList());
    Map<UUID, List<Post>> postsByUserId = postMapper.selectByUserIds(userIds) // 1回実行
        .stream()
        .collect(Collectors.groupingBy(Post::getUserId));

    return users.stream()
        .map(user -> UserDto.from(user, postsByUserId.get(user.getId())))
        .collect(Collectors.toList());
}

// Before: コンポーネントに直接実装
const UserProfile = () => {
  const [user, setUser] = useState<User | null>(null);
  const [loading, setLoading] = useState(true);
  const [error, setError] = useState<Error | null>(null);

  useEffect(() => {
    // データフェッチロジック
  }, []);

  // ...
};

// After: カスタムフック抽出
const useUserData = (userId: string) => {
  const [user, setUser] = useState<User | null>(null);
  const [loading, setLoading] = useState(true);
  const [error, setError] = useState<Error | null>(null);

  useEffect(() => {
    // データフェッチロジック
  }, [userId]);

  return { user, loading, error };
};

const UserProfile = () => {
  const { user, loading, error } = useUserData(userId);
  // ...
};

// Before: 1つのコンポーネントにすべて
const UserList = () => {
  const [users, setUsers] = useState<User[]>([]);
  // データフェッチ、状態管理、表示が混在

  return (
    <div>
      {/* 表示ロジック */}
    </div>
  );
};

// After: 分離
// Container
const UserListContainer = () => {
  const { users, loading, error } = useUsers();

  if (loading) return <Loading />;
  if (error) return <Error message={error.message} />;

  return <UserListPresentation users={users} />;
};

// Presentational
export const UserListPresentation = ({ users }: { users: User[] }) => {
  return (
    <div>
      {users.map(user => <UserCard key={user.id} user={user} />)}
    </div>
  );
};

cd backend
./gradlew bootRun

cd frontend
pnpm dev

## Refactoring Specialist 完了報告

### リファクタリング内容
- **対象ファイル**: [ファイルパス一覧]
- **改善内容**: [具体的な改善内容]

### 改善効果
- **パフォーマンス**: [改善前/改善後の測定値]
- **可読性**: [具体的な改善点]
- **保守性**: [具体的な改善点]

### テスト結果
- 既存テスト: 全件合格
- 新規テスト: [追加したテスト数] 件
- カバレッジ: [数値]%（維持または向上）

### Regression Guardian検証結果
- デグレーション: なし
- テスト成功率: 100%維持
- ビルド: 成功
- パフォーマンス: 劣化なし（または向上）

### サーバー起動確認
- [ ] サーバー起動成功（Backend: `./gradlew bootRun` または Frontend: `pnpm dev`）
- [ ] リファクタリング対象機能の動作確認済み
- [ ] エラーログなし

### 確認事項
- [ ] 作業前にブランチ確認済み（mainブランチでない）
- [ ] Issue番号確認済み
- [ ] Regression Guardian連携済み
- [ ] テストカバレッジ維持（80%以上）
- [ ] デグレーションなし
- [ ] サーバー起動・動作確認済み

### 次のステップ
リファクタリング完了。Orchestratorへ報告してください。

## Refactoring Specialist 報告

### デグレーション検出

Regression Guardianからデグレーション検出の報告を受けました。

### デグレーション内容
- [具体的なデグレーション内容]

### 原因分析
- [推測される原因]

### 対処方針
**即座にロールバック実施**

### ロールバック実施
- git revert [コミットハッシュ]
- テスト実行: 成功
- ビルド実行: 成功

### 次のステップ
ロールバック完了。リファクタリング計画を再検討します。