ไม่“ ถ้า (0 == ค่า) …” ทำอันตรายมากกว่าดีไหม? [ปิด]

นี่เป็นหนึ่งในสิ่งที่ฉันเกลียดที่สุดเมื่อฉันเห็นมันในรหัสของคนอื่น ฉันรู้ว่ามันหมายถึงอะไรและทำไมบางคนทำเช่นนี้ ("จะเกิดอะไรขึ้นถ้าฉันใส่ '=' แทนโดยบังเอิญ?") สำหรับฉันมันเหมือนเป็นอย่างมากเมื่อเด็กคนหนึ่งเดินลงบันไดเพื่อนับก้าวออกมาดัง ๆ

อย่างไรก็ตามนี่คือข้อโต้แย้งของฉันกับมัน:

• มันขัดขวางการไหลตามธรรมชาติของการอ่านรหัสโปรแกรม เรามนุษย์พูดว่า "ถ้าค่าเป็นศูนย์" และไม่ใช่ "ถ้าเป็นศูนย์คือค่า"
• คอมไพเลอร์สมัยใหม่เตือนคุณเมื่อคุณมีงานที่มอบหมายในสภาพของคุณหรือที่จริงแล้วถ้าเงื่อนไขของคุณประกอบด้วยเพียงการมอบหมายนั้นซึ่งใช่ดูน่าสงสัยอยู่แล้ว
• คุณไม่ควรลืมใส่ double '=' เมื่อคุณเปรียบเทียบค่าหากคุณเป็นโปรแกรมเมอร์ คุณอาจลืมใส่ "!" เช่นกัน เมื่อทดสอบความไม่เท่าเทียมกัน

อ๊ะใช่ "เงื่อนไขของโยดา" ("ถ้าค่าเป็นศูนย์ให้รันโค้ดนี้คุณต้อง!") ฉันมักจะชี้ให้ทุกคนที่อ้างว่าพวกเขา "ดีกว่า" ที่เครื่องมือเช่นผ้าสำลี (1) ปัญหานี้ได้รับการแก้ไขตั้งแต่ปลายยุค 70 ภาษาสมัยใหม่ส่วนใหญ่จะไม่ได้รวบรวมการแสดงออกเช่นif(x = 10)พวกเขาปฏิเสธที่จะบีบบังคับผลของการมอบหมายให้บูลีน

อย่างที่คนอื่น ๆ พูดกันแน่นอนว่ามันไม่ใช่ปัญหา แต่มันก่อให้เกิดความไม่ลงรอยกันทางปัญญาเล็กน้อย

มันน่ารังเกียจเพราะเรียกเก็บภาษีจิตขนาดเล็ก แต่เห็นได้ชัด

ผู้คนอ่านจากซ้ายไปขวาในแทบทุกภาษาโปรแกรม (และภาษาที่เป็นธรรมชาติที่สุด)

ถ้าฉันเห็น123 == xวิธีที่ฉันใช้ในการแยกวิเคราะห์จิตใจ:

• 123- เพื่ออะไร ข้อมูลไม่สมบูรณ์
• == - ดี 123 คือ 123 ทำไมต้องทดสอบ ...
• x- ตกลงนั่นคือสิ่งที่เราเป็นห่วง ตอนนี้ฉันมีบริบทเท่านั้น
• กลับไปที่การพิจารณาอีกครั้ง123และทำไมจึงเปรียบเทียบกับ x

เมื่อฉันเห็นการx == 123แยกวิเคราะห์ทางจิตคือ:

• x- ให้บริบทฉันรู้ว่าเงื่อนไขเป็นอย่างไร ฉันอาจเลือกที่จะไม่สนใจส่วนที่เหลือ ขึ้นอยู่กับขั้นตอนก่อนหน้านี้ฉันมีความคิดที่ดีว่าเพราะอะไรและจะเกิดอะไรขึ้น (และต้องแปลกใจถ้ามันแตกต่างกัน)
• == - ฉันคิดอย่างนั้น
• 123 - ได้.

การหยุดชะงักมีขนาดเล็ก (ในตัวอย่างง่ายๆ) แต่ฉันมักจะสังเกตเห็นมัน

ใส่ค่าแรกอาจเป็นความคิดที่ดีถ้าคุณต้องการที่if (LAUNCH_NUKES == cmd)จะดึงความสนใจกับมันเช่น ปกติแล้วนี่ไม่ใช่ความตั้งใจ

อันตรายหรือไม่ ไม่มันทำงานได้ทั้งสองทาง

การปฏิบัติที่ไม่ดี? เป็นที่ถกเถียงกันมากที่สุด มันเป็นโปรแกรมป้องกันอย่างง่าย

คุ้มค่าที่จะนอนไม่หลับใช่มั้ย Nah

นี่คือ flaimbait โดยทั่วไป

