求助 ~~~ shell脚本 函数调用的问题

ppiqq

各位大大~~~~大家辛苦了 小弟发一个脚本

………………………………………………………………
function PING_STATUS ()
{  local _ipadd
      _ipadd=$1
    PING_COUNT=`ping -c 4 $1`
    if [ $? -eq 0 ];then
#       return 0
         echo "ping is ok"          #ping success
        elif [ $? -ne 0 ];then
#        return 1
          echo "ping is error"  #ping error
    fi
}

PING_STATUS $IP
…………………………………………………………………………
IP变量是一样的，这个程序执行后的结果是 ping is error


这样的话 得到的结果是对的

但是 我要是这样的话~~~~


function PING_STATUS ()
{  local _ipadd
      _ipadd=$1
    PING_COUNT=`ping -c 4 $1`
    if [ $? -eq 0 ];then
       return 0
#         echo "ping is ok"          #ping success
        elif [ $? -ne 0 ];then
        return 1
#          echo "ping is error"  #ping error
    fi
}

#PING_STATUS $IP

if [ `PING_STATUS $IP`==0 ];then
   echo "ping is ok"
  else
    echo "ping is error"
fi

………………………………………………………………………………

得到的结果却是不一样的
第一问 已经解决




第二问如下：
#!/bin/sh
function abc()
{
  if [ $1 -eq 1 ];then
   return 222
  else
     return 333
  fi
  }
if [[ `abc 2` = "333" ]];then
   echo "test is success"
fi

那这样的应该怎样实现呢？？？？？

也就是说我能不能指定我定义的函数返回一个特定值，然后根据返回值来判断操作

[ 本帖最后由 ppiqq 于 2008-12-18 11:46 编辑 ]

haimming

abc 2
if [[ $? -eq 333 ]];then
   echo "test is success"
fi

为什么一定要放在[] 里面呢？
我在之前说的那样不好吗？

调用完立马用的话，$?就是返回值；
如果要用不止一次，或不是马上用，用个东东记住它吧
PING_STATUS $IP
return_v=$?

[ 本帖最后由 haimming 于 2008-12-18 11:53 编辑 ]

ly5066113

if [ `PING_STATUS $IP`==0 ];

1、=号两边要有空格，而且只用=就可以了，不用==
2、`` 捕获的标准输出，而不是命令的返回值，要得到命令的返回值可以用$?

function PING_STATUS ()
{  local _ipadd
      _ipadd=$1
    PING_COUNT=`ping -c 4 $1`
    if [ $? -eq 0 ];then
       return 0
#         echo "ping is ok"          #ping success
        elif [ $? -ne 0 ];then
        return 1
#          echo "ping is error"  #ping error
    fi
}

#PING_STATUS $IP

if PING_STATUS $IP ;then
   echo "ping is ok"
  else
    echo "ping is error"
fi

merlin852

绕个大圈子，还不如直接这样，

if ping -c 4 $IP ;
then
   echo "ping is ok"
  else
    echo "ping is error"
fi

ynchnluiti

原帖由 ppiqq 于 2008-12-18 10:15 发表
#PING_STATUS $IP

if [ `PING_STATUS $IP`==0 ];then
   echo "ping is ok"
  else
    echo "ping is error"
fi

echo `PING_STATUS $IP`看看结果

1. 
   
2. PING_STATUS $IP
   
3. if [  $? -eq 0 ];then
   
4.    echo "ping is ok"
   
5.   else
   
6.     echo "ping is error"
   
7. fi
   

复制代码

haimming

if [ `PING_STATUS $IP` = 0 ]；
用这种方法判断不行

1. 
   
2. abc(){
   
3. if [[ $1 = 1 ]]
   
4. then
   
5.    return 22
   
6. else
   
7.     return 33
   
8. fi
   
9. }
   
10. 
    
11. abc 1
    
12. echo $?
    
13. abc 2
    
14. echo $?
    
15. 
    
16. if [[ `abc 1` = "22" ]]
    
17. then
    
18. echo 22
    
19. else
    
20. echo 33
    
21. fi
    
22. 
    
23. echo `abc 1`
    
24. 
    
25. 
    
26. abc 1
    
27. a=$?
    
28. if [[ $a = 22 ]]
    
29. then
    
30. echo 22
    
31. else
    
32. echo 33
    
33. fi
    

复制代码

[rockfall@localhost mytest]$ ./kkkkkkkk
22
33
33

22
[rockfall@localhost mytest]$

[ 本帖最后由 haimming 于 2008-12-18 11:34 编辑 ]

yezhulin1982

if [ `PING_STATUS $IP` = 0 ]

整数比较使用-eq

我是DBA

同意楼上的观点

haimming

if [ `PING_STATUS $IP` = 0 ]
这用用，直接给你报错
line 18: [: -eq: unary operator expected
因为是空的，echo `PING_STATUS $IP` 就知道，`PING_STATUS $IP`并不是返回值

ppiqq

原帖由 ly5066113 于 2008-12-18 10:20 发表
if [ `PING_STATUS $IP`==0 ];

1、=号两边要有空格，而且只用=就可以了，不用==
2、`` 捕获的标准输出，而不是命令的返回值，要得到命令的返回值可以用$?

function PING_STATUS ()
{  local _ipadd
   ...

谢谢您了 大哥

1.== 还是有点儿C语言情缘
2. 谢谢大哥