คำเตือน:“ format not a string literal and no format arguments”

ตั้งแต่อัปเกรดเป็น Xcode 3.2.1 และ Snow Leopard ล่าสุดฉันได้รับคำเตือน

"format ไม่ใช่สตริงลิเทอรัลและไม่มีอาร์กิวเมนต์รูปแบบ"

จากรหัสต่อไปนี้:

NSError *error = nil;

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

}

ถ้าerrorMsgFormatเป็นNSStringกับตัวระบุรูปแบบ (เช่น :) "print me like this: %@"มีอะไรผิดปกติกับการNSLogเรียกข้างต้น? และวิธีใดที่แนะนำในการแก้ไขเพื่อไม่ให้เกิดคำเตือน

คุณวางวงเล็บของคุณถูกต้องหรือไม่? ฉันไม่คิดว่าNSLog()ชอบการโต้เถียงเพียงเรื่องเดียวซึ่งเป็นสิ่งที่คุณผ่านมันไป นอกจากนี้ยังจัดรูปแบบให้คุณอยู่แล้ว ทำไมไม่ทำแบบนี้

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

หรือเนื่องจากคุณบอกว่าerrorMsgFormatเป็นสตริงรูปแบบที่มีตัวยึดตำแหน่งเดียวคุณกำลังพยายามทำสิ่งนี้หรือไม่?

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

Xcode กำลังบ่นเนื่องจากนี่เป็นปัญหาด้านความปลอดภัย

นี่คือรหัสที่คล้ายกับของคุณ:

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

คำสั่ง NSLog สุดท้ายจะดำเนินการเทียบเท่ากับสิ่งนี้:

NSLog(@"Jon Hess %@");

นั่นจะทำให้ NSLog มองหาอาร์กิวเมนต์สตริงอีกหนึ่งชุด แต่ไม่มีเลย เนื่องจากวิธีการทำงานของภาษา C มันจะหยิบตัวชี้ขยะแบบสุ่มจากสแต็กและพยายามปฏิบัติเหมือน NSString ซึ่งมักจะทำให้โปรแกรมของคุณขัดข้อง ตอนนี้สตริงของคุณอาจไม่มี% @ อยู่ในนั้น แต่สักวันก็อาจ คุณควรใช้สตริงรูปแบบกับข้อมูลที่คุณควบคุมอย่างชัดเจนเป็นอาร์กิวเมนต์แรกของฟังก์ชันที่ใช้สตริงรูปแบบ (printf, scanf, NSLog, - [NSString stringWithFormat:], ... )

ดังที่ Otto ชี้ให้เห็นคุณควรทำบางสิ่งเช่น:

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

คำตอบสุดท้าย: ดังที่ Jon Hess กล่าวว่าเป็นปัญหาด้านความปลอดภัยเนื่องจากคุณกำลังส่งสตริง WHATEVER ไปยังฟังก์ชันที่คาดว่าจะมีสตริงรูปแบบ นั่นคือมันจะประเมินตัวระบุรูปแบบทั้งหมดภายในสตริงใด ๆ ถ้าไม่มีก็น่ากลัว แต่ถ้ามีสิ่งเลวร้ายอาจเกิดขึ้นได้

สิ่งที่ควรทำคือใช้สตริงรูปแบบโดยตรงตัวอย่างเช่น

NSLog(@"%@", myNSString);

ด้วยวิธีนี้แม้ว่าจะมีตัวระบุรูปแบบใน myNSString แต่ก็ไม่ได้รับการประเมินโดย NSLog

ฉันไม่แนะนำให้ใช้สิ่งนี้เป็นพิเศษเนื่องจากคำเตือนเป็นคำเตือนจริง .. ในการใช้ภาษาแบบไดนามิกจึงเป็นไปได้ที่จะทำสิ่งต่างๆรันไทม์กับสตริง (เช่นแทรกข้อมูลใหม่หรือแม้แต่ทำให้โปรแกรมหยุดทำงาน) .. อย่างไรก็ตามเป็นไปได้ บังคับปราบปรามถ้าคุณรู้ว่ามันควรจะเป็นแบบนี้และคุณไม่อยากถูกเตือนจริงๆ ..

#pragma GCC diagnostic ignored "-Wformat-security"

จะบอกให้ GCC เพิกเฉยต่อคำเตือนการรวบรวมชั่วคราว .. อีกครั้งมันไม่ได้แก้ปัญหาอะไรเลย แต่อาจมีบางครั้งที่คุณไม่สามารถหาวิธีที่ดีในการแก้ไขปัญหาได้จริง

แก้ไข: เมื่อเสียงดังขึ้น pragma มีการเปลี่ยนแปลง ดูสิ่งนี้: https://stackoverflow.com/a/17322337/3937

วิธีที่เร็วที่สุดในการแก้ไขคือเพิ่ม@"%@",อาร์กิวเมนต์แรกในการNSLogโทรของคุณนั่นคือ

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

อย่างไรก็ตามคุณควรพิจารณาคำตอบของอ็อตโตสิบหก

ฉันเพิ่งผ่านศูนย์เพื่อลบล้างคำเตือนนั่นอาจจะเหมาะกับคุณ?

NSLog (myString ศูนย์);

หากคุณต้องการกำจัดคำเตือน "รูปแบบไม่ใช่สตริงลิเทอรัลและไม่มีอาร์กิวเมนต์รูปแบบ" เพียงครั้งเดียวคุณสามารถปิดใช้งานการตั้งค่าคำเตือน GCC "Typecheck Calls to printf / scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) ในการตั้งค่าบิลด์ของเป้าหมาย

NSLog () คาดหวังสตริงรูปแบบสิ่งที่ส่งผ่านเป็นเพียงสตริง คุณไม่จำเป็นต้องใช้ stringWithFormat: คุณสามารถทำได้:

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

และนั่นจะทำให้คำเตือนหายไป

FWIW สิ่งนี้ใช้ได้กับ iPhone dev เช่นกัน ฉันกำลังเข้ารหัสกับ 3.1.3 SDK และมีข้อผิดพลาดเดียวกันกับปัญหาเดียวกัน (การซ้อน stringWithFormat ภายใน NSLog ()) Sixten และ Jon อยู่ในเงิน

การแจ้งให้ทุกคนทราบโดยใช้appendFormatบน NSMutableString ยังสามารถทำให้คำเตือนนี้ปรากฏขึ้นหากพยายามส่งผ่านสตริงที่จัดรูปแบบดังนี้:

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

ดังนั้นเพื่อหลีกเลี่ยงคำเตือนนี้ให้เปลี่ยนด้านบนเป็นสิ่งนี้:

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

กระชับและปลอดภัยมากขึ้น สนุก!

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