Tôi bị lẫn lộn giữa các hàm min và max, trong các ngữ cảnh nhất định.

Trong một ngữ cảnh, khi bạn đang sử dụng các hàm để lấy giá trị lớn hơn hoặc nhỏ hơn của hai giá trị, không có vấn đề gì. Ví dụ,

//how many autographed CD's can I give out?
int howManyAutographs(int CDs, int Cases, int Pens)
{
    //if no pens, then I cannot sign any autographs
    if (Pens == 0)
        return 0;

    //I cannot give away a CD without a case or a case without a CD
    return min(CDs, Cases);
}

Dễ dàng. Nhưng trong một bối cảnh khác, tôi bị lẫn lộn. Nếu tôi đang cố gắng đặt tối đa hoặc tối thiểu, tôi sẽ nhận được ngược lại.

//return the sum, with a maximum of 255
int cappedSumWRONG(int x, int y)
{
    return max(x + y, 255); //nope, this is wrong
}

//return the sum, with a maximum of 255
int cappedSumCORRECT(int x, int y)
{
    return min(x + y, 255); //much better, but counter-intuitive to my mind
}

Có phải là không thể thực hiện các chức năng của riêng tôi như sau?

//return x, with a maximum of max
int maximize(int x, int max)
{
    return min(x, max);
}

//return x, with a minimum of min
int minimize(int x, int min)
{
    return max(x, min)
}

Rõ ràng, sử dụng các nội trang sẽ nhanh hơn nhưng điều này có vẻ như là một sự tối ưu hóa vi mô không cần thiết đối với tôi. Có bất kỳ lý do khác điều này sẽ là không thể? Thế còn trong một dự án nhóm?

Như những người khác đã đề cập: không tạo một hàm có tên tương tự như hàm dựng sẵn, thư viện chuẩn hoặc hàm thường được sử dụng rộng rãi mà thay đổi hành vi của nó. Bạn có thể quen với quy ước đặt tên ngay cả khi nó không có ý nghĩa gì với bạn ngay từ cái nhìn đầu tiên nhưng sẽ không thể suy luận về chức năng của mã của bạn một khi bạn giới thiệu các chức năng khác làm điều tương tự nhưng có tên của họ hoán đổi.

Thay vì quá tải, các tên được sử dụng bởi thư viện tiêu chuẩn, hãy sử dụng các tên mới để truyền đạt chính xác ý của bạn. Trong trường hợp của bạn, bạn không thực sự quan tâm đến một mức tối thiểu của ED. Thay vào đó, bạn muốn giới hạn một giá trị. Về mặt toán học, đây là hoạt động tương tự nhưng về mặt ngữ nghĩa, nó không hoàn toàn. Vậy tại sao không chỉ là một chức năng

int cap(int value, int limit) { return (value > limit) ? limit : value; }

