混乱的方法 - if/then 结构和字符串操作问题

Question

我有一个方法，只要我的视图上的控件发生更改，它就会被调用，并且应该更新UILabel。它有两个UITextFields 和两个UISliders。首先，我检查UITextFields 中的任何一个是否为空，如果是，建议它们需要填写。否则我得到UITextFields' 值之间的差异并生成几个floats在我的NSStrings 中使用。

我收到一条警告说 message 没有被使用，我收到一个关于 NSStrings 的错误（现在不记得具体是什么了 - 我不在我的 Mac 上...）

即使我将消息简化为简单有效的方法，当delta == 0 时，它也会发送delta <= 0 消息。

哦，字符串不会将值放在 % 符号所在的位置，它们只是打印 % 符号。

我已经破解了太久，需要帮助......

- (void)updateAdvice {
    if ([chlorineSourceField.text isEqualToString:@""] || [poolVolumeField.text isEqualToString:@""]) {
        NSString *message =  [[NSString alloc] initWithString:@"Enter a chlorine source and pool volume."];
    }
    else {
        int delta = [targetLabel.text intValue] - [startingLabel.text intValue];
        float chlorineAmount = delta * [poolVolumeField.text intValue] * chlorineConstant;
        float percentRemove = (1 - ([targetLabel.text floatValue] / [startingLabel.text floatValue]));
        float gallonsRemove = percentRemove * [poolVolumeField.text intValue];
        if (delta == 0) {
            NSString *message = [[NSString alloc] initWithString:@"No adjustments necessary.  You're on target"];
        }
        if (delta >= 0) {
            NSString *message = [[NSString alloc] initWithFormat:@"To increase FC by %dppm, add %3.1f oz of %@.", delta, chlorineAmount, chlorineSourceField.text];
        }
        if (delta <= 0) {
            NSString *message = [[NSString alloc] initWithFormat:@"You're above target already.  Replace %d%% or %d gallons of water - or just wait for it to come down.", percentRemove*100, gallonsRemove];
        }
    }
    adviceLabel.text = message;
    [message release];
}

Answer 1

您缺少else 部分，因此所有三个if 语句都被连续评估。如果delta == 0，它将满足所有三个if 语句。因此，最后一个分配将覆盖前两个。 （而且你会泄漏内存）

此外，您的 message 变量的作用域是它所声明的 if 块的本地变量。您可能希望在函数级别范围内移动 message 声明。

就% 不工作而言，您正在使用实例初始化程序initWithFormat 和类初始化程序stringWithFormat 的语法。 initWithFormat 在单独的参数 - arguments: 中获取格式化字符串的参数。 （顺便说一句，它还需要locale:）

Answer 2

就message 变量而言，它是有范围的，因此不能在声明它的if/else 语句之外使用。您想在if 语句之前 声明它，因此您可以在if 语句之外使用它。所以是这样的：

- (void)updateAdvice {
    NSString *message = nil;

    if ([chlorineSourceField.text isEqualToString:@""] || [poolVolumeField.text isEqualToString:@""]) {
        message =  [[NSString alloc] initWithString:@"Enter a chlorine source and pool volume."];
    }
    else {
        int delta = [targetLabel.text intValue] - [startingLabel.text intValue];
        float chlorineAmount = delta * [poolVolumeField.text intValue] * chlorineConstant;
        float percentRemove = (1 - ([targetLabel.text floatValue] / [startingLabel.text floatValue]));
        float gallonsRemove = percentRemove * [poolVolumeField.text intValue];
        if (delta == 0) {
            message = [[NSString alloc] initWithString:@"No adjustments necessary.  You're on target"];
        }
        if (delta >= 0) {
            message = [[NSString alloc] initWithFormat:@"To increase FC by %dppm, add %3.1f oz of %@.", delta, chlorineAmount, chlorineSourceField.text];
        }
        if (delta <= 0) {
            message = [[NSString alloc] initWithFormat:@"You're above target already.  Replace %d%% or %d gallons of water - or just wait for it to come down.", percentRemove*100, gallonsRemove];
        }
    }
    adviceLabel.text = message;
    [message release];
}