Attenzione: "formato non una stringa letterale e senza argomenti formato"

Question

Poiché l'aggiornamento alla più recente Xcode 3.2.1 e Snow Leopard, Sono stato sempre l'avvertimento

formato

" Non una stringa letterale e non argomenti formato"

dal codice seguente:

NSError *error = nil; 

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

} 

If errorMsgFormat è un NSString con identificatori di formato (es: "print me like this: %@"), cosa c'è di sbagliato con la precedente chiamata NSLog? E qual è il modo consigliato per risolverlo in modo che l'avviso non venga generato?

Answer 1

Stai annidando correttamente le parentesi? Non penso che ai NSLog() piaccia prendere solo un argomento, che è ciò che stai passando. Inoltre, fa già la formattazione per te. Perché non farlo?

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

Oppure, dal momento che si dice errorMsgFormat è una stringa di formato con un singolo segnaposto, stai cercando di fare questo?

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

Answer 2

modo più veloce per risolvere il problema sarebbe di aggiungere @"%@", come primo argomento al NSLog chiamata, vale a dire,

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

Anche se, probabilmente si dovrebbe prendere in considerazione la risposta di Sedici Otto.

Answer 3

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

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

E che renderebbe l'avvertimento andare via.

Answer 4

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

Ecco il codice simile al tuo:

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

Quest'ultima affermazione NSLog sta per essere eseguendo l'equivalente di questo:

NSLog(@"Jon Hess %@"); 

che sta per causare NSLog cercare un altro argomento stringa ma non ce n'è uno A causa del modo in cui il linguaggio C funziona, raccoglierà un puntatore casuale dalla pila e cercherà di trattarlo come un NSString. Questo molto probabilmente causerà il crash del tuo programma. Ora le tue stringhe probabilmente non hanno% @ 's in loro, ma un giorno potrebbero. Si dovrebbe sempre usare una stringa di formato con i dati che si controllano esplicitamente come primo argomento delle funzioni che accettano le stringhe di formato (printf, scanf, NSLog, - [NSString stringWithFormat:], ...).

Come Otto fa notare, si dovrebbe probabilmente solo fare qualcosa di simile:

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

Answer 5

Se si vuole sbarazzarsi del avvertimento "formato non una stringa letterale e senza argomenti formato" una volta per tutte, si può disabilita l'impostazione di avviso GCC "Typecheck Calls to printf/scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) nelle impostazioni di costruzione del target.

Answer 6

Ho appena passato un nulla per negare gli avvisi, forse avrebbe funzionato per te?

NSLog (myString, nil);

Answer 7

FWIW, questo vale anche per iPhone dev. Sto codificando con l'SDK 3.1.3 e ho ottenuto lo stesso errore con lo stesso problema (nesting stringWithFormat all'interno di NSLog()). Sixten e Jon sono in the money.

Answer 8

Risposta finale: come ha detto Jon Hess, si tratta di un problema di sicurezza perché si passa una stringa WHATEVER a una funzione che si aspetta una stringa di formato. Vale a dire, valuterà tutti gli specificatori di formato ENTRO la stringa qualunque. Se non ce ne sono, fantastico, ma se ci sono, potrebbero accadere cose brutte.

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

NSLog(@"%@", myNSString); 

In questo modo, anche se ci sono indicatori di formato in myNSString, essi non vengono valutati da NSLog.

Answer 9

Non consiglio particolarmente l'utilizzo di questo, poiché l'avvertimento è un vero avvertimento .. in un uso dinamico del linguaggio è possibile fare cose runtime alla stringa (cioè inserire nuove informazioni o addirittura arrestare il programma). . Tuttavia è possibile forzare sopprimere se si sa che dovrebbe essere così e davvero non si desidera ricevere notifiche su di esso ..

#pragma GCC diagnostic ignored "-Wformat-security"

direbbe GCC di ignorare temporaneamente l'avviso di compilazione .. Ancora una volta non risolve nulla, ma ci possono essere momenti in cui non riesci a trovare un buon modo per risolvere il problema.

MODIFICA: A partire da clang, il pragma è cambiato. Vedere questo: https://stackoverflow.com/a/17322337/3937

Answer 10

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

Answer 11

Basta lasciare che qualcuno sa usare la appendFormat su NSMutableString può anche causare questo avviso a comparire se cercando di passare in una stringa formattata in questo modo:

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

Quindi, per evitare questo avviso , gira il sopra in questo:

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

Più conciso e più sicuro. Godere!