如何优化嵌套循环次数不定的代码

[清钟沁桐]

当前实现非常的不完美，根据循环次数走不同的条件分支，有什么办法能将这两个分支给统一吗？

echo trac_rows=${trac_rows}
count=0;
# start to dispose the options
if [ ${trac_rows} -eq 1 ]; then
{
	LOOP=$((${start[1]}))
	while [[ $LOOP -le $((${end[1]})) ]]; do
	{
		echo LOOP[1]=$LOOP
		echo count=$count
		count=$((count+1))
		LOOP=`expr ${LOOP} + $((${step[1]}))` 
	}
	done
}
elif [ ${trac_rows} -eq 2 ]; then
{
	LOOP[1]=$((${start[1]}))
	while [[ ${LOOP[1]} -le $((${end[1]})) ]]; do
	{
		LOOP[2]=$((${start[2]}))
		while [[ ${LOOP[2]} -le $((${end[2]})) ]]; do
		{
			echo LOOP[2]=$LOOP
			echo count=$count
			count=$((count+1))
			LOOP[2]=`expr ${LOOP[2]} + $((${step[2]}))`
		}
		done
		LOOP[1]=`expr ${LOOP[1]} + $((${step[1]}))` 
	}
	done
}
else
{
	echo "too many nesting options"
}
fi

[askandstudy]

不习惯写shell，看着有点头大。
if [ ${trac_rows} -eq 1 ]这部分不要，合并到elif [ ${trac_rows} -eq 2 ]里面的代码中，进入的条件也合并在一起，在最内层循环加上条件判断，符合条件做完一次操作后就break，不符合就不会break就是多层循环。
是不是这样呢

[askandstudy]

好像也很土，应该是在：
while [[ ${LOOP[2]} -le $((${end[2]})) ]]; do这句前面根据条件来判断，设置循环变量的值以便让最内层循环在${trac_rows}等于1时只循环一次。

[清钟沁桐]

在while侧加上 ${trac_rows} 值的判断，是不是可行呢？
LOOP[1]=$((${start[1]}))
while [[ ${LOOP[1]} -le $((${end[1]})) ]] && [[ ${trac_rows} -eq 1 ]]; do
{
LOOP[2]=$((${start[2]}))
while [[ ${LOOP[2]} -le $((${end[2]})) ]] && [[ ${trac_rows} -eq 2 ]]; do
{
echo LOOP[2]=$LOOP
echo count=$count
count=$((count+1))
LOOP[2]=`expr ${LOOP[2]} + $((${step[2]}))`
}
done
LOOP[1]=`expr ${LOOP[1]} + $((${step[1]}))`
}
done

[askandstudy]

[Quote=引用 3 楼 zhongyunde 的回复:]

