Ostrzeżenie: „format to nie literał ciągu i brak argumentów formatu”

Od czasu aktualizacji do najnowszej wersji Xcode 3.2.1 i Snow Leopard otrzymuję ostrzeżenie

„format nie jest literałem ciągu i brak argumentów formatu”

z następującego kodu:

NSError *error = nil;

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

}

Jeśli errorMsgFormatjest NSStringz specyfikatorami formatu (np. "print me like this: %@":), Co jest nie tak z powyższym NSLogwywołaniem? Jaki jest zalecany sposób naprawienia tego problemu, aby ostrzeżenie nie było generowane?

Czy prawidłowo zagnieżdżasz zamki? Myślę, że nie NSLog()lubi brać tylko jednego argumentu, czyli tego, co mu zdajesz. Ponadto wykonuje już formatowanie za Ciebie. Dlaczego po prostu tego nie zrobić?

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

A może skoro mówisz, że errorMsgFormatjest to ciąg formatu z pojedynczym symbolem zastępczym, czy próbujesz to zrobić?

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

Xcode narzeka, ponieważ jest to problem z bezpieczeństwem.

Oto kod podobny do twojego:

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

Ta ostatnia instrukcja NSLog będzie wykonywała odpowiednik tego:

NSLog(@"Jon Hess %@");

To spowoduje, że NSLog będzie szukał jeszcze jednego argumentu typu string, ale takiego nie ma. Ze względu na sposób działania języka C, pobierze on ze stosu jakiś losowy wskaźnik śmieci i spróbuje traktować go jak NSString. Najprawdopodobniej spowoduje to zawieszenie programu. Teraz twoje łańcuchy prawdopodobnie nie zawierają% @, ale któregoś dnia mogą. Zawsze należy używać łańcucha formatu z danymi, które jawnie kontrolujesz, jako pierwszego argumentu funkcji pobierających ciągi formatujące (printf, scanf, NSLog, - [NSString stringWithFormat:], ...).

Jak podkreśla Otto, prawdopodobnie powinieneś zrobić coś takiego:

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

Ostateczna odpowiedź: jak powiedział Jon Hess, jest to kwestia bezpieczeństwa, ponieważ przekazujesz ciąg WHATEVER do funkcji oczekującej ciągu formatu. Oznacza to, że oceni wszystkie specyfikatory formatu W RAMACH dowolnego ciągu. Jeśli ich nie ma, super, ale jeśli są, mogą się zdarzyć złe rzeczy.

Zatem właściwą rzeczą do zrobienia jest na przykład bezpośrednie UŻYCIE łańcucha formatującego

NSLog(@"%@", myNSString);

W ten sposób, nawet jeśli w myNSString istnieją specyfikatory formatu, nie są one oceniane przez NSLog.

Nie polecam szczególnie tego używania, ponieważ ostrzeżenie JEST prawdziwym ostrzeżeniem .. w dynamicznym używaniu języka możliwe jest wykonanie pewnych czynności na łańcuchu (np. Wstawienie nowych informacji lub nawet zawieszenie programu) .. Jednak jest to możliwe wymusić tłumienie, jeśli WIESZ, że tak powinno być i naprawdę nie chcesz być o tym ostrzeżony.

#pragma GCC diagnostic ignored "-Wformat-security"

Powiedziałby GCC, aby tymczasowo zignorował ostrzeżenie o kompilacji. Ponownie, to niczego nie rozwiązuje, ale mogą wystąpić chwile, kiedy nie możesz znaleźć dobrego sposobu na faktyczne naprawienie problemu.

EDYCJA: Od czasu clang pragma uległa zmianie. Zobacz: https://stackoverflow.com/a/17322337/3937

Najszybszym sposobem rozwiązania tego problemu byłoby dodanie @"%@",jako pierwszego argumentu do NSLogwywołania, tj.

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

Chociaż prawdopodobnie powinieneś rozważyć odpowiedź Sixteen Otto.

Właśnie przekazałem zero, aby zanegować ostrzeżenia, może to zadziała dla ciebie?

NSLog (myString, nil);

Jeśli chcesz raz na zawsze pozbyć się ostrzeżenia „format nie jest literałem łańcuchowym i brakiem argumentów formatu”, możesz wyłączyć ustawienie ostrzeżenia GCC „Typecheck wywołuje printf / scanf” (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) w ustawieniach kompilacji celu.

NSLog () oczekuje ciągu formatu, a przekazywany jest tylko ciąg. Nie musisz używać stringWithFormat :, możesz po prostu:

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

A to sprawi, że ostrzeżenie zniknie.

FWIW, dotyczy to również programistów iPhone'a. Koduję względem zestawu SDK 3.1.3 i otrzymałem ten sam błąd z tym samym problemem (zagnieżdżenie stringWithFormat wewnątrz NSLog ()). Sixten i Jon szukają pieniędzy.

Po prostu poinformowanie kogokolwiek o użyciu appendFormaton NSMutableString może również spowodować wyświetlenie tego ostrzeżenia, jeśli próbujesz przekazać sformatowany ciąg w następujący sposób:

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

Aby uniknąć tego ostrzeżenia, zamień powyższe na to:

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

Bardziej zwięzłe i bezpieczniejsze. Cieszyć się!

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