这样的风格好吗？

Question

我的函数中有一些参数接收和传递相同的值。我应该将所有参数变量命名为相同吗？

示例：

// assume numMonth = 12
// assume numYear = 2000

int promptMonth(); // returns numMonth
int promptYear();  // returns numYear

int daysInMonth(int numMonth, int numYear); // numMonth and numYear will always
int daysInYear(int numYear);                // be sent to these.

bool isLeapYear(int numYear);               // daysInYear() and daysInMonth() call
                                            // this function and send it numYear 
                                            // (which would be 2000 in this case).

由于将相同的值传递给每个参数，所有这些参数是否应该命名相同？

Answer 1

同样，我假设您指的是函数参数名称 numYear。

是的，以有意义的方式命名变量始终是一个好习惯，该方式表明所传递值的含义/目的。

这些变量是不同函数的一部分，因此范围仅限于这些函数，因此如果您考虑到这一点，就不存在多个定义的问题。

Answer 2

一般来说，这种风格是个好主意。你绝对应该遵循它，但仍然存在误用和误解的空间。

让我解释一下自己

• 误解：即使您确实一致地命名变量，读者也可能不相信您的意思。
• 误用：也许没有 int 适合作为年份值。

对于更复杂的情况，您应该考虑为输入变量使用特殊类型。
例如，考虑将一些整数更改为：

Month promptMonth();
Year promptYear();

int daysInMonth( Month m, Year y); 
int daysInYear( Year y );

bool isLeapYear( Year );

您是否发现现在不仅不必使用复杂的变量名称，而且还可以进行特殊的打印和检查？

cout << "the current month is " << promptMonth();  

可以输出

the current month is 'October'