đó là những gì cần thiết và nói như vậy từ tên của nó. (Bạn cũng có thể thực hiện capvề minnhư trong timster 's câu trả lời ).

Một tên hàm thường được sử dụng là clamp. Phải mất ba đối số và kẹp kẹp một giá trị được cung cấp vào khoảng được xác định bởi hai giá trị khác.

int clamp(int value, int lower, int upper) {
    assert(lower <= upper);  // precondition check
    if (value < lower) return lower;
    else if (value > upper) return upper;
    else return value;
}

Nếu bạn đang sử dụng tên hàm thường được biết đến như vậy, bất kỳ người mới nào tham gia nhóm của bạn (bao gồm cả tương lai bạn sẽ quay lại mã sau một thời gian) sẽ nhanh chóng hiểu điều gì đang xảy ra thay vì nguyền rủa bạn vì đã nhầm lẫn họ bằng cách phá vỡ chúng kỳ vọng về tên chức năng mà họ nghĩ rằng họ biết.

Nếu bạn thực hiện một chức năng như thế khi minimize(4, 10)trả về 10 , thì tôi sẽ nói điều đó là không thể bởi vì các lập trình viên đồng nghiệp của bạn có thể bóp nghẹt bạn.

(Được rồi, có thể họ sẽ không bóp cổ bạn đến chết theo nghĩa đen, nhưng nghiêm túc ... Đừng làm vậy.)

Bí danh một chức năng là tốt, nhưng đừng cố gắng thay đổi ý nghĩa của các thuật ngữ hiện có

Bạn có thể tạo bí danh cho hàm - các thư viện thông thường luôn làm điều đó .

Tuy nhiên, đó là một ý tưởng tồi khi sử dụng các thuật ngữ theo cách trái ngược với cách sử dụng thông thường, như ví dụ của bạn, nơi mà tâm trí của bạn tối đa và tối thiểu nên được lật. Điều đó gây nhầm lẫn cho các lập trình viên khác, và bạn sẽ tự làm mất tinh thần bằng cách tự rèn luyện để tiếp tục diễn giải các thuật ngữ này theo cách không chuẩn.

Vì vậy, trong trường hợp của bạn, hãy từ bỏ cách nói "tối thiểu / tối đa" mà bạn thấy khó hiểu và tạo mã riêng, dễ hiểu.

Tái cấu trúc ví dụ của bạn:

int apply_upper_bound(int x, int y)
{
    return min(x, y);
}


int apply_lower_bound(int x, int y)
{
    return max(x, y)
}

Là một phần thưởng bổ sung, mỗi khi bạn nhìn vào mã này, bạn sẽ tự nhắc nhở mình cách sử dụng tối thiểu và tối đa trong ngôn ngữ lập trình của bạn. Cuối cùng, nó sẽ có ý nghĩa trong đầu của bạn.

Tôi thích câu hỏi này. Hãy phá vỡ nó đi.

1: Bạn có nên bọc một dòng mã?

Vâng, tôi có thể nghĩ ra rất nhiều ví dụ mà bạn có thể làm điều này. Có thể bạn đang thực thi các tham số đã nhập hoặc ẩn một triển khai cụ thể đằng sau một giao diện. Trong ví dụ của bạn, về cơ bản bạn đang ẩn một cuộc gọi phương thức tĩnh.

Ngoài ra, bạn có thể làm rất nhiều thứ trong một dòng trong những ngày này.

2: Có phải tên 'Min' và 'Max' khó hiểu

Vâng! Họ hoàn toàn là vậy! Một guru mã hóa sạch sẽ đổi tên chúng thành "FunctionWhichReturnsTheLargestOfItsParameter" hoặc một cái gì đó. May mắn thay, chúng tôi có tài liệu và (nếu bạn may mắn) IntelliSense và nhận xét để giúp chúng tôi để bất kỳ ai bị nhầm lẫn với tên đều có thể đọc được những gì họ phải làm.

3: Bạn có nên đổi tên chúng thành một cái gì đó khác cho mình.

Yup, đi cho nó. Chẳng hạn, bạn có thể có:

class Employee
{
    int NumberOfHolidayDaysIShouldHave(int daysInLue, int maxAllowableHolidayDays)
    {
         // Return the number of days in lue, but keep the value under the max allowable holiday days!
         // Don't use max, you fool!!
         return Math.Max(daysInLue, maxAllowableHolidayDays)
    }
}

Nó thêm ý nghĩa và người gọi không cần phải biết cách tính giá trị.

4: Bạn có nên đổi tên "min" thành "tối đa hóa"

Không!! bạn điên à?! Nhưng có, câu hỏi nhấn mạnh điểm mà những người khác nhau đọc các ý nghĩa khác nhau thành tên hàm và đối tượng. Những gì một người tìm thấy rõ ràng và thông thường người khác tìm thấy mờ đục và khó hiểu. Đó là lý do tại sao chúng tôi có ý kiến. Thay vào đó bạn nên viết:

// Add x and y, but don't let it go over 255
s = min(x + y, 255);

Rồi khi ai đó đọc

// Add x and y, but don't let it go over 255
s = max(x + y, 255);

họ biết bạn đã làm sai.

Không . Không tạo các hàm có tên rất giống với các hàm dựng sẵn, nhưng thực tế thì ngược lại . Nó có vẻ trực quan với bạn, nhưng sẽ rất khó hiểu với các nhà phát triển khác, và thậm chí với chính bạn một thời gian trong tương lai khi bạn có nhiều kinh nghiệm hơn.

Ý nghĩa của max"tối đa là", nhưng sự hiểu biết "trực quan" của bạn là một cái gì đó như "đến mức tối đa". Nhưng điều này chỉ đơn giản là một sự hiểu biết sai về chức năng và việc thay đổi tên từ maxthành maximumkhông truyền đạt cách hiểu khác nhau của bạn. Ngay cả khi bạn tin tưởng mạnh mẽ rằng các nhà thiết kế ngôn ngữ đã mắc lỗi, đừng làm điều gì đó như thế này.

Nhưng thay đổi tên để nói cap(x, limit)như đã được đề xuất sẽ ổn, vì rõ ràng nó truyền đạt ý định, ngay cả khi nó chỉ kết thúc tốt đẹp min.

Điều có thể khiến bạn bối rối là sử dụng Capped trong tên hàm của bạn hoặc sự hiểu biết của bạn về việc đặt nắp có nghĩa là gì. Nó là một giới hạn và không yêu cầu tối đa bất cứ điều gì.

Nếu bạn được hỏi thấp nhất, nhỏ nhất hoặc sớm nhất, bạn có cảm thấy Max là chức năng phù hợp không?

Để lại tối thiểu và tối đa một mình. Viết bài kiểm tra để ít nhất bạn sẽ làm cho nó đúng lần thứ hai.

Nếu bạn được yêu cầu sử dụng các chức năng này rất nhiều trong dự án của mình, bạn sẽ đưa ra một số gợi ý để giúp bạn làm rõ nên sử dụng chức năng nào. Sắp xếp giống như <hoặc>, phần rộng của miệng phải đối mặt với giá trị lớn hơn.

Để trả lời câu hỏi của bạn: Có bất kỳ lý do nào khác điều này sẽ không thể hiểu được? Thế còn trong một dự án nhóm? Nó có ý nghĩa rằng bạn muốn các chức năng của riêng bạn, không có vấn đề. Chỉ cần đảm bảo rằng họ đang ở trong lớp người trợ giúp của riêng bạn và không dễ gọi cho người khác trừ khi họ nhập nó. (Joes.Utilities.)

Nhưng để nhìn lại vấn đề của bạn, về cơ bản, tôi sẽ nghĩ:

return (input >= 255) ? 255 : input;

Bạn đang bối rối vì bạn đang cố gắng áp dụng logic não của mình cho các chức năng tối thiểu / tối đa này. Thay vào đó chỉ nói bằng tiếng Anh. ifnhững inputlà greater than or equal to 255 then return 255trường hợp returncác input.

Đó là:

if (input >= 255) {
   255 
} else {
   input
}

Quan điểm của tôi. Bạn đang sử dụng các hàm max \ min vì những lý do sai, tốc độ của những thứ này là không đáng kể. Làm những gì có ý nghĩa.

Trong khi tôi hiểu vấn đề của bạn, tôi sẽ miễn cưỡng làm điều này. Sẽ tốt hơn nếu chỉ cần khoan vào hộp sọ của bạn những gì min () và max () làm.

Hầu hết các lập trình viên đều biết các hàm min () và max () làm gì - ngay cả khi, giống như bạn, đôi khi họ phải vật lộn với trực giác của mình để sử dụng bất cứ lúc nào. Nếu tôi đang đọc một chương trình và thấy max (x, y), tôi sẽ biết ngay nó làm gì. Nếu bạn tạo chức năng "bí danh" của riêng mình, thì bất kỳ ai khác đọc mã của bạn sẽ không biết bí danh này làm gì. Họ phải tìm chức năng của bạn. Nó không cần thiết phá vỡ dòng chảy của việc đọc và buộc người đọc phải suy nghĩ thêm để hiểu chương trình của bạn.

Nếu bạn gặp khó khăn trong việc tìm ra cách sử dụng tại một số điểm, tôi nói, hãy thêm một nhận xét giải thích nó. Sau đó, nếu một người đọc trong tương lai bị nhầm lẫn tương tự, bình luận của bạn sẽ làm rõ nó. Hoặc nếu bạn làm sai nhưng nhận xét giải thích những gì bạn đang cố gắng làm, người cố gắng gỡ lỗi sẽ có manh mối.

Khi bạn đặt bí danh cho một hàm vì tên đó đụng độ với trực giác của bạn ... đây có phải là trường hợp duy nhất có vấn đề không? Hoặc bạn sẽ bí danh các chức năng khác? Có thể bạn đang bối rối khi "đọc" và thấy dễ dàng hơn khi nghĩ nó là "chấp nhận", bạn thay đổi "nối" thành "StringTogether", "round" thành "DropDecimals", v.v., v.v ... Hãy biến điều này thành cực kỳ lố bịch và các chương trình của bạn sẽ không thể hiểu được.

Thật vậy, nhiều năm trước tôi đã làm việc với một lập trình viên không thích tất cả các dấu câu trong C. Vì vậy, anh ta đã viết một loạt các macro để cho anh ta viết "THEN" thay vì "{" và "END-IF" thay vì "}" và hàng tá những sự thay thế khác. Vì vậy, khi bạn cố đọc các chương trình của anh ấy, nó thậm chí không còn giống C nữa, nó giống như phải học một ngôn ngữ hoàn toàn mới. Bây giờ tôi không nhớ là "VÀ" được dịch thành "&" hay "&&" - và đó là điểm chính. Bạn làm suy yếu những người đầu tư đã thực hiện trong việc học ngôn ngữ và thư viện.

Điều đó nói rằng, tôi sẽ không nói rằng một hàm không làm gì ngoài việc gọi một hàm thư viện chuẩn là nhất thiết phải xấu. Nếu quan điểm của chức năng của bạn không phải là tạo bí danh, mà là đóng gói hành vi chỉ xảy ra là một chức năng duy nhất, thì điều này có thể tốt và đúng đắn. Ý tôi là, nếu theo logic và chắc chắn bạn phải thực hiện tối đa tại thời điểm này trong chương trình, thì chỉ cần gọi trực tiếp tối đa. Nhưng nếu bạn phải thực hiện một số tính toán mà ngày nay yêu cầu tối đa, nhưng điều đó có thể được sửa đổi trong tương lai để làm một cái gì đó khác, thì một hàm trung gian là phù hợp.

Bạn có thể đổi tên các hàm được xây dựng với điều kiện là các tên mới giúp mã của bạn rõ ràng hơn và sẽ không bị ai hiểu. (Nếu bạn đang sử dụng C / C. .

Bạn không phải là người duy nhất gặp vấn đề này với min và max, tuy nhiên tôi vẫn chưa thấy một giải pháp chung tốt nào hoạt động trên tất cả các miền. Tôi nghĩ một vấn đề với việc đặt tên của các hàm này là hai đối số có ý nghĩa logic khác nhau, nhưng được trình bày có nghĩa giống nhau.

Nếu ngôn ngữ của bạn cho phép, bạn có thể thử

return  calculatedDiscount.ButNoMoreThen(maxAllowedDiscount)

return  CDsInStocked.ButNoMoreThen(CasesInStock)

Không.

Bạn không viết giấy gói của bạn. Tên của các hàm bao không quan trọng lắm.

Những gì bạn đang cố gắng làm là một loại mã xác thực. Bạn đang phát minh ra một lớp bổ sung phục vụ 2 mục đích:

1. Người khác không thể hiểu mã của bạn.
2. Bạn không bao giờ học cách hiểu mã của người khác.

Bằng cách che giấu những thứ bạn không thoải mái, bạn chỉ làm tổn thương mã của bạn bây giờ và chính bạn trong tương lai. Bạn không thể phát triển bằng cách ở trong vùng thoải mái của bạn. Những gì bạn cần là tìm hiểu làm thế nào minvà maxlàm việc.

Đó là Ok, và không thực sự phản trực giác khi sử dụng Min, Max để kiểm tra lại và hoàn thành. Điều này cũng được thực hiện bằng cách sử dụng:

• sàn () và trần ()
• kẹp, giới hạn, giới hạn
• và tất nhiên mod với cắt ngắn thứ tự cao.

Trong phần sụn, nó xuất hiện xa hơn MMX, bản thân nó có trước đồ họa 3D hiện đại dựa trên extensivley này.

Thay thế một chức năng tiêu chuẩn công nghiệp ngay cả tại địa phương sẽ làm tôi lo lắng, một tên phái sinh có thể tốt hơn. Sinh viên C ++ có thể quá tải cho lớp tối nghĩa của họ có lẽ.

Đó là tốt trong một số trường hợp, nhưng không trong ví dụ của bạn, bởi vì có những cách tốt hơn để từ đó:
saturate, clamp, clipvv

Tôi muốn tạo ra một hàm chung có tên là 'giới hạn'

//Assumes lower_bound <= upper_bound
template <typename T>
T bounded(T value, T lower_bound, T upper_bound){
    if (value < lower_bound)
        return lower_bound;
    if (value > upper_bound)
        return upper_bound;
    return value;
}

//Checks an upper (by default) or lower bound
template <typename T>
T bounded(T value, T bound, bool is_upper_bound = true){
    if (is_upper_bound){
        if (value > bound)
            return bound;
    }
    else {
        if (value < bound)
            return bound;
    }
    return value;
}

hoặc với việc sử dụng 'min' và 'max'

//Assumes lower_bound <= upper_bound
template <typename T>
T bounded(T value, T lower_bound, T upper_bound){
    return max(min(value, upper_bound), lower_bound);
}

//Checks an upper (by default) or lower bound
template <typename T>
T bounded(T value, T bound, bool is_upper_bound = true){
    if (is_upper_bound)
        return min(value, bound);
    else
        return max(value, bound);
}

Điều gì về việc gọi các chức năng của bạn:

atmost(x,255): trả về mức thấp nhất của x hoặc 255.

atleast(10,x): trả về giá trị cao hơn của x hoặc ít nhất là 10.

min(x+y, MAX_VALUE); sẽ mang nhiều ý nghĩa hơn myCustomFunction(x, y);

Vì vậy, câu trả lời là CÓ, nó là không thể tin được . Nó chỉ phục vụ như một bí danh cho ngôn ngữ não của bạn.