Đây có phải là cách sử dụng tốt của ExtensionMethod không?

Question

Tôi chỉ viết một câu lệnh if trong dòng

if (value == value1 || value == value2 || value == value3 || value == value4) 
    //do something 

và nó làm phiền tôi mà tôi luôn phải lặp lại 'giá trị ==' phần. Theo tôi, điều này là không phục vụ mục đích nào khác ngoài việc làm cho nó khó đọc.

tôi viết ExtensionMethod sau đó nên làm cho kịch bản trên dễ đọc hơn:

public static bool IsEqualToAny<T>(this T value, params T[] objects) 
{ 
    return objects.Contains(value); 
} 

Bây giờ tôi chỉ có thể viết

if (value.IsEqualToAny(value1, value2, value3, value4)) 
    //do something 

Đây có phải là một cách sử dụng tốt của một ExtensionMethod?

EDIT:

Cảm ơn tất cả các câu trả lời tuyệt vời. Đối với hồ sơ: Tôi đã giữ phương pháp. Trong khi gợi ý rằng bạn chỉ có thể sử dụng new []{value1,value2,value3,value4}.Contains(value) là sự thật, tôi chỉ đơn giản là thích đọc loại này của câu lệnh if từ trái sang phải (nếu giá trị này tương đương với bất kỳ của các thay vì nếu những giá trị này chứa giá trị này). Có thêm một phương pháp hiển thị trong intellisense trên mỗi đối tượng không phải là một vấn đề đối với tôi.

Answer 1

Bạn chưa thêm chức năng chỉ hữu ích cho ứng dụng hoặc ngữ cảnh cụ thể, tiện ích mở rộng của bạn được đặt tên rõ ràng và hành vi hiển nhiên mà không cần phải xem xét triển khai.

Câu trả lời là "Có, nó là"

Answer 2

Có vẻ tốt với tôi, mặc dù có vẻ hơi độc đáo một chút.

Answer 3

Có vẻ khá công bằng, nhưng tôi sẽ lùi lại một bước. Bạn có thể đặt bất kỳ ý nghĩa kinh doanh nào trong so sánh không? Các giá trị đó là gì? Có lẽ bạn sẽ tốt hơn với một phương pháp gọi là IsSpecialCustomerLocation hoặc một cái gì đó thể hiện ý định thực sự của mã.

Answer 4

Bạn cũng có thể sử dụng phương pháp cú pháp LINQ cho nhiệm vụ đó (bằng cách sử dụng System.Linq namespace):

  object[] objects = new object[10]; 
     objects.Contains(new MyClass()); 

Hmm cho tôi nghĩ rằng một khoảnh khắc ... Oh bạn đã sử dụng nó. Nhưng bạn đã đặt nó vào một phương pháp riêng biệt thay vì gọi nó trực tiếp.

Answer 5

Nó là bất thường để viết một phương pháp mở rộng cho một hạn chế T. Không kém phần quan trọng, cách tiếp cận này sẽ nhanh chóng làm cho nội tâm của bạn trở nên khá khó sử dụng.

Trong khi hợp lệ, tôi có thể tránh phương thức này làm phương thức mở rộng - có lẽ chỉ cần sử dụng phương pháp tiện ích tĩnh chuẩn.

Cú pháp khởi tạo mảng C# 3 có thể dễ dàng hơn?

bool isTrue = new[] { 1, 2, 3 }.Contains(3); 

Tất nhiên, đối với bộ dữ liệu lớn, bạn có thể muốn một bộ nhớ cache HashSet<T> đâu đó ;-p

Answer 6

Tôi sẽ làm cho một lớp tĩnh cho mục đích đó.Tôi không thích giải pháp đó bởi vì nó thêm một phương thức cho tất cả các lớp có vẻ hơi quá mức. Tuy nhiên nó theo một cách đi với OOD bởi vì bạn yêu cầu các đối tượng để thực hiện các chức năng trên chính mình (kinda).

Tuy nhiên, tôi sẽ đi với một lớp có thể tái sử dụng thay vì bởi vì tôi có thể thấy cách thức một antipattern có thể hình thành. Tôi không gọi nó là một antipattern được nêu ra, nhưng nếu quá nhiều của những xây dựng bật lên tôi sẽ gọi đó là một antipattern của dễ đọc kể từ khi mọi đối tượng sẽ nhận được lộn xộn với các phương pháp mở rộng. Tôi kinda nhìn nó như ô nhiễm không gian tên, nhưng ô nhiễm thành viên lớp học.

if (ConditionHelper.IsEqualToAny(value, value1, value2, value3)) 
{ 
    // Do something 
} 

Có cùng một công việc và không gây ô nhiễm bất cứ điều gì.

Answer 7

Bạn có dự định thực hiện Chứa và đảm bảo bạn sẽ đăng ký Có trên tất cả các đối tượng có thể nơi bạn có thể sử dụng phương pháp mở rộng này không?

Nếu một số đối tượng nhất định kiểm tra bình đẳng bằng toán tử quá tải == thì giải pháp chung của bạn sẽ không thành công. Điều đó làm cho nó không phải là một sự tương đương thực sự của nhiều thử nghiệm ==. Đây cũng là một ví dụ điển hình về nguy cơ viết các phương pháp mở rộng!

Mã LINQ sau hoạt động khi bạn thực hiện quá tải toán tử cũng như nếu bạn đang sử dụng giá trị mặc định == của tham chiếu đối tượng, để nói rằng giá trị thực sự là cùng một đối tượng như value1, 2, 3 hoặc 4, V như các loại đối tượng của giá trị của bạn trong trường hợp đặc biệt này:

V[] lv = { value, value2, value3, value4 }; 
if (lv.Any(v => v==value)) 
    // do something 

Hoặc một phiên bản ngắn tay:

if (new List<V>{value, value2, value3, value4 }.Any(v => v==value)) 
    // do something 

tôi không thể có được các biểu thức lambda trên để làm việc trong một phương pháp khuyến nông chung .

Là một tốt (nếu không liên quan) ví dụ về những gì tôi nghĩ là một đáng yêu, cú pháp có thể đọc được, thành ngữ trong Python sẽ

if value in (value1, value2, value3, value4): 

Answer 8

Nếu bạn chỉ kiểm tra một giá trị Enum (như bạn nói trong bình luận về Rogers trả lời), bạn nên sử dụng một FlagsAttribute trên Enum.

[Flags] 
public enum Value 
{ 
    Value1 = 0, 
    Value2 = 1, 
    Value3 = 2, 
    Value4 = 4 
} 

Value value = Value.Value1; 
if (value | Value.Value1 | Value.Value2 | Value.Value3 | Value.Value4) 
{ 
    // You can also use other bitwise operations, like & (AND),^(XOR) and ~ (NOT) 
} 

Nếu không; nếu nó là tên miền cụ thể, hãy thêm nó vào logic nghiệp vụ của bạn. Nếu nó là chung chung, hãy tạo một lớp trợ giúp.