การจัดฟันแบบหยิกควรปรากฏบนเส้นของตัวเองหรือไม่? [ปิด]

การจัดฟันแบบหยิกควรอยู่ในแนวของตนเองหรือไม่? คุณคิดยังไงกับเรื่องนี้?

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

หรือมันควรจะเป็น

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

หรือแม้กระทั่ง

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

โปรดสร้างสรรค์! อธิบายว่าทำไมแบ่งปันประสบการณ์สำรองข้อมูลด้วยข้อเท็จจริงและการอ้างอิง

เมื่อฉันเป็นนักเรียนฉันเคยใส่เครื่องมือจัดฟันแบบหยิกเป็นเส้นเดียวกันเพื่อให้มีจำนวนบรรทัดน้อยลงและโค้ดจะพิมพ์บนหน้ากระดาษน้อยลง การดูตัวอักษรวงเล็บเดี่ยวที่พิมพ์ออกมาเป็นสิ่งเดียวในบรรทัดนั้นน่ารำคาญ (สภาพแวดล้อมการสิ้นเปลืองกระดาษ)

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

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

คุณไม่ควรทำวิธีที่ 3

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

เขียนรหัสของคุณเพื่อคนอื่น

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

เป็นที่สอดคล้องกันเป็นสิ่งที่สำคัญแม้ว่า ฉันเลยบอกว่าให้เราพลิกเหรียญแล้วเขียนโค้ด

ฉันเคยเห็นโปรแกรมเมอร์ต่อต้านการเปลี่ยนแปลงเช่นนี้มาก่อน ได้รับมากกว่านั้น! ฉันเปลี่ยนอาชีพของฉันหลายครั้ง ฉันใช้สไตล์ที่แตกต่างใน C # ของฉันมากกว่า PowerShell ของฉัน

ไม่กี่ปีที่ผ่านมาฉันทำงานกับทีม (นักพัฒนา ~ 20 คน) ที่ตัดสินใจขอข้อมูลจากนั้นทำการตัดสินใจจากนั้นจึงบังคับใช้สิ่งนั้นในทุกรหัสฐาน เรามีเวลา 1 สัปดาห์ในการตัดสินใจ

คร่ำครวญและน่าเกรงขามมากมาย "ฉันชอบวิธีของฉันเพราะมันดีกว่า" มากมาย แต่ไม่มีเนื้อหา

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

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

โปรดทราบว่าไม่ชัดเจนทันทีที่รายการพารามิเตอร์สิ้นสุดลงและเนื้อหาเริ่มต้นขึ้น เปรียบเทียบกับ:

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

เราได้อ่านเกี่ยวกับวิธีที่ผู้คนทั่วโลกจัดการกับปัญหานี้และพบรูปแบบของการเพิ่มบรรทัดว่างหลังจากวงเล็บปีกกาเปิด:

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

หากคุณกำลังจะหยุดพักสายตาคุณก็สามารถทำได้ด้วยการทำรั้ง จากนั้นทัศนวิสัยของคุณก็จะสม่ำเสมอเช่นกัน

แก้ไข : สองทางเลือกในโซลูชัน 'บรรทัดว่างพิเศษ' เมื่อใช้ K&R:

1 / เยื้องข้อโต้แย้งการทำงานแตกต่างจากร่างกายฟังก์ชั่น

2 / ใส่อาร์กิวเมนต์แรกในบรรทัดเดียวกันกับชื่อฟังก์ชั่นและจัดเรียงอาร์กิวเมนต์เพิ่มเติมในบรรทัดใหม่กับอาร์กิวเมนต์แรก

ตัวอย่าง:

1 /

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/ แก้ไข

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

กฎสำคัญคือ:

1. ปฏิบัติตามมาตรฐานการเข้ารหัสที่มีอยู่ของโครงการ
2. หากไม่มีมาตรฐานการเข้ารหัสและคุณกำลังแก้ไขฐานรหัสที่มีอยู่ของบุคคลอื่น - ให้สอดคล้องกับสไตล์ของรหัสที่มีอยู่ไม่ว่าคุณจะชอบ / ไม่ชอบก็ตาม
3. หากคุณกำลังทำงานในโครงการกรีนฟิลด์ - พูดคุยกับสมาชิกในทีมอื่นและมาที่ฉันทามติเกี่ยวกับมาตรฐานการเข้ารหัสที่เป็นทางการหรือไม่เป็นทางการ
4. หากคุณกำลังทำงานในโครงการกรีนฟิลด์ในฐานะผู้พัฒนาเพียงผู้เดียวจงตัดสินใจด้วยตนเองและจากนั้นก็จะสอดคล้องกันอย่างโหดเหี้ยม

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

