История изменений
Исправление Desmond_Hume, (текущая версия) :
Нет никакого смысла в конструкциях mv для переименования && mv для перемещения и iconv && mv. Надо сразу помещать файл в нужную директорию.
Смысла, возможно, и нет, но с переменными что-то не получалось, поэтому разбил задачу на подзадачи. Попробую сократить, если удастся.
В данном случае нет никакого смысла в пустых ветвях if. Их можно безболезненно удалить.
Имеется вввиду else c : - это сделал для непредвиденных случаев. Например, кодировка файла совсем другая или он не подходит по типу, к примеру, вдруг по ошибке попадёт видеофайл. В общем, если что-то пойдёт не по сценарию сделал вариант «ничего не делать».
Нет никакой нужды трижды вызывать file. Перепишите filetype_is не как функцию, а как переменную сохраняющую результат выполнения file. Будет то же самое, но быстрее и эффективнее.
Сделал так исключительно из-за того, что был нехороший опыт с переменными, где был большой конвейер последующих команд. Эмпирически пришёл к выводу, что в таких случаях лучше пользоваться функциями. Попробую с переменной. Ещё смущает то, что переменная это «то, что не меняется и статически держится в ОЗУ», а функция это то, что динамически меняется. Не лучше ли всё-таки оставить функцию, раз file будет меняться при прохождении цикла по файлам и изменении его результата?
Исходная версия Desmond_Hume, :
Нет никакого смысла в конструкциях mv для переименования && mv для перемещения и iconv && mv. Надо сразу помещать файл в нужную директорию.
Смысла, возможно, и нет, но с переменными что-то не получалось, поэтому разбил задачу на подзадачи. Попробую сократить, если удастся.
В данном случае нет никакого смысла в пустых ветвях if. Их можно безболезненно удалить.
Имеется вввиду else c : - это сделал для непредвиденных случаев. Например, кодировка файла совсем другая или он не подходит по типу, к примеру, вдруг по ошибке попадёт видеофайл. В общем, если что-то пойдёт не по сценарию сделал вариант «ничего не делать».
Нет никакой нужды трижды вызывать file. Перепишите filetype_is не как функцию, а как переменную сохраняющую результат выполнения file. Будет то же самое, но быстрее и эффективнее.
Сделал так исключительно из-за того, что был нехороший опыт с переменными, где был большой конвейер последующих команд. Эмпирически пришёл к выводу, что в таких случаях лучше пользоваться функциями. Попробую с переменной.