新手写了一个脚本 求高手拍砖

pandorabag

1. # !/bin/sh
   
2. day=`date +%Y-%m-%d`
   
3. lastday=`date -d "-1 day" +%Y-%m-%d`
   
4. hour=`date -d "-1 hour" +%H`
   
5. if [ $1 ]; then
   
6. day=$1
   
7. fi
   
8. if [ $2 ];then
   
9. hour=$2
   
10. fi
    
11. if [ $3 ];then
    
12. lastday=$3
    
13. fi
    
14. ###############declare#############
    
15. today=$day"-"$hour
    
16. last=$lastday"-"$hour
    
17. ###############func################
    
18. check ()
    
19. {
    
20. 
    
21. if [ $1 ];then
    
22. vt=$1
    
23. fi
    
24. if [ $2 ];then
    
25. user=$2
    
26. fi
    
27. 
    
28. database='/usr/local/mysql/bin/mysql  -uroot -pabdd  market -e'
    
29. text=`$database "select  a,b from  down where datestat ='$day' and hourstat='$hour' and product='$vt' ;"`
    
30. com=`echo $text |awk '{printf"%d",$3}' `
    
31. tdown=`echo $text |awk '{printf"%d",$4}' `
    
32. if (( `expr $com \< 130`)) && ((`expr $com \> 70` )) ;then
    
33. echo $1"ok  " $com
    
34. else
    
35. lasttext=`$database "select  a from  down where datestat ='$lastday' and hourstat='$hour' and product='$vt'; "`
    
36. ldown=`echo $lasttext |awk '{printf"%d",$2}' `
    
37. echo -e "\n\n"$1"  error 异常: \n\n\ndate\tdowns\n\n$day\t$tdown\n\n$lastday\t$ldown\n\n同比\t$com%\n" |/usr/local/bin/mutt  -s  $vt"下载量异常"  $user
    
38. fi
    
39. }
    
40. check a   xx@qq.com
    
41. check b   aa@126.com
    
42. check c   dd@123.com

复制代码

写了这段代码，求高手们拍砖

代码实现的左右就是从数据库表查询一个结果  并比较

高手们看看那里写的复杂了 或者难看

第一次自己写code  想多注意些格式

欢迎批评指正

Shell_HAT

要不要对用户输入的$1、$2、$3做个合法性检查？

zxpxyz

简单 实用 但是不好用。

建议楼主展开联想，后面增加一个循环，读取配置文件。

pandorabag

简单 实用 但是不好用。

建议楼主展开联想，后面增加一个循环，读取配置文件。
zxpxyz 发表于 2010-12-23 10:18

    多谢指点

nhw_cs

给你来点高级用法:

if [ $1 ]; then

day=$1

fi

if [ $2 ];then

hour=$2

fi

if [ $3 ];then

lastday=$3

fi

======>>


for i in day hour lastday; do

[ $# -gt 0 ] || break
eval $i=$1
shift

done

pandorabag

给你来点高级用法:

if [ $1 ]; then

day=$1

fi

if [ $2 ];then

hour=$2

fi

if [ $3 ] ...
nhw_cs 发表于 2010-12-23 11:55

    这位大侠能解释你下你的code吗？ 看不大明白

xiaopan3322

一个建议，注意缩进，风格有点。。。

pandorabag

一个建议，注意缩进，风格有点。。。
xiaopan3322 发表于 2010-12-24 09:43

    确实需要缩进。。 多谢