ในที่สุดสไตล์ที่นำไปใช้ / นำไปใช้งานโดยใช้ตัวตรวจสอบลักษณะที่มีอยู่และตัวจัดรูปแบบโค้ดจะดีกว่ารูปแบบที่ต้อง "บังคับใช้" ด้วยตนเอง

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

ฉันไม่เห็นเหตุผลที่จะต้องเสียทั้งบรรทัดสำหรับวงเล็บเมื่อคำสั่ง for / while / if ก่อนหน้านี้แสดงให้เห็นถึงจุดเริ่มต้นของบล็อกแล้ว

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

ฉันชอบ

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

เกิน

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

เพราะบรรทัดyou.postAnswer();นั้นง่ายต่อการอ่านและค้นหาได้อย่างรวดเร็วก่อน ในวิธีที่สองมันได้รับการผสมกับบรรทัดด้านบน ( you.hasAnswer()) ทำให้ตาของฉันต้องโฟกัสมากขึ้นเพื่ออ่าน

ฉันชอบวิธีแรก เครื่องมือจัดฟันนั้นไม่คุ้มค่ากับการแยกกัน

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

มีภาษา (Python, Haskell, Ruby) ซึ่งใช้ได้โดยไม่ต้องใส่เครื่องหมายวงเล็บเลย สิ่งนี้เป็นการยืนยันว่าเครื่องหมายวงเล็บเป็นถังขยะเท่านั้นและไม่ควรได้รับเส้นตรงเมื่อเป็นไปได้:

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

ใช้ Python และก้าวเท้าเลี่ยงการโต้เถียงอย่างสมบูรณ์

ตำแหน่งของเครื่องมือจัดฟันแบบหยิกควรเป็น

ข้อมูลเมตา

สามารถกำหนดค่าได้ใน IDE โดยโปรแกรมเมอร์ ด้วยวิธีนี้วงเล็บปีกกาที่น่ารำคาญในรหัสทั้งหมดโดยไม่คำนึงถึงผู้แต่งดูเหมือนกัน

มันขึ้นอยู่กับ.

ถ้าฉันกำลังเข้ารหัสใน Javascript หรือ jQuery ฉันจะใช้แบบฟอร์มแรก:

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

แต่ถ้าฉันเข้ารหัสใน C # ฉันจะใช้แบบฟอร์มที่สองเพราะนั่นเป็นวิธีที่บัญญัติในการทำใน C #

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

โปรดทราบว่าตัวอย่างของคุณสามารถเขียนได้

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

ใน C #

ฉันชอบสิ่งแรกเพราะมันยากสำหรับฉันที่จะเห็นความผิดพลาดในตัวอย่างนี้

if (value > maximum);
{
    dosomething();
}

กว่าที่เป็นในตัวอย่างนี้

if (value > maximum); {
    dosomething();
}

การที่ฉัน; {ดูผิดไปกว่าการเขียนบรรทัดที่ลงท้ายด้วย;ดังนั้นฉันจึงมีแนวโน้มที่จะสังเกตเห็น

ฉันชอบความแตกต่างเล็กน้อยจาก 1)

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

ทำไม?

• ผมคิดเสมอ วางวงเล็บในบรรทัดของตัวเองลดลงการอ่าน ฉันสามารถใส่ซอร์สโค้ดจำนวนหนึ่งลงบนหน้าจอของฉันได้เท่านั้น ลักษณะ Bracket 2) ทำให้อัลกอริธึมมีลูปซ้อนกันและมีเงื่อนไขที่ยาวมาก

• อย่างไรก็ตามฉันต้องการelseเริ่มต้นในบรรทัดใหม่เพราะifและelseอยู่ด้วยกันด้วยสายตา หากมีวงเล็บอยู่ด้านหน้าelseมันจะยากมากที่จะตรวจสอบสิ่งที่เป็นของ

• 3) ตัดสิทธิ์ตัวเอง เราทุกคนรู้ว่าสิ่งเลวร้ายอาจเกิดขึ้นได้หากคุณละทิ้งวงเล็บเหลี่ยมและลืมมันไป

