Warning: "format not a string literal and no format arguments"

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);

To ostatnie polecenie NSLog będzie wykonywało odpowiednik tego:

NSLog(@"Jon Hess %@");

To spowoduje, że NSLog będzie szukał jeszcze jednego argumentu string, ale go nie ma. Ze względu na sposób, w jaki działa język C, odbierze on losowy wskaźnik śmieci ze stosu i spróbuje traktować go jak NSString. Najprawdopodobniej spowoduje to awarię programu. Teraz twój struny prawdopodobnie nie mają w sobie %@' s, ale pewnego dnia mogą. Jako pierwszy argument funkcji przyjmujących ciągi formatujące (printf, scanf, NSLog, -[NSString stringWithFormat:], należy zawsze używać ciągu formatującego z danymi, które bezpośrednio kontrolujesz ...).

Jak wskazuje Otto, powinieneś prawdopodobnie zrobić coś w stylu:

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

Ostatnia odpowiedź: jak powiedział Jon Hess, jest to problem bezpieczeństwa, ponieważ przekazujesz dowolny ciąg znaków do funkcji oczekującej ciągu formatu. Oznacza to, że oceni wszystkie SPECYFIKATORY FORMATU w ciągu whatever. Jeśli nie ma żadnych, super, ale jeśli są, złe rzeczy mogą się zdarzyć.

Właściwą rzeczą do zrobienia jest użycie bezpośrednio ciągu formatującego, na przykład

NSLog(@"%@", myNSString);

W ten sposób, nawet jeśli w myNSString są określone formaty, nie są one oceniane przez NSLog.

Szczególnie nie polecam tego używać, ponieważ ostrzeżenie jest prawdziwym ostrzeżeniem.. w dynamicznym użyciu języka można wykonywać czynności runtime z łańcuchem znaków (tzn. wstawić nowe informacje lub nawet zawiesić program).. Jednak można wymusić stłumienie, jeśli wiesz, że powinno tak być i naprawdę nie chcesz być ostrzegany przed tym..

#pragma GCC diagnostic ignored "-Wformat-security"

Powie GCC, aby tymczasowo zignorowała ostrzeżenie o kompilacji.. Znowu nic nie rozwiązuje ale może być czasy, kiedy nie możesz znaleźć dobrego sposobu na naprawienie problemu.

EDIT: od clang, pragma się zmieniła. Zobacz to: https://stackoverflow.com/a/17322337/3937

Najszybszym sposobem na poprawienie tego byłoby dodanie @"%@", jako pierwszego argumentu do wywołania NSLog, tzn.

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

Chociaż, prawdopodobnie powinieneś rozważyć szesnaście odpowiedzi Otto.

Właśnie przekazałem zero, żeby zanegować ostrzeżenia, może to by Ci pomogło?

NSLog (myString, nil);

Jeśli chcesz pozbyć się ostrzeżenia" format not a string literal and no format arguments "raz na zawsze, możesz wyłączyć ustawienie Ostrzeżenia GCC "Typecheck Calls to printf / scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) w Ustawieniach budowania celu.

NSLog () oczekuje ciągu w formacie, to, co jest przekazywane, jest tylko ciągiem. Nie musisz używać stringWithFormat:, możesz po prostu zrobić:

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

I to sprawi, że Ostrzeżenie zniknie.

FWIW, dotyczy to również iPhone dev. Koduję na SDK 3.1.3 i mam ten sam błąd z tym samym problemem (zagnieżdżanie stringWithFormat wewnątrz NSLog ()). Sixten i Jon są na forsie.

Po prostu poinformowanie kogokolwiek za pomocą appendFormat na NSMutableString może również spowodować wyświetlenie tego ostrzeżenia, jeśli próbuje przekazać sformatowany łańcuch jak tak:

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

Więc aby uniknąć tego ostrzeżenia, zamień powyższe w to:

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

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