c# – 方法调用不正确的做法？

这是我对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)维护的难度.

我已经在线搜索了我可以构建的关于这个问题的每一个短语,但是空洞地出现了.我确实看到了罗伯特·C·马丁的一本名为“清洁代码”的书,我没有.

我与学生的关系有点像父母的关系.除非我能用独立的来源证实,否则他们有时候并没有把我说的大量存货.如果我可以指出一个关于在一个方法调用中放置作业的明确声明,我的学生将更倾向于停止这种烦人的习惯.

这种用法会困扰其他人吗？我是否希望他改变他一直做事的方式？他15岁,但在他前面有一个充满希望的未来.他是那些只是“得到它”的学生之一.我不希望他发展不好的做法.

感谢您的时间和意见.

var i = 0;
Foo(i += 1);
Bar(i);