Не получается заставить работать скрипт с IF, в чем может быть причина?

Question

[maniac_by]

Всем привет. Получил задание, не сложное, но никогда не имел дел с программированием даже близко. Потому нужна помощь.

Нужно написать скрипт, который будет выдавать топ-10 файлов по величине из выбранной директории.

#!/bin/bash
dir=$1
dir2=$2

if [[ "$dir" = "$1" ]]; then find $2 -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
fi

if [[ "$dir" = '-h' ]]; then echo "help me"
fi

if [[ "$dir" = '' ]]; then echo "No args"
fi

Проблема в том, что когда я пишу, например, /opt , то скрипт ничего не выводит. Я понимаю, что дело в объявлении переменной. Но не могу сам дойти, что конкретно не так. Пожалуйста, помогите с простейшим примером. Спасибо

Comments

[Lynn «Кофеман»]

Меня смущает первый же if. Вы только что присвоили в $dir значение $1 и сравниваете с ним же

[maniac_by]

Алексей Тен, Да, вроде разобрался

#!/bin/bash
dir=$1
dir2=$dir


if [[ "$dir" = "$1" ]]; then find $dir2 -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
fi

if [[ "$dir" = '-h' ]]; then echo "help me"
fi

if [[ "$dir" = '' ]]; then echo "No args"
fi

Так работает

[maniac_by]

Алексей Тен, Напишите, пожалуйста, корректно как будет. Если не трудно. Мне надо , чтобы, грубо говоря, я написал /opt и в нем выполнился сценарий. Как подтянуть переменную из командной строки в сценарий я еще не знаю.

[maniac_by]

Алексей Тен, Пока жду ответа уже дошел до

#!/bin/bash
dir=$1
dir2=$2


if [[ -d  "$dir" ]]; then find $dir -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
        else echo "Не директория!"
fi

if [[ "$dir" = '-h' ]]; then echo "help me"
fi

if [[ "$dir" = '--help' ]]; then echo "help me"
fi

if [[ "$dir" = '' ]]; then echo "No args"
fi

[Roman Ratkin]

maniac_by, зачем вам лишние переменные? Достаточно же

#!/bin/bash

if [ -z "$1" ]
then
    echo "Введите директорию!"
    exit 1
fi

if [[ -d  "$1" ]]
then
    find "$1" -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
elif [[ "$1" == "-h" || "$1" == "--help" ]]
then
    echo "Help me!"
else
    echo "Не директория!"
fi

[vman]

#!/bin/bash

usage() {
	echo "usage $0 <dir>"
	exit 1
}

[ -z "$1" ] || [ "$1" == "-h" ] || [ ! -d "$1" ] && usage

find $1 -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'

[maniac_by]

Roman Ratkin, Как зачем? Потому что не умею) Спасибо за ответ)

Навыдумывал вот такую "хитрую" конструкцию:

#!/bin/bash
dir=$1
dir2=$2

if [[ -d  "$dir" ]]; then find $dir -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
fi

if [[ -f "$dir" ]]; then echo "It's not a directory!"
fi

if [[ -z "$dir" ]]; then echo "Enter directory name"
fi

if [[ "$dir" = '-h' ]]; then echo "help me"
fi

if [[ "$dir" = '--help' ]]; then echo "help me"
fi

[maniac_by]

Roman Ratkin, vman, Если не трудно.

В задании есть ещё пункт:

More than 1 argument reports error, returns proper error code and show help message

Ноу меня идей нет на эту проверку, подскажите?

[Roman Ratkin]

maniac_by, вкратце про аргументы. Необязательно присваивать аргумент $1 переменной в скрипте $dir. Можно везде использовать $1. В find "$dir" лучше взять в двойные кавычки, чтобы экранировать спецсимволы (в том числе пробелы). Либо, нужно указать -print0, как дополнительный параметр для find.

[Roman Ratkin]

maniac_by, что проверить сколько пришло аргументов используется переменная $#. В вашем случае что-то типа

if [ "$#" -gt "1" ]
then
    echo "Аргументов слишком много"
    exit 1
fi

[maniac_by]

Roman Ratkin, Большое спасибо за ответ. Проверяет, если в пути нет / . Например var проверяет, а /var уже нет. Но и на том спасибо)

#!/bin/bash
dir=$1

if [[ -d  "$dir" ]]; then find "$dir" -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
elif [[ "$#" -gt "1" ]]; then echo "More than one arguments, see -h for help"
fi

if [[ -f "$dir" ]]; then echo "It's not a directory!"
fi

if [[ -z "$dir" ]]; then echo "Enter directory name or -h , --help for help menu"
fi

if [[ "$dir" = '-h' ]]; then echo "help me"
elif [[ $dir == "--help" ]]; then echo "help me"
fi

[Roman Ratkin]

maniac_by, так, а если у вас пусть начинается не с корня, то он пишет, что это не директория. Или я не правильно вас понял..)

[Roman Ratkin]

maniac_by, а, я понял почему у вас не ругается если путь начинается не от корня. Посмотрите внимательно эту часть:

if [[ -d  "$1" ]]
then
    find "$1" -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
elif [[ "$1" == "-h" || "$1" == "--help" ]]
then
    echo "Help me!"
else
    echo "Не директория!"
fi

Если $1 директория, то выполняем код. Если нет, то проверяем указаны ли там аргументы -h/--help, если аргументов нет, то мы явно и указали не директорию, и не запросили помощь.

[maniac_by]

Roman Ratkin, Немножко меня не поняли, наверное) Если указать /var /var , к примеру, то код выполняется. После первого корня он не видит дальше аргумента. А должен был ругнуться на то, что их там два. Ну да ладно, и так сойдет. Все равно спасибо)

[Roman Ratkin]

maniac_by, поставьте проверку количества аргументов раньше остальных проверок. Чтобы если аргументов

if [ "$#" -gt "1" ]; then echo "To many arguments"; exit 1; fi

[maniac_by]

Roman Ratkin, Огромное спасибо! Идеально, с Вашей помощью малость разобрался в построении)

Если Вам не трудно, вынесите пару сообщений основных в отдельное сообщение, чтобы я мог пометить ваш ответ лучшим.

Ещё раз большое спасибо)

[Roman Ratkin]

maniac_by, не за что) Можете поиграть со сложными условиями. Так код будет более компактным. Вот неплохая статья по этой теме.

Answer 1

[Roman Ratkin]

#!/usr/bin/env bash

if [ "$#" -gt "1" ]
then
    echo "To many arguments"
    exit 1
fi

if [ -z "$1" ]
then
    echo "Введите директорию!"
    exit 1
fi

if [[ -d  "$1" ]]
then
    find "$1" -type f -ls 2> /dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
elif [[ "$1" == "-h" || "$1" == "--help" ]]
then
    echo "Help me!"
else
    echo "$1 не директория!"
fi

Answer 2

[Saboteur]

#!/bin/bash

if [[ -z "$1" -o "$1" == "-h" -o "$1" == "--help" ]]; then
  echo "provide directory name";
  exit 1
fi

if [[ -d "$1" ]]; then
  find "$dir" -type f -ls 2>/dev/null | sort -rnk7 | head -10 | awk '{printf "%10d MB\t%s\n",($7/1024)/1024,$NF}'
else
  echo "it is not a directory"
fi