Avviso: "formato non una stringa letterale e nessun argomento di formato"

Dall'aggiornamento all'ultimo Xcode 3.2.1 e Snow Leopard, ho ricevuto l'avviso

"formato non una stringa letterale e nessun argomento di formato"

dal seguente codice:

NSError *error = nil;

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

}

Se errorMsgFormatè un NSStringcon specificatori di formato (ad esempio:) "print me like this: %@", cosa c'è di sbagliato nella NSLogchiamata precedente ? E qual è il modo consigliato per risolverlo in modo che l'avviso non venga generato?

Stai annidando correttamente le parentesi? Non credo che gli NSLog()piaccia accettare un solo argomento, che è quello che stai passando. Inoltre, fa già la formattazione per te. Perché non farlo semplicemente?

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

Oppure, poiché dici che errorMsgFormatè una stringa di formato con un unico segnaposto, stai cercando di farlo?

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

Xcode si lamenta perché questo è un problema di sicurezza.

Ecco un codice simile al tuo:

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

L'ultima istruzione NSLog eseguirà l'equivalente di questo:

NSLog(@"Jon Hess %@");

Questo farà sì che NSLog cerchi un altro argomento stringa, ma non ce n'è uno. A causa del modo in cui funziona il linguaggio C, raccoglierà alcuni puntatori di spazzatura casuali dallo stack e proverà a trattarli come un NSString. Molto probabilmente questo farà andare in crash il tuo programma. Ora le tue stringhe probabilmente non contengono% @, ma un giorno potrebbero. Dovresti sempre usare una stringa di formato con i dati che controlli esplicitamente come primo argomento per le funzioni che accettano stringhe di formato (printf, scanf, NSLog, - [NSString stringWithFormat:], ...).

Come sottolinea Otto, probabilmente dovresti fare qualcosa del tipo:

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

Risposta finale: come ha detto Jon Hess, è un problema di sicurezza perché stai passando QUALSIASI stringa a una funzione che si aspetta una stringa di formato. Cioè, valuterà tutti gli specificatori di formato ALL'INTERNO della stringa qualunque. Se non ce ne sono, fantastico, ma se ci sono, potrebbero accadere cose brutte.

La cosa giusta da fare, quindi, è USARE direttamente una stringa di formato, ad esempio

NSLog(@"%@", myNSString);

In questo modo, anche se sono presenti identificatori di formato in myNSString, non vengono valutati da NSLog.

Non consiglio particolarmente di usarlo, poiché l'avviso È un vero avvertimento .. in un uso dinamico del linguaggio è possibile fare cose runtime sulla stringa (cioè inserire nuove informazioni o addirittura mandare in crash il programma) .. Tuttavia è possibile per forzare la soppressione se SAPETE che dovrebbe essere così e davvero non volete essere avvertiti al riguardo ..

#pragma GCC diagnostic ignored "-Wformat-security"

Direbbe a GCC di ignorare temporaneamente l'avviso di compilazione .. Anche in questo caso non risolve nulla, ma a volte non è possibile trovare un buon modo per risolvere effettivamente il problema.

EDIT: A partire dal clang, il pragma è cambiato. Vedi questo: https://stackoverflow.com/a/17322337/3937

Il modo più rapido per risolverlo sarebbe aggiungere @"%@",come primo argomento alla tua NSLogchiamata, cioè

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

Tuttavia, dovresti probabilmente considerare la risposta di Sixteen Otto.

Ho appena passato uno zero per negare gli avvertimenti, forse potrebbe funzionare per te?

NSLog (myString, nil);

Se vuoi eliminare l'avvertimento "formato non una stringa letterale e nessun argomento di formato" una volta per tutte, puoi disabilitare l'impostazione di avviso di GCC "Typecheck Calls to printf / scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) nelle impostazioni di compilazione del tuo target.

NSLog () si aspetta una stringa di formato, ciò che viene passato è solo una stringa. Non è necessario utilizzare stringWithFormat :, puoi semplicemente fare:

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

E questo farebbe scomparire l'avvertimento.

FWIW, questo vale anche per lo sviluppo di iPhone. Sto codificando contro l'SDK 3.1.3 e ho ricevuto lo stesso errore con lo stesso problema (annidamento di stringWithFormat all'interno di NSLog ()). Sixten e Jon sono sui soldi.

Anche solo far sapere a chiunque di utilizzare appendFormatNSMutableString può causare la visualizzazione di questo avviso se si tenta di passare una stringa formattata in questo modo:

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

Quindi, per evitare questo avviso, trasforma quanto sopra in questo:

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

Più conciso e più sicuro. Godere!

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