专业代码

Question

我已经编程了大约3年，并且对我的技能感到自信。但是最近我开始与嵌入式系统一起工作，并在其他人的代码上工作，并开始质疑我的代码有多好。

我看到了所有这些复杂的答案，并认为我会用矢量和if语句来做这些事情，并且想知道我是否是初学者，因为我是自学成才，并且不了解自己的水平。

所以我想知道更多有经验的程序员能否让我更好地做事。

这是我写给rpg游戏挑选攻击目标的代码。以此为例，您可以向我展示更好/更高级/更专业的方式。

int FindTarget(Player &player); 
{ 
    int aimPoint[5] = 0; 

    for(int i = 0; i <= 5; i++) 
    { 
     if(player.team[i].exist == true) 
     { 
      // set random starting point between 1 - 3 
      aimPoint[i] = random /3; 

      // add a point if you hve an elemental advantage minus if not 
      if(player.team[i].type == weak) 
      { 
       aimPoint += 1; 
      } 
      else if(player.team[i].type == strong) 
      { 
       aimPoint -= 1; 
      } 

       //add for front row minus for back 
      if(i == 1 || i== 3) 
      { 
       aimPoint += 1; 
      } 
      else 
      { 
       aimPoint -= 1; 
      } 
     } 
    } 

    return 0; 
} 

编辑：如果你没有时间或精力给我看的例子，我将不胜感激一本好书，一个初级/中级能理解。

Answer 1

啊哈，因为没有人提到这个问题，让我点这一个出：

int FindTarget(Player &player); 
{ 
    int aimPoint[5] = 0; 

    ... 
    return 0 
} 

to rough此外，由于不知道代码应该做什么（以及'aimPoint'如何与团队有关;猜测并没有帮助，因为使用没有它，瞄准点被丢弃？），我没有任何东西比固定破损明显，这是上面

从评论

---- 编辑

random的处理似乎被误导了。有人建议你可能意思是random %3 + 1;

我注意的是太多，但决定没有什么基础上的假设。也许随机已经是一个整数在范围[3, 12)。

此外，正确地使用时random % 3将不会产生的均匀分布，所以you'd need to do something else

对于许多应用，兰特（）将极好执行，但与事务的当前状态悲伤，兰特（ ）很少正确使用。

的问题是分配

Answer 2

这是毫无意义的;这不是一个循环，而是一次通过。

for(int i = 0; i <= 0; i++) 

Answer 3

• 评论＃1：

此行从0开始，并且去正好一次？为什么它是一个循环？

for(int i = 0; i <= 0; i++) 

• 评论＃2：

不要对true比较一个布尔值。

if(player.team[i].exist == true) 

只要将它重新写：

if(player.team[i].exist) 

• 评论＃3：

专业人员很少使用硬编码值。
为什么这个值是5？

int aimPoint[5] = 0; 

相反，要清楚：

int aimPoint[TARGET_SIZE] = 0; 

同样，改变

aimPoint[i] = random /3; 

到：

aimPoint[i] = random /INITIAL_TARGET_VALUE; 

Answer 4

这是我会怎么写。

void FindTarget(Player &player); 
{ 
    int aimPoint[5]; 

    for(int i = 0; i <= 5; ++i) 
    { 
     if(!player.team[i].exist) 
      continue; 

     // set random starting point between 1 - 3 
     aimPoint[i] = random/3; 

     // add a point if you hve an elemental advantage minus if not 
     switch(player.team[i].type) { 
      case weak: 
      ++aimPoint[i]; 
      break; 

      case strong: 
      --aimPoint[i]; 
      break; 
     } 

     //add for front row minus for back 
     if(i == 1 || i== 3) 
      ++aimPoint[i]; 
     else 
      --aimPoint[i]; 
    } 
} 

Answer 5

这一段代码：

// set random starting point between 1 - 3 
      aimPoint[i] = random /3; 

没有得到你1和3之间的随机数，试试这个：

aimPoint[i] = random % 3 + 1