警告：“格式不是字符串文字，也没有格式参数”

Question

自从升级到最新的 Xcode 3.2.1 和 Snow Leopard 以来，我一直收到警告

“格式不是字符串文字，也没有格式参数”

来自以下代码的

NSError *error = nil;

if (![self.managedObjectContext save:&error]) 
{
    NSLog([NSString stringWithFormat:@"%@ %@, %@", 
       errorMsgFormat, 
       error, 
       [error userInfo]]);      

}

：如果 errorMsgFormat 是带有格式说明符的 NSString（例如：“像这样打印我： %@")，上面的 NSLog 调用有什么问题？建议的修复方法是什么以免生成警告？

Answer 1

Xcode 正在抱怨，因为这是一个安全问题。

下面是与您的代码类似的代码：

NSString *nameFormat = @"%@ %@";
NSString *firstName = @"Jon";
NSString *lastName = @"Hess %@";
NSString *name = [NSString stringWithFormat:nameFormat, firstName, lastName];
NSLog(name);

最后一条 NSLog 语句将执行与此等效的操作：

NSLog(@"Jon Hess %@");

这将导致 NSLog 寻找另一个字符串参数，但没有一个。由于 C 语言的工作方式，它将从堆栈中拾取一些随机垃圾指针，并尝试将其视为 NSString。这很可能会让你的程序崩溃。现在您的字符串中可能没有 %@，但有一天它们可能会。您应该始终使用包含明确控制的数据的格式字符串作为采用格式字符串的函数（printf、scanf、NSLog、-[NSString stringWithFormat:]、...）的第一个参数。

正如奥托指出的那样，你可能应该这样做：

NSLog(errorMsgFormat, error, [error userInfo]);

Answer 2

您是否正确嵌套括号？我不认为 N​​SLog() 喜欢只接受一个参数，这就是您传递给它的参数。此外，它已经为您完成了格式化。为什么不直接这样做呢？

NSLog(@"%@ %@, %@", 
   errorMsgFormat, 
   error, 
   [error userInfo]);              

或者，既然您说 errorMsgFormat 是带有单个占位符的格式字符串，那么您是否尝试这样做？

NSLog(@"%@, %@", [NSString stringWithFormat:errorMsgFormat, error], 
   [error userInfo]);              

Answer 3

最终答案：正如 Jon Hess 所说，这是一个安全问题，因为您将 WHATEVER 字符串传递给需要格式字符串的函数。也就是说，它将评估任何字符串内的所有格式说明符。如果没有，那就太好了，但如果有，则可能会发生不好的事情。

那么，正确的做法是直接使用格式字符串，例如

NSLog(@"%@", myNSString);

这样，即使 myNSString 中有格式说明符，NSLog 也不会评估它们。

Answer 4

我不特别推荐使用这个，因为警告是一个真正的警告..在动态使用该语言时，可以在运行时对字符串执行操作（即插入新信息甚至使程序崩溃）..但是这是可能的如果您知道它应该是这样的并且您真的不想收到警告，则强制抑制。

#pragma GCC Diagnostic Beenigned "-Wformat-security"

会告诉 GCC 暂时忽略编译警告..同样，它并不能解决任何问题，但有时您可能找不到真正解决问题的好方法。

编辑：从 clang 开始，编译指示已经改变。请参阅：https://stackoverflow.com/a/17322337/3937

Answer 5

解决此问题的最快方法是将 @"%@", 添加为 NSLog 调用的第一个参数，即，

NSLog(@"%@", [NSString stringWithFormat: ....]);

尽管如此，您可能应该考虑十六奥托的答案。

Answer 6

我刚刚传递了一个 nil 来否定警告，也许这对你有用？

NSLog(myString, nil);

Answer 7

如果您想一劳永逸地摆脱警告“格式不是字符串文字且没有格式参数”，您可以在目标的构建设置中禁用 GCC 警告设置“Typecheck Calls to printf/scanf”(GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO)。

Answer 8

NSLog() 需要一个格式字符串，传入的只是一个字符串。您不需要使用 stringWithFormat:，您只需执行以下操作：

NSLog(@"%@ %@, %@", errorMsgFormat, error, [error userInfo])

这将使警告消失离开。

Answer 9

FWIW，这也适用于 iPhone 开发人员。我正在针对 3.1.3 SDK 进行编码，并且遇到了同样的错误和同样的问题（在 NSLog() 中嵌套 stringWithFormat）。 Sixten 和 Jon 很赚钱。

Answer 10

如果尝试像这样传入格式化字符串，只是让任何人知道在 NSMutableString 上使用 appendFormat 也会导致出现此警告：

NSMutableString *csv = [NSMutableString stringWithString:@""];
NSString *csvAddition = [NSString stringWithFormat:@"%@",WHATEVERYOUAREPUTTINGINYOURSTRING];
[csv appendFormat:csvAddition];

因此，为了避免此警告，请将上面的内容变成这样：

NSMutableString *csv = [NSMutableString stringWithString:@""];
[csv appendFormat:@"%@",WHATEVERYOUAREPUTTINGINYOURSTRING];

更简洁、更多安全的。享受！

Answer 11

NSLog(@"%@ %@, %@", 
       errorMsgFormat, 
       error, 
       [error userInfo]);