if 里面嵌套for，能优化一下代码吗？

#定义if的条件变量shuffle shuffle = true # shuffle = false
if(shuffle) { 口口g = 100; 口口m = 2; 口口blocks; # 这段代码输出是b，输入是x 口口for（int i=0; i< m; i++）{ 口口口口a = f1(b) + g; 口口} } else { 口口g= 200; 口口m = 3; 口口blocks; # 这段代码输出是b，输入是x，与上面那个无论是输入还是输出的数值都是一模一样的。 口口for（int i=0; i< m; i++）{ 口口口口a = f2(b) + g; 口口} }
请问这段代码还能优化吗？
因为这个blocks非常长，大概1万行。而这个是完全一模一样的。大家怎么处理这段代码？ 1） 把blocks变成一个函数。这个方法好吗？ 2） 有啥办法让blocks代码只写一次，然后把if嵌入到for里面去？ 3） 其他方法
类似上面这段代码，在编程的时候，有大量行的blocks是一模一样的。大家有不写成函数的时候吗？比如：输入参数太多了。输出参数多且类型也多。比如1000个输入参数，100个不同维数的矩阵输出。 不写成函数，但是重复代码，有什么利？有什么不利？ 会影响程序的运行速度吗？（好像还会比写成函数运算快一些。对吧。） 会占用更多的内存吗？

#定义if的条件变量shuffle shuffle = true # shuffle = false
if(shuffle) { 口口g = 100; 口口m = 2; 口口blocks; # 这段代码输出是b，输入是x 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口} } else { 口口g= 200; 口口m = 3; 口口blocks; # 这段代码输出是b，输入是x，与上面那个无论是输入还是输出的数值都是一模一样的。 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口} }
请问这段代码还能优化吗？
因为这个blocks非常长，大概1万行。而这个是完全一模一样的。大家怎么处理这段代码？ 1） 把blocks变成一个函数。这个方法好吗？ 2） 有啥办法让blocks代码只写一次，然后把if嵌入到for里面去？ 3） 其他方法


gooog 发表于 2023-09-14 10:29

你是新手吗？
为什么不写成这样：
if(shuffle) { 口口g = 100; 口口m = 2;
} else { 口口g= 200; 口口m = 3;
}
口口blocks; # 这段代码输出是b，输入是x，与上面那个无论是输入还是输出的数值都是一模一样的。 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口}

#定义if的条件变量shuffle shuffle = true # shuffle = false
if(shuffle) { 口口g = 100; 口口m = 2; 口口blocks; # 这段代码输出是b，输入是x 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口} } else { 口口g= 200; 口口m = 3; 口口blocks; # 这段代码输出是b，输入是x，与上面那个无论是输入还是输出的数值都是一模一样的。 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口} }
请问这段代码还能优化吗？
因为这个blocks非常长，大概1万行。而这个是完全一模一样的。大家怎么处理这段代码？ 1） 把blocks变成一个函数。这个方法好吗？ 2） 有啥办法让blocks代码只写一次，然后把if嵌入到for里面去？ 3） 其他方法


gooog 发表于 2023-09-14 10:29

可以把blocks弄成一个函数 在进入if statement之前算一次就好了 f（b）也可以只算一次在if 之前只算一次

可以把blocks弄成一个函数 在进入if statement之前算一次就好了 f（b）也可以只算一次在if 之前只算一次
xcosmos 发表于 2023-09-14 10:39

就是 b=blocks(x) new variable=f(b） if(shuffle){ for loop } else{ for loop }

你是新手吗？
为什么不写成这样：
if(shuffle) { 口口g = 100; 口口m = 2;
} else { 口口g= 200; 口口m = 3;
}
口口blocks; # 这段代码输出是b，输入是x，与上面那个无论是输入还是输出的数值都是一模一样的。 口口for（int i=0; i< m; i++）{ 口口口口a = f(b) + g; 口口}
dustflying 发表于 2023-09-14 10:38

不好意思。for循环里的f（b）在条件if下，一个是f1，一个是f2，它们有些细微的区别。

回复 1楼gooog的帖子
for loop 跟if 没有任何关系啊，为啥要写在if里面？ if 只管赋值就好

chatgpt一下

不好意思。for循环里的f（b）在条件if下，一个是f1，一个是f2，它们有些细微的区别。
gooog 发表于 2023-09-14 10:53

那就修改f1，f2成一样的，或者加parameter控制行为 软件可复用的基本原则就是模块化和consolidate重复逻辑

我觉得当年我果断放弃编程是正确的…

那就修改f1，f2成一样的，或者加parameter控制行为 软件可复用的基本原则就是模块化和consolidate重复逻辑
nickbear 发表于 2023-09-14 10:59

f1，f2不能改成一样的。受shuffle这个参数控制。

g = shuffle ? 100 : 200; m = shuffle ? 2 : 3; blocks; for（int i=0; i< m; i++）{ tmp = shuffle ? f1(b) : f2(b); a = tmp + g; }
马工的脸都让你们给丢尽了。

不好意思。for循环里的f（b）在条件if下，一个是f1，一个是f2，它们有些细微的区别。
gooog 发表于 2023-09-14 10:53

那就只把blocks放到if之前就好了

把for loop写成function take 你的各种变量 b x f 就不用重复了 我刚开始写代码的时候mentor说 never repeat yourself