帮忙鉴定下我这个shell逻辑正常吗

651260546

帮忙鉴定下我这个shell逻辑正常吗，不对的话给点建议
给点意见，谢谢
#!/bin/bash
t=423
t2=423
f=`df -h /file |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
s=`df -h /sata |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
while [ $s -ge 95 ] || [ $f -ge 95 ];
do
f=`df -h /file |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
s=`df -h /sata |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
if [ $s -ge 95 ];then
        find /sata  -name '*20*' -type f -amin +"$t" -exec rm -f {} \;
        t=$(awk 'BEGIN{print '$t' - 60 }')

fi

if [ $f -ge 95 ] && [ $s -ge 95 ];then
        find /file  -name '*20*' -type f -amin +"$t2" -exec mv {} /sata \;
        t2=$(awk 'BEGIN{print '$t' - 60 }')
fi

done

seanking1987

不正常，while的终止条件在哪？

#s=`df -h /ssd |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
#f=`df -h /sata |awk '{print $5}'|sed 's/.$//'|grep -v "Use"
这两行给注释掉了

651260546

回复 2# seanking1987
while里也用加个这3行是吗？


   

lrz

猜测楼主的意思是如果[ $s -ge 95 ] || [ $f -ge 95 ]，则执行while内的内个if，那不应该用while，直接用if呀

seanking1987

回复 3# 651260546


原始需求是什么样的？
从脚本看，while循环有不能终止的风险。
而且while里面的两个if也有相悖，第一个if满足条件的都删除，第二个if满足条件的都移走。
然后第一个if肯定先满足条件执行。

651260546

回复 5# seanking1987
我改了这样子
##############
#!/bin/bash
t=423
t2=423
f=`df -h /file |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
s=`df -h /sata |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
while [ $s -ge 95 ] || [ $f -ge 95 ];
do
f=`df -h /file |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
s=`df -h /sata |awk '{print $5}'|sed 's/.$//'|grep -v "Use"`
if [ $s -ge 95 ];then
        find /sata  -name '*20*' -type f -amin +"$t" -exec rm -f {} \;
        t=$(awk 'BEGIN{print '$t' - 60 }')

fi

if [ $f -ge 95 ] && [ $s -lt 95 ];then
        find /file  -name '*20*' -type f -amin +"$t2" -exec mv {} /sata \;
        t2=$(awk 'BEGIN{print '$t' - 60 }')
fi

done


   

651260546

回复 4# lrz
我那个查找删除的时间要按条件一直去减的