方法中的赋值调用不良做法？

问：

这是我向 Stackoverflow 提出的第一个问题，尽管我已经是多年的消费者。如果我违反规则，请原谅我。这当然不是我的本意。我已经仔细审查了这些规则，并相信我在可接受的范围内。但是，如果存在使用错误，请您指出这些错误，以便我将来更加合规。

我教高中生编程。这学期我们正在学习 C#。上周我们研究了递归。我分配了几个可以用递归解决的经典问题，其中之一就是幂。

我的一个学生提交了以下代码作为使用递归的幂运算解决方案（他确实允许我在这里发布它）。方法调用中的任务给了我威利斯，但当我告诉他这是不好的做法时，他抗议说“它有效”，对他来说“有意义”，而且他“一直都在这样做”。

static void Recur(int n1, int n2, int n3) 
{ 
   if (n2 > 0) 
   { 
      Recur(n1, n2 - 1, n3 *= n1);   // this is the line in question
   } 
   else 
   { 
      Console.WriteLine("The number calculated recursively is: {0}", n3); 
   } 
} 

我很难想出一些具体的东西来告诉我的学生，为什么在方法调用中进行作业通常是不好的做法，除了 1） 意外副作用的可能性，以及 2） 维护的困难。

我在网上搜索了关于这个问题的每一个短语，但结果却是空手而归。我确实看到了一本名为“Clean Code”的书，作者是Robert C. Martin，但我不拥有。

我和学生的关系有点像父母的关系。他们有时不会对我所说的话投入太多精力，除非我能用独立消息来源证实。如果我能指出一个关于将作业放在方法调用中的明确声明，我的学生会更倾向于停止这种烦人的习惯。

这种用法会打扰其他人吗？我是否期望他改变他做事的方式？他今年15岁，但前途一片光明。他是那些“明白”的学生之一。我不希望他养成不良习惯。

感谢您的时间和意见。