Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавить проверку размера файла после скачивания #16

Open
sh2ka opened this issue Mar 20, 2016 · 2 comments

Comments

@sh2ka
Copy link

sh2ka commented Mar 20, 2016

Я обнаружил, что некоторые записи оказываются обрезанными по какой-то причине, возможно, из-за какой-то ошибки скачивания. И повторный запуск vmd.pl ничего не даёт - пишет, что файлы уже скачаны. Удаляю эти файлы, запускаю vmd.pl - скачиваются эти файлы, но уже нормального размера. В общем можно было бы добавить проверку размера файла после скачивания и, если размер отличается, то хотя бы удалять этот кусок, чтобы при следующем запуске была попытка скачать его заново. Хоть это и не критично, но всё равно не удобно.

@FedyuninV
Copy link

Еще один, возможно, полезный вариант: дополнительная проверка перед скачиванием. Т.е., если файл существует, но другого размера или еще по каким параметрам не подходит - то удаляем тот, что был на диске.

@sh2ka
Copy link
Author

sh2ka commented Aug 17, 2016

Я не стал такой вариант предлагать, потому что часто конвертирую кодировку тегов в UTF-8 и при этом возможно размер файла изменяется - врать не буду - не проверял. Однако, сам файл - его данные - в любом случае меняются и это будет уже другой файл. Сейчас у меня куча файлов, которые по той или иной причине недокачались или испортились при скачивании (появились артефакты в звуке). Приходится удалять такой файл вручную, скачивать, и проверять проигрыванием в плейере. По идее простая проверка размера файла после скачивания решила бы проблему.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants