ฉันจะโทรด้วยบูลีนที่ชัดเจนขึ้นได้อย่างไร? กับดักบูลีน

ตามที่ระบุไว้โดยในความคิดเห็นโดย @ benjamin-gruenbaum สิ่งนี้เรียกว่ากับดักบูลีน:

สมมติว่าฉันมีฟังก์ชั่นเช่นนี้

UpdateRow(var item, bool externalCall);

และในคอนโทรลเลอร์ของฉันค่าexternalCallนั้นจะเป็นจริงเสมอ วิธีที่ดีที่สุดในการเรียกใช้ฟังก์ชันนี้คืออะไร ฉันมักจะเขียน

UpdateRow(item, true);

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

bool externalCall = true;
UpdateRow(item, externalCall);

PD: ไม่แน่ใจว่าคำถามนี้ตรงกับที่นี่จริงหรือไม่ถ้าไม่ฉันจะขอข้อมูลเพิ่มเติมเกี่ยวกับเรื่องนี้ได้จากที่ไหน

PD2: ฉันไม่ได้ติดแท็กภาษาใด ๆ 'เพราะฉันคิดว่ามันเป็นปัญหาทั่วไปมาก อย่างไรก็ตามฉันทำงานกับ c # และคำตอบที่ยอมรับนั้นใช้ได้กับ c #

ไม่มีวิธีแก้ปัญหาที่สมบูรณ์แบบเสมอไป แต่คุณมีทางเลือกมากมายให้เลือก:

• ใช้อาร์กิวเมนต์ที่มีชื่อหากมีในภาษาของคุณ มันใช้งานได้ดีมากและไม่มีข้อเสีย ในบางภาษาอาร์กิวเมนต์ใด ๆ ที่สามารถส่งผ่านเป็นอาร์กิวเมนต์ที่กำหนดชื่อเช่นupdateRow(item, externalCall: true)( C # ) หรือupdate_row(item, external_call=True)(Python)

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

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

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

• ใช้enum ปัญหาเกี่ยวกับบูลีนคือพวกเขาถูกเรียกว่า "จริง" และ "เท็จ" ดังนั้นแทนที่จะแนะนำประเภทที่มีชื่อที่ดีกว่า (เช่นenum CallType { INTERNAL, EXTERNAL }) ในฐานะที่เป็นประโยชน์เพิ่มเติมนี้จะเพิ่มความปลอดภัยประเภทของโปรแกรมของคุณ (หากภาษาของคุณใช้ enums เป็นประเภทที่แตกต่างกัน) ข้อเสียเปรียบของ enums ก็คือพวกเขาจะเพิ่มประเภทให้กับ API ที่เปิดเผยต่อสาธารณะของคุณ สำหรับฟังก์ชั่นภายในอย่างแท้จริงนี้ไม่สำคัญและ enums ไม่มีข้อเสียอย่างมีนัยสำคัญ

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

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

วิธีแก้ไขที่ถูกต้องคือทำสิ่งที่คุณแนะนำ แต่บรรจุลงในซุ้มขนาดเล็ก:

void updateRowExternally() {
  bool externalCall = true;
  UpdateRow(item, externalCall);
}

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

บอกว่าฉันมีฟังก์ชั่นเช่น UpdateRow (รายการ var, บูล externalCall);

ทำไมคุณมีฟังก์ชั่นเช่นนี้

ภายใต้สถานการณ์ใดที่คุณจะเรียกมันด้วยอาร์กิวเมนต์ externalCall ที่ตั้งค่าเป็นค่าที่ต่างกัน

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

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

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

UpdateRow(item, true /* row is an external call */);

หรือ:

UpdateRow(item, true); // true means call is external

หรือ (ถูกต้องตามที่แนะนำโดย Frax):

UpdateRow(item, /* externalCall */true);

1. คุณสามารถ 'ตั้งชื่อ' คนขายเครื่องของคุณ ด้านล่างเป็นตัวอย่างสำหรับภาษา OO (ซึ่งสามารถแสดงในชั้นเรียนได้UpdateRow()) อย่างไรก็ตามแนวคิดนี้สามารถนำไปใช้ในภาษาใดก็ได้:

   class Table
   {
   public:
       static const bool WithExternalCall = true;
       static const bool WithoutExternalCall = false;
   

   และในเว็บไซต์โทร:

   UpdateRow(item, Table::WithExternalCall);
   

2. ฉันเชื่อว่ารายการ # 1 ดีกว่า แต่ก็ไม่ได้บังคับให้ผู้ใช้ใช้ตัวแปรใหม่เมื่อใช้ฟังก์ชัน หากความปลอดภัยของประเภทมีความสำคัญสำหรับคุณคุณสามารถสร้างenumประเภทและUpdateRow()ยอมรับสิ่งนี้แทนbool:

   UpdateRow(var item, ExternalCallAvailability ec);

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

   UpdateRowWithExternalCall(var item, bool externalCall)

ตัวเลือกอื่นที่ฉันยังไม่ได้อ่านที่นี่คือ: ใช้ IDE ที่ทันสมัย

ตัวอย่างเช่นความคิด IntelliJ พิมพ์ชื่อตัวแปรของตัวแปรในวิธีการที่คุณโทรถ้าคุณผ่านตัวอักษรเช่นtrueหรือหรือnull username + “@company.comสิ่งนี้ทำในแบบอักษรขนาดเล็กดังนั้นจึงไม่ใช้พื้นที่มากเกินไปบนหน้าจอของคุณและดูแตกต่างจากรหัสจริงมาก

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

ตัวอย่างดัดแปลงจากส่วนหนึ่งของการตั้งค่าการทดสอบของฉัน:

2 วันและไม่มีใครพูดถึงความแตกต่าง?

target.UpdateRow(item);

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

ทำอย่างนั้นและมันจะกลายเป็นส่วนหนึ่งของปัญหาการก่อสร้าง ที่สามารถแก้ไขได้หลายวิธี นี่คือหนึ่ง:

Target xyzTargetFactory(TargetBuilder targetBuilder) {
    return targetBuilder
        .connectionString("some connection string")
        .table("table_name")
        .external()
        .build()
    ; 
}

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

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

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

function updateRow(var item, bool externalCall) {
  Database.update(item);

  if (externalCall) {
    Service.call();
  }
}

นั่นอาจเป็นกลิ่นรหัส ฟังก์ชั่นอาจมีพฤติกรรมที่แตกต่างกันอย่างมากขึ้นอยู่กับexternalCallตัวแปรที่ตั้งค่าไว้ซึ่งในกรณีนี้มันมีความรับผิดชอบที่แตกต่างกันสองอย่าง การปรับโครงสร้างใหม่เป็นสองฟังก์ชันที่มีความรับผิดชอบเพียงอย่างเดียวสามารถปรับปรุงความสามารถในการอ่านได้:

function updateRowAndService(var item) {
  updateRow(item);
  Service.call();
}

function updateRow(var item) {
  Database.update(item);
}

ตอนนี้ทุกที่ที่คุณเรียกใช้ฟังก์ชั่นเหล่านี้คุณสามารถบอกได้ทันทีว่ามีการให้บริการภายนอกหรือไม่

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

หาก UpdateRow อยู่ใน codebase ที่ควบคุมฉันจะพิจารณารูปแบบกลยุทธ์:

public delegate void Request(string query);    
public void UpdateRow(Item item, Request request);

โดยที่ Request หมายถึง DAO บางประเภท (การโทรกลับในกรณีเล็กน้อย)

กรณีจริง:

UpdateRow(item, query =>  queryDatabase(query) ); // perform remote call

กรณีเท็จ:

UpdateRow(item, query => readLocalCache(query) ); // skip remote call

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

UpdateRow(item, query => {
  var data = readLocalCache(query);
  if (data == null) {
    data = queryDatabase(query);
  }
  return data;
} );

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