ฉันได้อ่านที่ไหนสักแห่งที่ผู้แต่งหนังสือบางเล่มต้องการโค้ดของพวกเขาในรูปแบบนี้:

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

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

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

ตอนนี้ฉันไม่รู้ว่ามันเป็นเรื่องจริง (เพราะฉันหามันไม่ได้อีกแล้ว) แต่สไตล์หลังนี้แพร่หลายมากในหนังสือ

ในระดับส่วนบุคคลฉันชอบวงเล็บในบรรทัดแยกเป็น:

ก) พวกเขาระบุขอบเขตใหม่
ข) ง่ายต่อการตรวจสอบเมื่อคุณมีความไม่ตรงกัน (แม้ว่าปัญหานี้จะน้อยกว่าใน IDE ที่เน้นข้อผิดพลาดให้คุณ)

ah, หนึ่งทรูสไตล์รั้ง

มันมีทุกสิ่งที่จำเป็นสำหรับทางศักดิ์สิทธิ์ - แม้กระทั่งผู้เผยพระวจนะ (ริชาร์ด "ทางของฉันหรือทางหลวง" สตอลแมน)

ชายคนนี้ผิดเกี่ยวกับหลายสิ่งหลายอย่างมาก แต่ GNU เป็นจุดเมื่อพูดถึงการจัดฟัน

[อัพเดท] ฉันได้เห็นแสงสว่างและตอนนี้ก็เคารพบูชาAllman

ตัวอย่างที่สองฉันใหญ่มากในการอ่าน ฉันทนไม่ไหวที่จะดูว่าบล็อกอื่น ๆ =

คำตอบง่ายๆ: การดีบักง่ายกว่าอะไร?

// Case 1:
void dummyFunction() {
  for (i = 0; i != 10; ++i) {
    if (i <= 10)
      std::cout << "i is: " << i << "\n";
      std::cout << 10 - i << " steps remaining\n";

      // Some hard work here
      // which is really hard
      // and does take some screen estate
    }
    else
      std::cout << "We'll never get there";
  }
} // COMPILER ERROR HERE


// Case 2:
void dummyFunction()
{
  for (i = 0; i != 10; ++i)

    if (i <= 10)
    {
      std::cout << "i is: " << i << "\n";
      std::cout << 10 - i << " steps remaining\n";

      // Some hard work here
      // which is really hard
      // and does take some screen estate
    }
    else
      std::cout << "We'll never get there\n";
  }
} // COMPILER ERROR HERE

คุณวินิจฉัยปัญหานี้ในกรณีใดก่อน

ฉันไม่สนใจความชอบส่วนบุคคลมากนัก (มีหลายสไตล์รวมถึง whitesmith และ al.) และฉันไม่สนใจมาก ... ตราบใดที่มันไม่ได้ขัดขวางความสามารถในการอ่านรหัสและดีบักมัน

ตามอาร์กิวเมนต์ "เปลืองพื้นที่" ฉันไม่ซื้อ: ฉันมักจะเพิ่มบรรทัดว่างระหว่างกลุ่มตรรกะต่อไปเพื่อให้โปรแกรมชัดเจนขึ้น ...

ไม่ใช่ว่าทุกคนจะสังเกตเห็น แต่นี่คือเหตุผลว่าทำไมเครื่องมือจัดฟันอยู่ในบรรทัดเดียวกับเงื่อนไข (ยกเว้นเงื่อนไขที่ยาวมาก แต่เป็นกรณีที่มีขอบ):

ใน C นี่คือโครงสร้างที่ถูกต้อง:

ในขณะที่ (จริง);
{
    ถ่านค;
    getchar (); // รอการป้อนข้อมูล
}

ด่วน! รหัสนี้ทำอะไร? หากคุณตอบว่า "วงวนไม่สิ้นสุดขอให้ป้อนข้อมูล" คุณผิด! มันไม่ได้ไปที่อินพุต while(true)จะได้รับการติดที่ สังเกตุว่าอัฒภาคท้าย รูปแบบนี้เป็นเรื่องธรรมดามากที่ดูเหมือนว่าควรจะเป็น; C กำหนดให้คุณต้องประกาศตัวแปรของคุณที่จุดเริ่มต้นของบล็อกซึ่งเป็นสาเหตุที่เริ่มใหม่

