Shell嵌套逻辑表达式实现:数据库状态检查单行命令求助
单行命令替代数据库状态检查脚本的验证与优化
首先咱们先明确你原来脚本的核心逻辑:
- 检查日志文件
serverlog.log中是否存在Current state: Booted,如果不存在就直接退出 - 接着检查日志中是否存在
Status: OK,如果不存在也直接退出 - 只有两个条件都满足时,才输出
0
先把你原来的脚本贴出来方便对照:
# 检查数据库是否在线;若不在线则退出 isbooted=`grep 'Current state: Booted' serverlog.log | wc -l` if [ $isbooted -eq 0 ] then exit fi # 检查数据库是否崩溃;若崩溃则退出 iscrashed=`grep 'Status: OK' serverlog.log | wc -l` if [ $iscrashed -eq 0 ] then exit fi echo 0
你自己写的这条命令逻辑是反的哦:
[ $(grep 'Current state: Booted' serverlog.log | wc -l) -eq 0 ] \ && [ $(grep 'Status: OK' serverlog.log | wc -l) -eq 0 ] \ && echo 0
它的意思是当两个关键词都找不到的时候才输出0,和你原本的需求完全相反,而且只要有一个关键词存在,这条命令就什么都不做也不退出,不符合要求。
正确的单行写法
我们可以利用grep的静默模式-q来简化逻辑(-q会让grep找到匹配就立刻返回0,找不到返回非0,比用wc -l统计行数更高效),有两种简洁的写法:
写法一:用&&串联成功条件,||处理退出
grep -q 'Current state: Booted' serverlog.log && grep -q 'Status: OK' serverlog.log && echo 0 || exit
逻辑解释:
- 先检查是否存在
Current state: Booted,找不到就直接执行exit - 找到的话继续检查
Status: OK,找不到也执行exit - 两个都找到就输出
0
写法二:用分号分隔独立判断,更直观
grep -q 'Current state: Booted' serverlog.log || exit; grep -q 'Status: OK' serverlog.log || exit; echo 0
逻辑解释:
- 第一个判断:找不到
Current state: Booted就退出,否则继续 - 第二个判断:找不到
Status: OK就退出,否则继续 - 最后输出
0
额外优化点
- 用
grep -q替代grep | wc -l:避免不必要的输出和行数统计,提升执行效率 - 不需要把结果存入变量,直接利用命令的返回值做判断,让代码更简洁
内容的提问来源于stack exchange,提问作者Srini V
相关产品推荐
相关产品推荐

