我刚刚完成了为编程课程编写程序,我想避免使用魔术数字,所以这是我的问题：

在下面的函数中,我的数组索引器是否会被视为幻数？

码：

string CalcGrade(int s1,int s2,int s3,double median)
{
const int SIZE = 23;
const int LETTER_GRADE_BARRIERS[SIZE] = { 400,381,380,361,360,341,340,321,320,301,300,281,280,261,260,241,240,221,220,201,200,181,180 }; 
double finalGrade;
string letterGrade;

finalGrade = s1 + s2 + s3 + median;

if (finalGrade >= LETTER_GRADE_BARRIERS[1] && finalGrade <= LETTER_GRADE_BARRIERS[0])
{
    letterGrade = "A";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[3] && finalGrade <= LETTER_GRADE_BARRIERS[2])
{
    letterGrade = "A-";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[5] && finalGrade <= LETTER_GRADE_BARRIERS[4])
{
    letterGrade = "B+";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[7] && finalGrade <= LETTER_GRADE_BARRIERS[6])
{
    letterGrade = "B";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[9] && finalGrade <= LETTER_GRADE_BARRIERS[8])
{
    letterGrade = "B-";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[11] && finalGrade <= LETTER_GRADE_BARRIERS[10])
{
    letterGrade = "C+";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[13] && finalGrade <= LETTER_GRADE_BARRIERS[12])
{
    letterGrade = "C";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[15] && finalGrade <= LETTER_GRADE_BARRIERS[14])
{
    letterGrade = "C-";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[17] && finalGrade <= LETTER_GRADE_BARRIERS[16])
{
    letterGrade = "D+";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[19] && finalGrade <= LETTER_GRADE_BARRIERS[18])
{
    letterGrade = "D";
}
else if (finalGrade >= LETTER_GRADE_BARRIERS[21] && finalGrade <= LETTER_GRADE_BARRIERS[20])
{
    letterGrade = "D-";
}
else if (finalGrade <= LETTER_GRADE_BARRIERS[22])
{
    letterGrade = "Fail";
}

return letterGrade;
}

谢谢！

是的,除了-1,0或1之外的任何数字都可能是一个神奇的数字.

除非你是一个真正的大师,否则你可能也被允许自由使用两个权力:-)

顺便说一句,你可能可以重构那些代码,使其更容易理解,例如：

string CalcGrade (int s1,double median) {
    // Grade lookup arrays. If grade is >= limit[n],string is grades[n].
    // Anything below D- is a fail.
    static const int Limits[] = {400,180 }; 
    static const int Grades[] = {"A+","A","A-","B+","B","B-","C+","C","C-","D+","D","D-"};

    double finalGrade = s1 + s2 + s3 + median;

    // Check each element of the array and,if the final grade is greater
    //   than or equal to,return the grade string.
    for (int i = 0; i < sizeof(Limits) / sizeof(*Limits); i++)
        if (finalGrade >= Limits[i])
            return Grades[i];

    // Otherwise,Failed.
    return "Fail";
}

这将删除遍布代码的魔术数字到一个区域,在这个区域中,它们的工作方式很明显(假设您很好地对齐它们).

它还消除了原始解决方案的一个问题,即我们对得分为380.5的人做了什么 – 失败那些人并不公平:-)或者为400以上的人分配一个等级(因为那里没有)看起来似乎是回归“A”的一种方式.

为什么隐式铸造甚至被允许？我的意思是隐式地将浮点数转换为int的好处是什么？显式转换是否使代码更易读和更容易调试？

第一：C中的大量隐式转换是由于
历史原因,没有别的.我认为没有人会考虑
他们都是个好主意.另一方面,有很多不同
隐式转换的类型,其中一些几乎是必不可少的
语言：如果您需要显式转换,您不会喜欢它
传递MyType x;一个函数采用MyType const&amp ;;我很漂亮
确定有一个共识,const转换添加const,比如
这个,应该是隐含的.

关于没有达成共识的转换：

>几乎没有人似乎有无损转换的问题;
比如int to long,或者float to double.大多数人也
似乎接受从整数类型到浮点的转换(例如
int to double),虽然在某些情况下这些可能会松散.
(int i = 123456789; float f = i ;,例如.)
>在C 98标准化过程中有一项提议要求弃用
缩小转换次数,例如float to int. (作者)
提议是Stroustrup;如果你不喜欢这样的转换,那你就进去了
好公司.)没有通过;我不知道为什么,但我怀疑
这是一个从C的传统中挣脱太多的问题.
在C 11中,在某些较新的结构中禁止进行此类转换,
像新的初始化序列.所以听起来像是有的
一致认为这些隐含的转换并不是一个好主意,
但是因为害怕破坏代码或者只是因为它们而无法删除它们
打破了C的传统.(我知道不止一些人
不喜欢someString = 3.14159;是一份法律声明,
在字符串的末尾添加一个ETX字符.)
> bool的原始提议建议弃用所有
将数字和指针类型转换为bool.这被删除了;
很快就会明白,如果提出的话,该提案就不会通过
if(somePointer)之类的东西(相对于
if(somePointer！= NULL))非法.还有很大一部分
人们(包括我自己)认为这种转变“糟糕”,并避免
他们.

最后：编译器可以自由发出任何感觉的警告
喜欢.如果市场坚持要求进行此类转换,编译器
会实施它们(可能作为一种选择).我怀疑了
他们不这样做的原因是由于警告的声誉不好
初始实现产生太多警告.积分
促销会带来许多无人想要的缩小转化
消除：

char ch = '0' + v % 10;

例如,涉及int到char的转换(即
变窄);在C 11：

char ch{ '0' + v % 10 };

是非法的(但VC和g都接受它,g警告).一世怀疑可以使用,禁止缩小转换率至少会必须对更广泛类型本身所致的情况做出例外积分推广,混合型算术和来源的案例expression是一个编译时常量,它“适合”目标类型.