在某些情况下，我对最小和最大函数感到困惑。

在一种情况下，当您使用函数取两个值中的更大或更小值时，就没有问题。例如，

//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);
}

简单。但是在另一种情况下，我感到困惑。如果尝试设置最大值或最小值，则会倒退。

//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
}

不能按以下方式进行我自己的功能？

//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)
}

显然，使用内置函数会更快，但这对我来说似乎是不必要的微优化。还有其他原因不建议这样做吗？在小组项目中呢？

正如其他人已经提到的那样：不要创建名称与内置，标准库或普遍使用的函数相似的函数，而要更改其行为。即使乍一看对您来说意义不大，也可能会习惯于命名约定，但是一旦您引入了其他功能相同但又具有相同功能的代码，就不可能推理出代码的功能他们的名字互换了。

不要“重载”标准库使用的名称，而是使用可以准确传达您的意思的新名称。在您的情况下，您对“最低”并不真正感兴趣。相反，您想限制一个值。从数学上讲，这是相同的操作，但从语义上讲，则不尽相同。那么为什么不只是一个功能

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

它会执行所需的操作，并从名称中告诉您。（您也可以cap按照timster的答案min所示实施）。

另一个常用的函数名称是clamp。它使用三个参数，并将提供的值“限制”在其他两个值定义的间隔中。

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;
}

如果您使用的是这样一个众所周知的函数名称，那么任何新加入团队的人（包括将来您会在一段时间后返回代码的人）都将迅速了解发生了什么，而不必因为破坏它们而使他们感到困惑，从而招致您的诅咒对他们认为知道的函数名称的期望。

如果您在minimize(4, 10)返回10的情况下创建这样的函数，那我不建议您这样做，因为您的其他程序员可能会扼杀您。

（好吧，也许他们不会从字面上勒死您，但要认真对待……不要那样做。）

给函数起别名是可以的，但是不要尝试更改现有术语的含义

可以创建函数的别名-通用库始终会这样做。

但是，以与常用用法相反的方式使用术语是个坏主意，例如您的示例中应该颠倒max和min。这会使其他程序员感到困惑，并且通过训练自己以非标准的方式继续解释这些术语，将会对您造成伤害。

因此，在您的情况下，请放弃使您感到困惑的“最小/最大”用语，并创建自己的易于理解的代码。

重构您的示例：

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


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

另外，每次查看此代码时，您都会想起自己在编程语言中如何使用min和max。最终，它在您的脑海中变得有意义。

我喜欢这个问题。让我们分解一下。

1：您应该包装一行代码吗？

是的，我可以想到很多可以执行此操作的示例。也许您是在强制键入类型的参数，还是在接口后面隐藏了具体的实现。在您的示例中，您实际上隐藏了一个静态方法调用。

另外，这些天您可以在一行中完成很多工作。

2：名称“ Min”和“ Max”是否令人困惑

是! 他们完全是！干净的编码专家会将其重命名为“ FunctionWhichReturnsTheLargestOfItsParameters”或其他名称。幸运的是，我们有文档以及（如果您幸运的话）IntelliSense和注释来帮助我们，以便每个对名称感到困惑的人都可以阅读他们应该做的事情。

3：您是否应该自己将它们重命名为其他名称。

是的，去吧。例如，您可能有：

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)
    }
}

它增加了含义，调用者不必或不想知道如何计算值。

4：您是否应该将“最小”重命名为“最大化”

没有！！你疯了吗？！但是是的，这个问题强调了这样一个观点，即不同的人将不同的含义理解为功能和对象名称。一个人发现清楚而又传统的另一个发现却不透明且令人困惑。这就是为什么我们要发表评论。您应该改写：

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

然后当有人读

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

他们知道您犯了一个错误。

没有。不要使名称与内置函数非常相似的函数，但实际上却相反。对于您来说，这似乎很直观，但是对于其他开发人员，甚至当您有更多经验的将来，对于您自己来说，这都会非常混乱。

的意思max是“最大的”，但是您对“直觉”的理解就像是“最大的”。但是，这是函数的只是一个错误的认识，并更改名称max以maximum不沟通你不同的解释。即使您坚信语言设计师犯了一个错误，也不要做这样的事情。

但是将名称更改cap(x, limit)为建议的名称就可以了，因为即使它只是换行，它也确实传达了意图min。

可能使您感到困惑的是在函数名称中使用了上限，或者您对上限的含义有所了解。它是一个限制器，不需要任何东西最多。

如果要求您提供最低，最小或最早的功能，您是否觉得Max是合适的功能？

保留最小和最大。编写测试，以便至少让您第二次正确。

如果您需要在项目中大量使用这些功能，则将提供一些提示，以帮助您明确使用哪个功能。有点像<或>，嘴巴的较大部分面对较大的值。

