Làm thế nào tôi có thể thực hiện một cuộc gọi với một boolean rõ ràng hơn? Boolean Bẫy

Như đã lưu ý trong các bình luận của @ benjamin-gruenbaum, đây được gọi là bẫy Boolean:

Nói rằng tôi có một chức năng như thế này

UpdateRow(var item, bool externalCall);

và trong bộ điều khiển của tôi, giá trị đó externalCallsẽ luôn luôn ĐÚNG. Cách tốt nhất để gọi chức năng này là gì? Tôi thường viết

UpdateRow(item, true);

Nhưng tôi tự hỏi mình, tôi có nên khai báo boolean không, chỉ để cho biết giá trị 'thật' đó là gì? Bạn có thể biết rằng bằng cách xem khai báo của hàm, nhưng rõ ràng nó nhanh hơn và rõ hơn nếu bạn vừa thấy một cái gì đó như

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

PD: Không chắc câu hỏi này có thực sự phù hợp ở đây không, nếu không, tôi có thể lấy thêm thông tin về vấn đề này ở đâu?

PD2: Tôi không gắn thẻ bất kỳ ngôn ngữ nào vì tôi nghĩ đó là một vấn đề rất chung chung. Dù sao, tôi làm việc với c # và câu trả lời được chấp nhận hoạt động cho c #

Luôn luôn không có một giải pháp hoàn hảo, nhưng bạn có nhiều lựa chọn thay thế:

