Проверьте скрипт пожалуйста.

[Axa-Ru]

И я и arcfi вам написали: заключите в кавычки переменные в выражении if [ "$a" = "$b" ]

[izamoo]

И я и arcfi вам написали: заключите в кавычки переменные в выражении if [ "$a" = "$b" ]

так вообще не работает

вот скрипт, проверте:

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

#!/bin/bash
down="/home/vlad/Загрузки"
pic="/home/vlad/Картинки"
cd $down
files=`ls $down`
jpg="*.jpg"
bmp="*.bmp"
for j in $files;  do
if [ "$j" = "$jpg" ]; then mv $j $pic; echo "Файл $j перемещен"
fi
done
for b in $files; do
if [ "$b" = "$bmp" ]; then mv $b $pic; echo "Файл $b перемещен"
fi
done


[Axa-Ru]

Что не работает?
Ошибку скрипт не выдает,
Но работать и не должен. Потому как я вам написал выше, что в сравнении wildcard "*...." работать не будет.
Используйте функцию работы со строками http://www.thegeekstuff.com/2010/07/bash-string-manipulation/

[izamoo]

Что не работает?
Ошибку скрипт не выдает,
Но работать и не должен. Потому как я вам написал выше, что в сравнении wildcard "*...." работать не будет.
Используйте функцию работы со строками http://www.thegeekstuff.com/2010/07/bash-string-manipulation/

ок, будем читать. Но это же выражение

Код: [Выделить]

[ $j = $jpg ]сравнивает строки.

[Axa-Ru]

Код: [Выделить]

#!/bin/bash

str1="abcdefg"
str2="*fg"

if [ $str1 = $str2 ]; then
  echo "строки равны"
else
  echo "строки НЕ равны"
fi

Запускаем:

Код: [Выделить]

$ ./sts2.sh
строки НЕ равны
$
Где и что работает?

Да, еще. Пока вы  изучаете все это дело, пишите один оператор/выражение в одной строке. Так проще отлаживать.

[izamoo]

Код: [Выделить]

#!/bin/bash

str1="abcdefg"
str2="*fg"

if [ $str1 = $str2 ]; then
  echo "строки равны"
else
  echo "строки НЕ равны"
fi

Запускаем:

Код: [Выделить]

$ ./sts2.sh
строки НЕ равны
$
Где и что работает?

Да, еще. Пока вы  изучаете все это дело, пишите один оператор/выражение в одной строке. Так проще отлаживать.

Спасибо буду разбиратся.

[izamoo]

Переделал скрипт, теперь вроде всё работает, я думаю можно маленько доработать и использовать данный скрипт для сортировки файлов по каталогам.

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

#!/bin/bash
down=/home/vlad/Загрузки
pic=/home/vlad/Картинки
files_jpg=`ls $down | grep .jpg`
cd $down

if [ -n "$files_jpg" ]; then
for jpg in $files_jpg; do
mv $jpg $pic; echo "Файл $jpg перемещён"
done

else echo "В каталоге $down нет файлов jpg"
fi


Пользователь решил продолжить мысль 27 Сентября 2011, 12:35:48:помоему строка

Код: [Выделить]

cd $down
здесь никчему.

[Axa-Ru]

Красиво. Молодец.
Маленькое замечание. Строковые переменные (в команде mv тоже) заключайте в кавычки.
Иначе, если попадется имя файла с пробелом возникнет ошибка.

Можно я предложу такой вариант? Он почти похож на ваш, но чуть покороче.

Код: [Выделить]

#!/bin/bash

down=/home/vlad/Загрузки
pic=/home/vlad/Картинки

for jpg in ${down}/*.jpg ; do
  mv "$jpg" "$pic"
  echo "mv $jpg $pic"
done


[izamoo]

спасибо за замечание.

[lukinfore]

Смысл в непосредственно написании ? А то кеп сказал что "mv pic/*.jpg ../*picture /and\ smth.bmp target/dir" баш, да и д'аш вроде, раскрывает без всяких файндов. И есть --verbose для желающих знать, что там переместилось.

[izamoo]

Смысл в непосредственно написании ? А то кеп сказал что "mv pic/*.jpg ../*picture /and\ smth.bmp target/dir" баш, да и д'аш вроде, раскрывает без всяких файндов. И есть --verbose для желающих знать, что там переместилось.

Да смысл в написании и тренировке.

[lukinfore]

Я к тому, что

можно маленько доработать и использовать данный

намекает, что смыслы кроме написания таки есть

[izamoo]

Я к тому, что

можно маленько доработать и использовать данный

намекает, что смыслы кроме написания таки есть

Основной смысл это тренировка написания скриптов, также можно приспособить для сортировки файлов по папкам. Так же я в курсе что подобных скриптов (сортировки файлов по папкам) написано уже море, но.......как я уже писал основной смысл это треннировка и получение опыта написания скриптов.

[censor]

нельзя не заметить что прежде чем писать скрипты стоит ознакомиться с инструментами системы для которой пишите скрипт, зная инструменты можно добиться максимальной производительности вместо того чтобы топором строгать зубочистки.

[izamoo]

censor  молодец 5, я же написал, я учусь. Вы наверное сразу как родились и сразу начали ваять. ну вы же гений. Спасибо.