Rork Max が生成した SwiftUI コードを本番品質まで磨き上げる10のリファクタリング・パターン

// ❌ Before — Rork Max がよく出すパターン
struct TaskListView: View {
    @ObservedObject var viewModel = TaskViewModel()  // 再生成される
    var body: some View {
        List(viewModel.tasks) { task in
            TaskRow(task: task)
        }
    }
}

// ✅ After — ライフサイクルが明確
struct TaskListView: View {
    @StateObject private var viewModel = TaskViewModel()  // ビューと同じ寿命
    var body: some View {
        List(viewModel.tasks) { task in
            TaskRow(task: task)
        }
    }
}
 
struct TaskRow: View {
    @ObservedObject var task: Task  // 親から渡される — @ObservedObject が正しい
    var body: some View {
        HStack {
            Text(task.title)
            Spacer()
            if task.isCompleted { Image(systemName: "checkmark") }
        }
    }
}

// ❌ Before — キャンセルされない非同期処理
struct FeedView: View {
    @State private var posts: [Post] = []
    var body: some View {
        List(posts) { Text($0.title) }
            .onAppear {
                Task {
                    posts = try await API.fetchPosts()  // ビュー破棄後も走る
                }
            }
    }
}

// ✅ After — .task はビュー破棄時に自動キャンセル
struct FeedView: View {
    @State private var posts: [Post] = []
    @State private var loadError: Error?
    var body: some View {
        List(posts) { Text($0.title) }
            .task {
                do {
                    posts = try await API.fetchPosts()
                } catch is CancellationError {
                    return  // 静かにキャンセル
                } catch {
                    loadError = error
                }
            }
            .alert("読み込みに失敗しました", isPresented: .constant(loadError != nil)) {
                Button("OK") { loadError = nil }
            }
    }
}

// ❌ Before — メモリを食い続ける
ScrollView {
    LazyVStack {
        ForEach(items) { item in
            ComplexRow(item: item)  // 全行が一度生成されたら破棄されない
        }
    }
}

// ✅ After — List はセル再利用される
List(items) { item in
    ComplexRow(item: item)
        .listRowSeparator(.hidden)
        .listRowBackground(Color.clear)
}
.listStyle(.plain)

// ❌ Before — タイトルが同じだと表示が崩れる
ForEach(comments, id: \.text) { comment in
    CommentRow(text: comment.text)
}

// ✅ After — 一意な ID を使う
struct Comment: Identifiable {
    let id: UUID  // または UUID().uuidString
    let text: String
    let author: String
    let createdAt: Date
}
 
ForEach(comments) { comment in  // Identifiable なら id 指定不要
    CommentRow(comment: comment)
}

// ❌ Before — Xcode 16 の警告 + 古い iOS でクラッシュ
class FeedViewModel: ObservableObject {
    @Published var posts: [Post] = []
    func load() async {
        let result = try? await API.fetchPosts()
        self.posts = result ?? []  // バックグラウンドスレッドから @Published 更新
    }
}

// ✅ After — @MainActor で全メソッドをメインスレッド固定
@MainActor
final class FeedViewModel: ObservableObject {
    @Published private(set) var posts: [Post] = []
    @Published private(set) var isLoading = false
 
    func load() async {
        isLoading = true
        defer { isLoading = false }
        do {
            posts = try await API.fetchPosts()
        } catch {
            // エラーは別の @Published プロパティで保持
        }
    }
}

// ❌ Before — プログラム的に戻れない
NavigationStack {
    HomeView()
}

// ✅ After — パスを保持して制御可能に
@MainActor
final class AppRouter: ObservableObject {
    @Published var homePath = NavigationPath()
    func popToHomeRoot() { homePath.removeLast(homePath.count) }
}
 
struct ContentView: View {
    @StateObject private var router = AppRouter()
    var body: some View {
        NavigationStack(path: $router.homePath) {
            HomeView()
                .navigationDestination(for: Product.self) { product in
                    ProductDetailView(product: product)
                }
                .navigationDestination(for: CheckoutStep.self) { step in
                    CheckoutView(step: step)
                }
        }
        .environmentObject(router)
    }
}

// ❌ Before — 同じ URL を何度もロードする
List(items) { item in
    AsyncImage(url: URL(string: item.imageURL))  // キャッシュなし
        .frame(width: 80, height: 80)
}

// ✅ After — Kingfisher 等のキャッシュライブラリを使う
import Kingfisher
 
List(items) { item in
    KFImage(URL(string: item.imageURL))
        .placeholder { Color.gray.opacity(0.2) }
        .fade(duration: 0.25)
        .resizable()
        .scaledToFill()
        .frame(width: 80, height: 80)
        .clipped()
        .cornerRadius(8)
}

// ❌ Before — 日本語ベタ書き
Button("購読する") { subscribe() }
Text("月額¥980で全機能利用可能")

// ✅ After — Localizable.xcstrings 経由
Button(String(localized: "subscribe.button.title")) { subscribe() }
Text(String(localized: "subscribe.price.monthly"))

// ❌ Before — VoiceOver で「ボタン」としか読まれない
Button(action: addToFavorites) {
    Image(systemName: "heart")
}
 
// ❌ Before — 装飾画像が「画像」と読まれて邪魔
Image("background-pattern")

// ✅ After — 意味のあるラベルとトレイト
Button(action: addToFavorites) {
    Image(systemName: "heart")
}
.accessibilityLabel(isFavorite ? "お気に入りから削除" : "お気に入りに追加")
.accessibilityHint("ダブルタップで状態が切り替わります")
 
Image("background-pattern")
    .accessibilityHidden(true)  // 装飾画像は読み上げ対象外に

// ❌ Before — GitHub に push した瞬間に流出
struct API {
    static let openAIKey = "YOUR_API_KEY_PLACEHOLDER"
    static let stripeKey = "YOUR_STRIPE_PUBLISHABLE_KEY"
}

// ✅ After — Info.plist + 環境別 xcconfig で分離
// API.swift
struct API {
    static var openAIKey: String {
        Bundle.main.object(forInfoDictionaryKey: "OPENAI_API_KEY") as? String ?? ""
    }
}
 
// Debug.xcconfig (gitignore)
OPENAI_API_KEY = sk-debug-xxxxxxxxxxxx
 
// Release.xcconfig (gitignore)
OPENAI_API_KEY = sk-prod-xxxxxxxxxxxx
 
// Info.plist
// <key>OPENAI_API_KEY</key>
// <string>$(OPENAI_API_KEY)</string>