• Sử dụng các đối số được đặt tên , nếu có sẵn trong ngôn ngữ của bạn. Điều này hoạt động rất tốt và không có nhược điểm cụ thể. Trong một số ngôn ngữ, bất kỳ đối số nào cũng có thể được chuyển qua dưới dạng đối số được đặt tên, ví dụ updateRow(item, externalCall: true)( C # ) hoặc update_row(item, external_call=True)(Python).

  Đề xuất của bạn sử dụng một biến riêng biệt là một cách để mô phỏng các đối số được đặt tên, nhưng không có lợi ích an toàn liên quan (không có gì đảm bảo rằng bạn đã sử dụng tên biến chính xác cho đối số đó).

• Sử dụng các chức năng riêng biệt cho giao diện công cộng của bạn, với tên tốt hơn. Đây là một cách khác để mô phỏng các tham số được đặt tên, bằng cách đặt các giá trị paremeter trong tên.

  Điều này rất dễ đọc, nhưng dẫn đến rất nhiều bản tóm tắt cho bạn, người đang viết các chức năng này. Nó cũng không thể đối phó tốt với vụ nổ tổ hợp khi có nhiều đối số boolean. Một nhược điểm đáng kể là khách hàng không thể thiết lập giá trị này một cách linh hoạt mà phải sử dụng if / other để gọi hàm chính xác.

• Sử dụng một enum . Vấn đề với booleans là chúng được gọi là "đúng" và "sai". Vì vậy, thay vì giới thiệu một loại có tên tốt hơn (ví dụ enum CallType { INTERNAL, EXTERNAL }). Là một lợi ích bổ sung, điều này làm tăng sự an toàn của loại chương trình của bạn (nếu ngôn ngữ của bạn thực hiện enum như các loại khác biệt). Hạn chế của enums là họ thêm một loại vào API hiển thị công khai của bạn. Đối với các chức năng hoàn toàn bên trong, điều này không quan trọng và enum không có nhược điểm đáng kể.

  Trong các ngôn ngữ không có enums, chuỗi ngắn đôi khi được sử dụng thay thế. Điều này hoạt động, và thậm chí có thể tốt hơn so với booleans thô, nhưng rất dễ bị lỗi chính tả. Sau đó, hàm sẽ ngay lập tức xác nhận rằng đối số khớp với một tập hợp các giá trị có thể.

Không có giải pháp nào trong số này có tác động cấm hiệu quả. Các tham số và enum được đặt tên có thể được giải quyết hoàn toàn tại thời gian biên dịch (đối với ngôn ngữ được biên dịch). Sử dụng các chuỗi có thể liên quan đến so sánh chuỗi, nhưng chi phí đó không đáng kể đối với các chuỗi ký tự nhỏ và hầu hết các loại ứng dụng.

Giải pháp chính xác là làm những gì bạn đề xuất, nhưng gói nó thành một mặt tiền nhỏ:

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

Khả năng đọc vượt qua tối ưu hóa vi mô. Bạn có thể đủ khả năng gọi thêm chức năng, chắc chắn tốt hơn bạn có thể đủ khả năng cho nhà phát triển nỗ lực tìm kiếm ngữ nghĩa của cờ Boolean dù chỉ một lần.

Nói rằng tôi có một chức năng như UpdateRow (var item, bool bên ngoài);

Tại sao bạn có một chức năng như thế này?

Trong trường hợp nào bạn sẽ gọi nó với đối số bên ngoài được đặt thành các giá trị khác nhau?

Nếu một là từ, giả sử, một ứng dụng khách bên ngoài và ứng dụng khác nằm trong cùng một chương trình (nghĩa là các mô-đun mã khác nhau), thì tôi cho rằng bạn nên có hai phương thức khác nhau , một phương thức cho mỗi trường hợp, thậm chí có thể được xác định trên riêng biệt Giao diện.

Tuy nhiên, nếu bạn đang thực hiện cuộc gọi dựa trên một số mốc được lấy từ một nguồn không phải là chương trình (giả sử, tệp cấu hình hoặc cơ sở dữ liệu đã đọc), thì phương thức chuyển Boolean sẽ có ý nghĩa hơn.

Mặc dù tôi đồng ý rằng lý tưởng là sử dụng một tính năng ngôn ngữ để thực thi cả khả năng đọc và an toàn giá trị, bạn cũng có thể chọn cách tiếp cận thực tế : nhận xét trong thời gian gọi. Như:

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

hoặc là:

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

hoặc (đúng, theo đề xuất của Frax):

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

1. Bạn có thể 'đặt tên' bools của bạn. Dưới đây là một ví dụ cho ngôn ngữ OO (nơi nó có thể được thể hiện trong lớp cung cấp UpdateRow()), tuy nhiên bản thân khái niệm này có thể được áp dụng trong bất kỳ ngôn ngữ nào:

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

   và trong trang web cuộc gọi:

   UpdateRow(item, Table::WithExternalCall);
   

2. Tôi tin rằng Mục số 1 tốt hơn nhưng nó không bắt buộc người dùng sử dụng các biến mới khi sử dụng hàm. Nếu loại an toàn là quan trọng đối với bạn, bạn có thể tạo một enumloại và UpdateRow()chấp nhận điều này thay vì bool:

   UpdateRow(var item, ExternalCallAvailability ec);

3. Bạn có thể sửa đổi tên của hàm bằng cách nào đó nó phản ánh ý nghĩa của booltham số tốt hơn. Không chắc lắm nhưng có lẽ:

   UpdateRowWithExternalCall(var item, bool externalCall)

Một tùy chọn khác tôi chưa đọc ở đây là: Sử dụng IDE hiện đại.

Ví dụ: IntelliJ IDEA in tên biến của các biến trong phương thức bạn đang gọi nếu bạn chuyển một chữ như true, nullhoặc username + “@company.com. Điều này được thực hiện trong một phông chữ nhỏ để nó không chiếm quá nhiều không gian trên màn hình của bạn và trông rất khác so với mã thực tế.

Tôi vẫn không nói rằng đó là một ý tưởng tốt để ném vào booleans ở mọi nơi. Đối số nói rằng bạn đọc mã thường xuyên hơn bạn viết thường rất mạnh, nhưng trong trường hợp cụ thể này, nó phụ thuộc rất nhiều vào công nghệ mà bạn (và đồng nghiệp của bạn!) Sử dụng để hỗ trợ công việc của bạn. Với một IDE, vấn đề ít hơn nhiều so với vim chẳng hạn.

Ví dụ sửa đổi một phần của thiết lập thử nghiệm của tôi:

2 ngày và không có ai đề cập đến đa hình?

target.UpdateRow(item);

Khi tôi là khách hàng muốn cập nhật một hàng với một mục tôi không muốn nghĩ về tên của cơ sở dữ liệu, URL của dịch vụ vi mô, giao thức được sử dụng để liên lạc hoặc có hay không một cuộc gọi bên ngoài sẽ cần phải được sử dụng để làm cho nó xảy ra. Ngừng đẩy những chi tiết này vào tôi. Chỉ cần lấy mục của tôi và đi cập nhật một hàng ở đâu đó.

Làm điều đó và nó trở thành một phần của vấn đề xây dựng. Điều đó có thể được giải quyết bằng nhiều cách. Đây là một:

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

Nếu bạn đang nhìn vào đó và suy nghĩ "Nhưng ngôn ngữ của tôi có tên là đối số, tôi không cần điều này". Tốt, tuyệt, đi sử dụng chúng. Chỉ cần tránh những điều vô nghĩa này ra khỏi ứng dụng khách đang gọi mà thậm chí không biết nó đang nói gì.

Cần lưu ý rằng đây là nhiều hơn một vấn đề ngữ nghĩa. Đó cũng là một vấn đề thiết kế. Vượt qua trong một boolean và bạn buộc phải sử dụng phân nhánh để đối phó với nó. Không hướng đối tượng lắm. Có hai hoặc nhiều booleans để vượt qua? Muốn ngôn ngữ của bạn có nhiều công văn? Nhìn vào những gì bộ sưu tập có thể làm khi bạn lồng chúng. Cấu trúc dữ liệu phù hợp có thể làm cho cuộc sống của bạn dễ dàng hơn rất nhiều.

Nếu bạn có thể sửa đổi updateRowhàm, có lẽ bạn có thể cấu trúc lại thành hai hàm. Cho rằng hàm lấy tham số boolean, tôi nghi ngờ nó trông như thế này:

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

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

Đó có thể là một chút mùi mã. Hàm có thể có hành vi khác nhau đáng kể tùy thuộc vào externalCallbiến được đặt thành gì, trong trường hợp đó, nó có hai trách nhiệm khác nhau. Tái cấu trúc nó thành hai chức năng chỉ có một trách nhiệm có thể cải thiện khả năng đọc:

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

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

Bây giờ, bất cứ nơi nào bạn gọi các chức năng này, bạn có thể biết trong nháy mắt liệu dịch vụ bên ngoài có được gọi hay không.

Tất nhiên, đây không phải là trường hợp. Đó là tình huống và một vấn đề của hương vị. Tái cấu trúc một hàm lấy tham số boolean thành hai hàm thường đáng để xem xét, nhưng không phải lúc nào cũng là lựa chọn tốt nhất.

Nếu UpdateRow nằm trong một cơ sở mã được kiểm soát, tôi sẽ xem xét mô hình chiến lược:

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

Trong đó Yêu cầu đại diện cho một số loại DAO (một cuộc gọi lại trong trường hợp tầm thường).

Trường hợp thật:

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

Trường hợp sai:

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

Thông thường, việc thực hiện điểm cuối sẽ được cấu hình ở mức độ trừu tượng cao hơn và tôi sẽ chỉ sử dụng nó ở đây. Là một hiệu ứng phụ miễn phí hữu ích, điều này cho tôi một tùy chọn để thực hiện một cách khác để truy cập dữ liệu từ xa mà không có bất kỳ thay đổi nào đối với mã:

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

Nói chung, việc đảo ngược điều khiển như vậy đảm bảo khớp nối thấp hơn giữa lưu trữ dữ liệu và mô hình của bạn.