บรรทัดของรหัสคือความคิด เครื่องมือจัดฟันเป็นส่วนหนึ่งของความคิดที่มีเงื่อนไขหรือห่วง ดังนั้นจึงอยู่ในบรรทัดเดียวกัน

ฉันชอบวิธีแรก ดูเหมือนว่าจะเป็น IMO ที่สั้นกว่าและกะทัดรัดกว่าที่ฉันชอบ

แก้ไข: อ่าหนึ่งในสาม ฉันชอบที่ดีที่สุดเมื่อเป็นไปได้เพราะมันมีขนาดเล็กลง / neater

คุณสามารถเขียนมัน:

you.hasAnswer() ? you.postAnswer() : you.doSomething();

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

คำตอบเกือบทั้งหมดในที่นี้พูดถึงการเปลี่ยนแปลงบางอย่างเกี่ยวกับ "ไม่ว่าคุณจะทำอะไร

ดังนั้นฉันจึงคิดอยู่ครู่หนึ่งและต้องยอมรับว่าฉันไม่เห็นว่ามันสำคัญ ใครสามารถบอกฉันได้อย่างสุจริตว่าสิ่งต่อไปนี้ยากที่จะติดตาม

int foo(int a, Bar b) {
    int c = 0;
    while(a != c)
    {
        if(b.value[a] == c) {
            c = CONST_A;
        }
        c++;
    }
    return c;
}

ฉันไม่แน่ใจเกี่ยวกับคนอื่น ... แต่ฉันมีปัญหาอย่างไม่มีศูนย์ทางจิตใจสลับไปมาระหว่างสไตล์ ฉันใช้เวลาสักครู่เพื่อค้นหาว่าโค้ดทำอะไร แต่นั่นเป็นผลมาจากฉันที่เพิ่งพิมพ์ไวยากรณ์ C-like แบบสุ่ม :)

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

FWIW สไตล์การเขียนโค้ดของเราในที่ทำงานใช้รูปแบบที่มีโครงสร้างมากขึ้นเล็กน้อย 1 และแบบฟอร์มที่แก้ไข 3 (C ++)

            // blank line is required here
if (x) {
            //This blank line is required
   y = z;
}
            // blank line is required here too, unless this line is only another '}'

if (x) y = z; //allowed

if (x)
    y = z;  // forbidden

ฉันอยากรู้ว่าผู้ที่ชอบแบบฟอร์ม 2 มากจะพบว่าแบบฟอร์ม 1 นี้ดีกว่าหรือเปล่าเพียงเพราะบรรทัดว่างให้ภาพแยกที่ชัดเจนขึ้น

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

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

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

ฉันชอบวิธีที่สองเป็นการส่วนตัว

อย่างไรก็ตามวิธีที่ฉันจะแสดงให้เห็นก็คือในความคิดของฉันที่ดีที่สุดเพราะมันส่งผลให้ความปลอดภัยในการทำงานที่ยิ่งใหญ่ที่สุด! เพื่อนนักเรียนจากมหาวิทยาลัยของฉันขอให้ฉันช่วยบ้านของเขาและนี่คือลักษณะรหัสของเขา โปรแกรมทั้งหมดดูเหมือนบล็อกเดียว สิ่งที่น่าสนใจคือ 95% ของข้อบกพร่องในโปรแกรมที่เขาทำนั้นมาจากเครื่องมือจัดฟันที่ไม่ตรงกัน อีก 5% เห็นได้ชัดเมื่อจัดฟันเข้าคู่กัน