回答您的问题：还有其他不明智的理由吗？在小组项目中呢？您想要自己的功能很有意义，这没问题。只需确保它们在您自己的帮助器类中，并且除非他人导入，否则其他人不容易调用它们。（Joes.Utilities。）

但是要再次查看您的问题，基本上我会在想：

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

您会感到困惑，因为您试图将大脑逻辑应用于这些最小/最大函数。而是只说英语。if的input是greater than or equal to 255 then return 255，否则return的input。

这是：

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

我的意见。您出于错误原因选择了max \ min函数，这些东西的速度可以忽略不计。做有意义的事情。

虽然我了解您的问题，但我不愿意这样做。最好将min（）和max（）的内容简单钻入您的头骨。

大多数程序员都知道min（）和max（）函数的作用-即使像您一样，有时他们在任何给定时间都难以理解要使用的函数。如果我正在阅读程序并看到max（x，y），我会立即知道它的作用。如果您创建自己的“别名”函数，那么其他读取您的代码的人将不知道该别名的作用。他们必须找到您的功能。它不必要地打断了阅读流程，并迫使读者多加思考以理解您的程序。

如果您在某个时候无法确定要使用哪个，则可以添加一条注释对其进行解释。然后，如果将来的读者也有类似的困惑，您的评论应将其清除。或者，如果您做错了，但注释说明了您要执行的操作，则尝试进行调试的人员将有一个提示。

一旦您为函数加上别名，因为名称与您的直觉冲突……这是唯一有问题的情况吗？还是要别名其他功能？也许您对“阅读”感到困惑，并发现将其更容易理解为“接受”，将“追加”更改为“ StringTogether”，将“舍入”更改为“ DropDecimals”，依此类推。并且您的程序将变得令人费解。

确实，几年前，我曾与一个不喜欢C语言中所有标点符号的程序员一起工作。因此，他写了一堆宏，让他写“ THEN”而不是“ {”和“ END-IF”而不是“}”。以及其他数十种此类替代。因此，当您尝试阅读他的程序时，它甚至看起来都不再像C，这就像是必须学习一种全新的语言。我现在不记得是将“ AND”翻译成“＆”还是“ &&”了，这就是重点。您破坏了人们在学习语言和图书馆方面的投入。

就是说，我不会说只调用标准库函数就什么也不做的函数肯定是不好的。如果函数的目的不是创建别名，而是封装恰好是单个函数的行为，那么这可能是好方法。我的意思是，如果在逻辑上和不可避免的情况下必须在程序中进行此操作，则直接调用max。但是，如果您必须执行一些今天需要最大的计算，但是将来可能会修改以执行其他操作，则可以使用中间函数。

重命名内置函数是可以的，只要新名称使您的代码清晰明了，任何人都不会错过。（如果使用的是C / C ++，请不要使用#define，因为这样会使很难看到正在发生的事情。）函数的名称应像注释一样，解释调用代码在做什么以及为什么这样做。 。

您不是唯一遇到最小和最大问题的人，但是我还没有看到一种适用于所有领域的通用解决方案。我认为这些函数的命名存在一个问题，即这两个参数具有不同的逻辑含义，但被表示为相同的含义。

如果您的语言允许，您可以尝试

return  calculatedDiscount.ButNoMoreThen(maxAllowedDiscount)

return  CDsInStocked.ButNoMoreThen(CasesInStock)

没有。

您不写包装纸。这些包装的名称不是很重要。

您正在尝试做的是一种代码混淆。您正在发明一个具有两个目的的额外层：

1. 其他人无法理解您的代码。
2. 您永远不会学会理解别人的代码。

通过隐藏您不满意的内容，您现在只会伤害您的代码，将来也会伤害您自己。不能停留在舒适区来成长。你需要的是学习如何min和max工作。

可以，使用Min，Max来控制过冲和欠冲并不是真的违反直觉。也可以使用以下命令完成此操作：

• floor（）和ceil（）
• 钳位，限制，界限
• 当然还有具有高阶截断的mod。

在固件中，它的历史要比MMX早，后者本身早于依靠这种扩展功能的现代3D图形。

即使在本地替换行业标准功能也会让我担心，派生名称可能会更好。C ++学生可能会因为他们晦涩的课而超负荷工作。

这在某些情况下罚款，但不是在你的例子，因为有字这更好的方法：
saturate，clamp，clip等。

我宁愿创建一个名为“ bounded”的泛型函数

//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;
}

或使用“最小”和“最大”

//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);
}

调用函数呢：

atmost(x,255)：最多返回x或255的最小值。

atleast(10,x)：返回x的最大值或至少10。

min(x+y, MAX_VALUE); 会比 myCustomFunction(x, y);

因此答案是肯定的，这是不明智的。它仅用作您的大脑语言的别名。