didSelectRowAt中if语句引发myCellChoice未修改警告及功能异常求助

Hey there! Let's break down your problem and fix it together.

First, let's clarify the root cause of that warning: the "Variable 'myCellChoice' was never..." alert pops up because either you declared the variable but never used it in your code, or there are code paths where the variable isn't assigned a value before it's accessed. Since removing the if/else makes the code run (but only shows back training), your logic is almost there—we just need to tweak the if/else structure to cover all cases and ensure myCellChoice is properly utilized.

Step 1: Diagnose the likely code issue #

Chances are your original code looks something like this (where the warning triggers):

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    var myCellChoice: String?
    
    if myButtonChoiceString2 == "肩部训练" {
        // 处理肩部训练内容
        myCellChoice = "肩部训练"
    } else if myButtonChoiceString2 == "背部训练" {
        // 处理背部训练内容
        myCellChoice = "背部训练"
    }
    // 这里可能没用到myCellChoice，或者存在未覆盖的分支导致它没被赋值
}

Either you're not using myCellChoice after assigning it, or there's a scenario where neither condition matches (leaving myCellChoice unassigned), which triggers the warning.

Step 2: Fix the warning and get your logic working #

Here's a revised version that eliminates the warning and correctly switches between shoulder/back training content:

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    // 先给myCellChoice一个默认值，覆盖所有未匹配的情况
    var myCellChoice = "背部训练"
    
    // 切换训练类型逻辑
    if myButtonChoiceString2 == "肩部训练" {
        myCellChoice = "肩部训练"
        // 替换成你的肩部训练数据源更新逻辑
        self.trainingDataSource = shoulderWorkoutData
    } else {
        // 处理背部训练（包括所有非肩部的情况）
        self.trainingDataSource = backWorkoutData
    }
    
    // 确保myCellChoice被使用（比如打印日志或用于后续UI更新）
    print("当前选中训练类型：\(myCellChoice)")
    // 刷新表格展示对应内容
    self.tableView.reloadData()
}

This fixes the warning because:

• myCellChoice is always assigned a value (either the default or the updated value in the if branch)
• We explicitly use the variable (in the print statement, or you could use it for UI labels, etc.)

Step 3: Bonus optimization (avoid string typos!) #

To make your code more robust, replace string comparisons with an enum—this eliminates the risk of typos breaking your logic:

// 定义训练类型枚举
enum TrainingType: String {
    case shoulder = "肩部训练"
    case back = "背部训练"
}

// 在didSelectRowAt中使用枚举判断
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    var myCellChoice: String
    // 转换字符串为枚举值，更安全
    if let selectedType = TrainingType(rawValue: myButtonChoiceString2) {
        switch selectedType {
        case .shoulder:
            myCellChoice = selectedType.rawValue
            self.trainingDataSource = shoulderWorkoutData
        case .back:
            myCellChoice = selectedType.rawValue
            self.trainingDataSource = backWorkoutData
        }
    } else {
        // 处理未知的训练类型，默认显示背部训练
        myCellChoice = TrainingType.back.rawValue
        self.trainingDataSource = backWorkoutData
    }
    
    print("当前选中训练类型：\(myCellChoice)")
    self.tableView.reloadData()
}

This way, you won't accidentally break the logic with a misspelled string like "肩训练" instead of "肩部训练".

内容的提问来源于stack exchange，提问作者Marcus Levi