shell脚本简化

cbb1133

cat ip.txt
root 192.168.1.1 22 root123
abc 192.168.1.2 23 abc123
xyz 192.168.1.3 24 xyz123

下面这个脚本有没有办法写的更加精简一点
cat expect.sh

1. #!/bin/bash
   
2. line=`cat ip.txt|awk '{print NR}'|tail -1`
   
3. cmd="puppet agent --test --server server.domain.com"
   
4. for ((NR=1;NR<=$line;NR++));do
   
5.         host=`cat ip.txt|awk 'NR=='$NR' {print $2}'`
   
6.         password=`cat ip.txt|awk 'NR=='$NR' {print $4}'`
   
7.         port=`cat ip.txt|awk 'NR=='$NR' {print $3}'`
   
8.         user=`cat ip.txt|awk 'NR=='$NR' {print $1}'`
   
9.         expect -c "
   
10.         spawn ssh -l$user $host -p$port \"$cmd\"
    
11.         expect {
    
12.             \"yes/no\" { send \"yes\n\";exp_continue;}
    
13.             \"assword:\" { send \"$password\r\"; }
    
14.         }
    
15.         expect eof
    
16.         "
    
17. done

复制代码

关阴月飞

回复 1# cbb1133


    把for 改用while read 简单一些:

1. #!/bin/bash
   
2. cmd="puppet agent --test --server server.domain.com"
   
3. while read line
   
4. do
   
5.         host=`echo $line |awk '{print $2}'`
   
6.         password=`echo $line |awk '{print $4}'`
   
7.         port=`echo $line |awk '{print $3}'`
   
8.         user=`echo $line |awk '{print $1}'`
   
9.         expect -c "
   
10.         spawn ssh -l$user $host -p$port \"$cmd\"
    
11.         expect {
    
12.             \"yes/no\" { send \"yes\n\";exp_continue;}
    
13.             \"assword:\" { send \"$password\r\"; }
    
14.         }
    
15.         expect eof
    
16.         "
    
17. done < ip.txt

复制代码

tomgrean

可以不用执行那么多awk，整个expect -c 后的字符串可以用一个awk命令生成出来。

1. 
   
2. fullcmd=`echo $line|awk '{print "\nspawn ssh -l" $1, $2, "-p" $3, "'"$cmd"'";
   
3.         print "expect {";
   
4.         print "\"yes/no\" { send \"yes\n\";exp_continue;}";
   
5.         print "\"assword:\" { send \"" $4 "\r\"; }";
   
6.         print "}\nexpect eof"}'`
   
7. expect -c "$fullcmd"

复制代码

cbb1133

谢谢楼上各位哈
2L的方法不错
3L的方法虽然精简了代码，但是整个结构看上去感觉有点怪

关阴月飞

回复 4# cbb1133

四个变量的赋值可以写成一条命令, 只是看起来不那么直观.
        host=`echo $line |awk '{print $2}'`
        password=`echo $line |awk '{print $4}'`
        port=`echo $line |awk '{print $3}'`
        user=`echo $line |awk '{print $1}'`

1. eval  $(echo $line |awk '{print "user="$1,"host="$2,"port="$3,"password="$4}')

复制代码

cbb1133

回复 5# 关阴月飞

在我们平时写shell时
应该寻求代码简短
还是应该寻求直观点

   

关阴月飞

回复 6# cbb1133


    这个LZ可以开个主题来讨论了,  更多的时候简洁和直观并不冲突, 简洁就是为了更直观.  

FaintKnowledge

回复 1# cbb1133

1. #!/usr/bin/expect -f
   
2. set IP     [lindex $argv 1]
   
3. set PASSWD [lindex $argv 2]
   
4. set USER [lindex $argv 0]
   
5. set timeout -1
   
6. 
   
7. spawn ssh $USER@$IP
   
8. expect  "*yes/no*" { send "yes\r"; };
   
9. expect  "*assword*" { send "$PASSWD\r"; };
   
10. 
    
11. expect "*#" { send " puppet agent --test --server server.domain.com\r"}
    
12. expect "*#" { send "exit \r"}
    
13. expect eof

复制代码

1. while read line;do ./expect $line ;done < iplist

复制代码

personball

关阴月飞 发表于 2013-02-19 14:57
回复 6# cbb1133

我反倒觉得5楼的相当直观啊

追加：3楼的也直观