ไม่มันไม่ได้ทำอันตรายมากกว่าความดี ง่าย

คำอื่น ๆ

อาร์กิวเมนต์คอมไพเลอร์? เอ่อบางทีอาจจะ - อย่าใส่ความเชื่อมั่นมากเกินไปในการเขียนเพื่อช่วยคุณจากตัวคุณเอง

"คุณไม่ควรลืม" - ดีduh - ไม่มีของหลักสูตรที่คุณไม่ควรขณะที่ฉันเหนื่อยฉันได้รับการเข้ารหัสทุกวันที่ฉันได้มีการใช้สองภาษาที่แตกต่างกันและบางครั้งเพียงแค่บางครั้งการเป็นมนุษย์ที่ฉันทำ ความผิดพลาด.

ประเด็นของพฤติกรรมแบบนี้ก็คือการป้องกันมันไม่ได้อยู่ที่นั่นเพราะคุณคาดว่าจะทำผิดพลาดมากกว่าที่คุณมีประกันเพราะคุณคาดหวังว่าจะผิดพลาด ... แต่ถ้าคุณทำดีที่ได้รับความคุ้มครอง

อ่านยาก คุณกำลังบ่นว่าโปรแกรมเมอร์ที่ดีควรมี == เดินสาย (ซึ่งทำให้ทุกชนิดของสมมติฐานที่ไม่ดี) แต่ที่โปรแกรมเมอร์ที่ดีเหมือนกันตัวเองไม่สามารถอ่าน 0 == ค่า ??

ไม่เป็นอันตรายมีผลประโยชน์ที่อาจเกิดขึ้นคำถามงี่เง่าปล่อยให้คนอื่นทำถ้าพวกเขาต้องการและดำเนินการต่อไป

ฉันจะไม่เรียกมันว่าเป็นอันตราย แต่มันน่ารังเกียจ ดังนั้นฉันจะไม่บอกว่าจะทำ

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

บางคนใช้เพื่อทำให้ชัดเจนว่าสิ่งที่มีเงื่อนไขกำลังทำอยู่ ตัวอย่างเช่น

วิธีที่ 1:

FILE *fp;

fp = fopen("foo.txt", "w+");
if (fp == NULL) {

วิธีที่ 2:

FILE *fp;

if (NULL == (fp = fopen("foo.txt", "w+"))) {

บางคนรู้สึกว่าตัวอย่างที่สองนั้นมีความกระชับมากกว่าหรือมีข้อโต้แย้งย้อนกลับแสดงให้เห็นถึงจุดของการทดสอบ (มีเงื่อนไข) ก่อนการทดสอบ

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

ผสมให้เข้ากับประเด็นที่ดูเหมือนว่ามีคนแตกต่างกันถึงหกคนที่มีสไตล์ที่เป็นเอกลักษณ์ของตัวเองที่ทำงานในครั้งเดียวฉันรู้สึกรำคาญเล็กน้อย

สำหรับฉันมันเป็นเงื่อนไขที่เรียบง่าย ในฐานะคนที่เรียนรู้ C (ในยุค 90) C และ C ++ ฉันเริ่มคุ้นเคยกับมันมากขึ้นและยังคงใช้มันแม้ว่าเหตุผลจะลดน้อยลง

เมื่อคุณ "ปรับ" เพื่อดูด้านซ้ายสำหรับ "คงที่" มันจะกลายเป็นธรรมชาติที่สอง

ฉันยังใช้เพื่อความเท่าเทียมเท่านั้น

ฉันเห็นด้วยกับคำตอบของ @ Wonko

กรณีหนึ่งที่ฉันพบว่ามีประโยชน์คือส่วนที่ตัวแปรของ if นั้นค่อนข้างยาวและการเห็นค่าต่างๆทำให้โค้ดอ่านง่ายขึ้น ภาษาเนมสเปซประมีตัวอย่างที่ดีที่สุด

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

if (2 <= application.httpcontext.current.session["thenameofmysessiontoken"].items.count())

เป็นที่ยอมรับในตัวอย่างนี้มีวิธีอื่นในการทำเช่นนี้ แต่ในกรณีนี้อาจเป็นรุ่นที่อ่านได้ครั้งแรก

และยังมีข้อผิดพลาดเกิดขึ้น และบางครั้งคุณต้องการการมอบหมายในโอเปอเรเตอร์ลูปที่คุณอาจตรวจสอบความเท่าเทียมกันหรืออย่างน้อยก็เป็นแบบฝึกหัดมาตรฐานที่จะใช้

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

ดังนั้นฉันจะพูดว่า:

if ( 0 <= value )

แต่ฉันก็จะพูดว่า:

if ( value <= 100 )

ความเท่าเทียมกันฉันมักจะตรวจสอบกับตัวแปรทางซ้ายแม้ว่ามันจะอ่านง่ายขึ้น