Avertissement: "format pas un littéral de chaîne et aucun argument de format"

Depuis la mise à niveau vers le dernier Xcode 3.2.1 et Snow Leopard, j'ai reçu l'avertissement

"format pas un littéral de chaîne et aucun argument de format"

à partir du code suivant:

NSError *error = nil;

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

}

Si errorMsgFormatest un NSStringavec des spécificateurs de format (par exemple:) "print me like this: %@", quel est le problème avec l' NSLogappel ci - dessus ? Et quelle est la méthode recommandée pour résoudre ce problème afin que l'avertissement ne soit pas généré?

Emboitez-vous correctement vos supports? Je ne pense pas que j'aime NSLog()prendre un seul argument, c'est ce que vous lui transmettez. En outre, il effectue déjà le formatage pour vous. Pourquoi ne pas faire ça?

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

Ou, puisque vous dites qu'il errorMsgFormats'agit d'une chaîne de format avec un seul espace réservé, essayez-vous de le faire?

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

Xcode se plaint car c'est un problème de sécurité.

Voici un code similaire au vôtre:

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

Cette dernière instruction NSLog va exécuter l'équivalent de ceci:

NSLog(@"Jon Hess %@");

Cela va amener NSLog à rechercher un autre argument de chaîne, mais il n'y en a pas. En raison de la façon dont le langage C fonctionne, il va récupérer un pointeur de déchets aléatoire dans la pile et essayer de le traiter comme une NSString. Cela plantera très probablement votre programme. Maintenant, vos chaînes ne contiennent probablement pas de% @, mais un jour, elles le pourraient. Vous devez toujours utiliser une chaîne de format avec des données que vous contrôlez explicitement comme premier argument des fonctions qui acceptent des chaînes de format (printf, scanf, NSLog, - [NSString stringWithFormat:], ...).

Comme le souligne Otto, vous devriez probablement faire quelque chose comme:

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

Réponse finale: comme Jon Hess l'a dit, c'est un problème de sécurité car vous passez une chaîne WHATEVER à une fonction qui attend une chaîne de format. Autrement dit, il évaluera tous les spécificateurs de format DANS la chaîne de caractères. S'il n'y en a pas, c'est génial, mais s'il y en a, de mauvaises choses pourraient arriver.

La bonne chose à faire, alors, est d'utiliser directement une chaîne de format, par exemple

NSLog(@"%@", myNSString);

De cette façon, même s'il existe des spécificateurs de format dans myNSString, ils ne sont pas évalués par NSLog.

Je ne recommande pas spécialement de l'utiliser, car l'avertissement EST un vrai avertissement .. dans une utilisation dynamique du langage, il est possible de faire des choses à l'exécution sur la chaîne (c'est-à-dire insérer de nouvelles informations ou même planter le programme) .. Cependant, c'est possible pour forcer la suppression si vous SAVEZ que cela devrait être comme ça et que vous ne voulez vraiment pas être averti à ce sujet.

#pragma GCC diagnostic ignored "-Wformat-security"

Je dirais à GCC d'ignorer temporairement l'avertissement de compilation. Encore une fois, cela ne résout rien, mais il peut arriver que vous ne puissiez pas trouver un bon moyen de résoudre le problème.

EDIT: Depuis clang, le pragma a changé. Voir ceci: https://stackoverflow.com/a/17322337/3937

Le moyen le plus rapide de résoudre ce problème serait d'ajouter @"%@",comme premier argument à votre NSLogappel, c'est-à-dire,

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

Cependant, vous devriez probablement considérer la réponse de Sixteen Otto.

Je viens de passer un zéro pour annuler les avertissements, peut-être que cela fonctionnerait pour vous?

NSLog (myString, nil);

Si vous voulez vous débarrasser de l'avertissement "format pas un littéral de chaîne et aucun argument de format" une fois pour toutes, vous pouvez désactiver le paramètre d'avertissement GCC "Typecheck Calls to printf / scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) dans les paramètres de construction de votre cible.

NSLog () attend une chaîne de format, ce qui est passé n'est qu'une chaîne. Vous n'avez pas besoin d'utiliser stringWithFormat :, vous pouvez simplement faire:

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

Et cela ferait disparaître l'avertissement.

FWIW, cela s'applique également aux développeurs iPhone. Je suis en train de coder avec le SDK 3.1.3 et j'ai eu la même erreur avec le même problème (nesting stringWithFormat dans NSLog ()). Sixten et Jon sont sur l'argent.

Le simple fait de faire savoir à quiconque d'utiliser le appendFormaton NSMutableString peut également provoquer l'apparition de cet avertissement si vous essayez de transmettre une chaîne formatée comme ceci:

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

Donc, pour éviter cet avertissement, transformez ce qui précède en ceci:

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

Plus concis et plus sûr. Prendre plaisir!

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