在while侧加上 ${trac_rows} 值的判断，是不是可行呢？
LOOP[1]=$((${start[1]}))
while [[ ${LOOP[1]} -le $((${end[1]})) ]] && [[ ${trac_rows} -eq 1 ]]; do
{
LOOP[2]=$((${start[2]}))
……
[/Quote]


这样不对吧？当${trac_rows} -eq 1时你最初贴的代码还是有可能会执行：
echo LOOP[1]=$LOOP
echo count=$count
count=$((count+1))
LOOP=`expr ${LOOP} + $((${step[1]}))`
这几句的，
像你最后贴的这样那不是${trac_rows} -eq 1时永远也不会执行那里面那四条语句了？

我觉得可能还是在while [[ ${LOOP[2]} -le $((${end[2]})) ]]; do这句之前做个条件判断，然后设置循环次数变量以便保证在${trac_rows}等于1那四条语句只循环一次，在${trac_rows}等于2时多次循环。

[askandstudy]

不好意思，上面的回复我又想错了，&&我又想成其它语言里面的意思了。先回复一下，还需要再看看。

[skyzhg]

askandstudy 回复了您的帖子：如何优化嵌套循环次数不定的代码 45分钟前
屏蔽 askandstudy 回复了您的帖子：如何优化嵌套循环次数不定的代码 1小时前
屏蔽 askandstudy 回复了您的帖子：如何优化嵌套循环次数不定的代码 12-14 14:44
屏蔽 askandstudy 回复了您的帖子：如何优化嵌套循环次数不定的代码 12-14 14:08

为什么别人帖子的通知都到我这来啊

[askandstudy]

每天在各种语言之间转，头脑一篇浆糊，但是又想了下觉得你在3楼这样写的会不对，我没有测试。我自己试了下我的方式好像可以：

[root@RHEL6A shcode]# cat temp.sh 
#!/bin/sh
trac_rows=$1
count=0;
start[1]=0
end[1]=2
step[1]=1
start[2]=0
end[2]=3
step[2]=1
# start to dispose the options
if [ ${trac_rows} -eq 1 ]; then
{
    LOOP=$((${start[1]}))
	echo LOOP:$LOOP $((${end[1]}))
    while [[ $LOOP -le $((${end[1]})) ]]; do
    {
        echo LOOP[1]=$LOOP
        echo count=$count
        count=$((count+1))
        LOOP=`expr ${LOOP} + $((${step[1]}))` 
		sleep 1 
    }
    done
}
elif [[ ${trac_rows} -eq 2 ]] || [[ ${trac_rows} -eq 3 ]]; then
{
    LOOP[1]=$((${start[1]}))
    while [[ ${LOOP[1]} -le $((${end[1]})) ]]; do
    {
        LOOP[2]=$((${start[2]}))
		if [[ ${trac_rows} -eq 3 ]]
		then
			LOOP[2]=${end[2]}
		fi
        while [[ ${LOOP[2]} -le $((${end[2]})) ]]; do
        {
            echo LOOP[2]=$LOOP
            echo count=$count
            count=$((count+1))
            LOOP[2]=`expr ${LOOP[2]} + $((${step[2]}))`
			sleep 1
        }
        done
        LOOP[1]=`expr ${LOOP[1]} + $((${step[1]}))` 
    }
    done
}
else
{
    echo "too many nesting options"
}
fi

[root@RHEL6A shcode]# ./temp.sh 1
LOOP:0 2
LOOP[1]=0
count=0
LOOP[1]=1
count=1
LOOP[1]=2
count=2
[root@RHEL6A shcode]# ./temp.sh 2
LOOP[2]=
count=0
LOOP[2]=
count=1
LOOP[2]=
count=2
LOOP[2]=
count=3
LOOP[2]=
count=4
LOOP[2]=
count=5
LOOP[2]=
count=6
LOOP[2]=
count=7
LOOP[2]=
count=8
LOOP[2]=
count=9
LOOP[2]=
count=10
LOOP[2]=
count=11
[root@RHEL6A shcode]# ./temp.sh 3
LOOP[2]=
count=0
LOOP[2]=
count=1
LOOP[2]=
count=2
[root@RHEL6A shcode]# 

我加了个[[ ${trac_rows} -eq 3 ]]来测试的，替换成等于1的判断再把前面那段删除就是在一段代码里处理了。我还是用的我在4楼说的思路的做法。
写得比较菜，有什么不对的地方或者我理解错误请指正。

[shuyunhai]

不清楚lz为什么要优化 这段代码根据 ${trac_rows}的值做不同的循环

如果仅仅是为了减少代码量 就是 在if判断${trac_rows}的时候加或
然后在循环时再判断一次是否执行内侧循环,这样的话代码的可读性降低了

要是改成别的方式就跟上面那段代码的原意不同了

[shuyunhai]

7楼这个就是我说的 还是觉得楼主那段代码已经足够清晰了 也没浪费资源 何必非要改呢

[Quote=引用 7 楼 askandstudy 的回复:]
每天在各种语言之间转，头脑一篇浆糊，但是又想了下觉得你在3楼这样写的会不对，我没有测试。我自己试了下我的方式好像可以：


Assembly code

[root@RHEL6A shcode]# cat temp.sh
#!/bin/sh
trac_rows=$1
count=0;
start[1]=0
end[1]=2
step[1]=1
start[2]=0
en……
[/Quote]

[清钟沁桐]

关键是我要处理循环次数 不定的，也可能是4次 ，5次的。。。

[shuyunhai]

[Quote=引用 10 楼 zhongyunde 的回复:]
关键是我要处理循环次数 不定的，也可能是4次 ，5次的。。。
[/Quote]

那应该是递归调用 ，我写了一个

#!/bin/sh
trac_rows=$1

if [ $# = 2 ]; then
	count=$2
else
	count=0
fi

start[1]=1
step[1]=1
end[1]=2

start[2]=1
step[2]=1
end[2]=1

max=4

if [ ${trac_rows} -le ${max} ]; then
	times=${trac_rows}
	LOOP[1]=${start[1]}
	while [ ${LOOP[1]} -le ${end[1]} ]; do
	{
		while [ ${times} -gt 0 ]; do
		{
			if [ ${times} -gt 1 ]; then
				times=`expr ${times} - 1`
				#echo "tmp.sh ${times} ${count}"
				tmp.sh ${times} ${count}
				count=$?
			else
				LOOP[2]=${start[2]}
				while [ ${LOOP[2]} -le ${end[2]} ]; do
		        {
		            echo LOOP=${LOOP[2]}
		            echo count=$count
		            count=`expr ${count} + 1`
		            LOOP[2]=`expr ${LOOP[2]} + ${step[2]}`
		            sleep 1
		        }
		    	done
		    	times=`expr ${times} - 1`
		    fi
		}
		done
		LOOP[1]=`expr ${LOOP[1]} + ${step[1]}`
	}
	done
	exit ${count};
else
{
    echo "too many nesting options"
    exit -1;
}
fi

[askandstudy]

这都可以通过判断参数值来设置循环变量的起止值实现啊

[askandstudy]

或者用11楼说的试试看

[清钟沁桐]

楼上同时熟悉很多语言 真的很难得

[清钟沁桐]

不是这么执行吗 ？
[zhongyunde@hcc-67 ~/hc3_sdv_test/toolset]$sh tmp.sh 1 2
: command not found
tmp.sh: line 58: syntax error: unexpected end of file

[askandstudy]

熟悉很多东西会变成打杂工的，精通一门的才是专家。

shell说：你的tmp.sh第58行那附近有问题，语法错误，检查看看

[shuyunhai]

我这里执行是没问题的

应该是你粘代码的时候 文件结尾处有问题

[Quote=引用 15 楼 zhongyunde 的回复:]
不是这么执行吗 ？
[zhongyunde@hcc-67 ~/hc3_sdv_test/toolset]$sh tmp.sh 1 2
: command not found
tmp.sh: line 58: syntax error: unexpected end of file
[/Quote]

[shuyunhai]

我的执行结果

hnjyfx1@infor $ tmp.sh 1 2
LOOP=1
count=2

[清钟沁桐]

是的。
使用 dos2unix tmp.sh 即可