while(1){
i=0;
printf("Enter coded text:\n");
while((s=getchar())!='\n'){
         if(i%1==0){
            start=(char*)realloc(input,(i+1)*sizeof(char));
if(start==NULL){
printf("Memory allocation failed!");
exit(1);}
input=start;}
      input[i++]=s;}
start=(char*)realloc(input,(i+1)*sizeof(char));
if(start==NULL){
printf("Memory allocation failed!!!");
exit(1);}
input=start;
input[i]='\0';
                puts(input);

ความชอบส่วนตัวของฉันสำหรับวิธีแรกอาจเป็นเพราะนั่นคือวิธีที่ฉันเรียนรู้ PHP ครั้งแรก

สำหรับifคำสั่งบรรทัดเดียวฉันจะใช้

if (you.hasAnswer()) you.postAnswer();

หากไม่ใช่you.postAnswer();แต่มีบางสิ่งที่ยาวกว่ามากเช่นyou.postAnswer(this.AnswerId, this.AnswerText, this.AnswerType);ฉันอาจเปลี่ยนกลับไปใช้ประเภทแรก:

if (you.hasAnswer) {
    you.postAnswer(this.AnswerId, this.AnswerText, this.AnswerType);
}

ฉันจะไม่ใช้ตัวแบ่งบรรทัดและฉันจะไม่ใช้วิธีนี้หากมีelseคำสั่งด้วย

if (you.hasAnswer()) you.postAnswer();
else you.doSomething()

เป็นไปได้ทางทฤษฎี แต่ไม่ใช่สิ่งที่ฉันเคยใช้ สิ่งนี้จะต้องกลายเป็น

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

พวกเขาไม่ควร; วิธีแรกสำหรับฉัน

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

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

บวกตามที่ระบุไว้แล้วจะช่วยให้พอดีกับรหัสเพิ่มเติมในหน้าจอซึ่งมิฉะนั้นจะต่อต้านเล็กน้อย

ขึ้นอยู่กับแพลตฟอร์ม / ภาษา / การประชุม

ใน Java:

void someMethod() { 
     if (you.hasAnswer()) {
         you.postAnswer();
     } else {
       you.doSomething();
     }
}

ใน C #

void someMethod() 
{ 
     if (you.hasAnswer()) 
     {
         you.postAnswer();
     } 
     else 
     {
       you.doSomething();
     }
}

ใน C:

void someMethod() 
{ 
     if (you_hasAnswer()) {
         you.postAnswer();
     } else {
       you_doSomething();
     }
}

ฉันเกลียดเมื่อ Java ใช้สไตล์ของพวกเขาในรหัส C # และในทางกลับกัน

ทั้งหมดที่ฉันสามารถพูดได้คือถ้าคุณเป็นแฟนของวิธี # 3 คุณจะถูกข่มเหงโดยทุกฟอร์แมตโค้ด IDE บนโลก

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

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

แต่; สิ่งที่สำคัญที่สุดคือความมั่นคง ใช้อย่างใดอย่างหนึ่ง - ไม่เคยทั้งคู่!

เมื่อฉันเรียนรู้การเขียนโปรแกรมครั้งแรกเมื่ออายุ 12 ปีฉันใส่วงเล็บปีกกาในบรรทัดถัดไปเนื่องจากบทช่วยสอนการเข้ารหัสของ Microsoft เป็นเช่นนั้น ฉันก็เยื้องกับแท็บ 4 พื้นที่ในเวลานั้น

หลังจากไม่กี่ปีที่ฉันเรียนรู้ Java และ JavaScript และเห็นรหัสวงเล็บปีกกา -on-line เดียวกันดังนั้นฉันจึงเปลี่ยน ฉันก็เริ่มที่จะเยื้องกับช่องว่าง 2 พื้นที่

มีตัวเลือกที่ 4 ที่ทำให้การจัดฟันอยู่ในแนวเดียวกัน แต่ไม่เปลืองเนื้อที่:

if (you.hasAnswer())
{    you.postAnswer();
     i.readAnswer();
}
else
{   you.doSomething();
}

ปัญหาเดียวก็คือว่าตัวจัดรูปแบบอัตโนมัติของ IDE ส่วนใหญ่ทำให้หายใจไม่ออก

ทุกอย่างขึ้นอยู่กับคุณตราบใดที่คุณไม่ได้ทำงานในโปรเจ็กต์ที่มีข้อ จำกัด ในการเข้ารหัสหรือมาตรฐานบางอย่างที่ผู้จัดการโครงการกำหนดไว้

ฉันเองชอบวิธีที่ 1

นอกจากนี้ฉันไม่ได้รับสิ่งที่คุณต้องการแสดงด้วยวิธีที่ 3?

นั่นเป็นวิธีที่ผิดหรือเปล่า? ตัวอย่างเช่นพิจารณาสถานการณ์เช่น ..

if (you.hasAnswer())
  you.postAnswer();
else
  you.doSomething();

ตอนนี้จะเกิดอะไรขึ้นถ้ามีคนต้องการเพิ่มคำสั่งเพิ่มเติมในบล็อกif ?

ในกรณีนั้นถ้าคุณใช้วิธีที่ 3 คอมไพเลอร์จะโยนข้อผิดพลาดทางไวยากรณ์

if (you.hasAnswer())
   you.postAnswer1();
   you.postAnswer2();
else
   you.doSomething();