Показать сообщение отдельно
  #4  
Старый 10.02.2019, 10:38
crazyman crazyman вне форума
Новичок
 
Регистрация: 22.04.2012
Сообщения: 99
Репутация: 10
По умолчанию

Цитата:
Сообщение от lmikle
Кхм... ну у тебя вообще предствлвенный код не код, а багодром.
И это не смотря на то, что имена переменных для потоков попутаны (ну тут подозреваю последствия вырезания кода для поста).
А по сути:
1. Не надо повторно использовать объекты. Создавай новые, это не так дорого стоить на современных системах.
2. Try..Finally у тебя является частью цикла, так лучше не делать.
3. Непонятен смысл испльзования промежуточного jpeg, а что, сразу в TFileStream религия не позволяет сохранять?
4. И вообще, такие вещи лучше переписать на многопоточность, не на модемах давно уже сидим, можно качать потоков так в 10, будет качаться раза в 3 быстрее, чем в один поток (да, кода будет чуть поболее, зато быстрее будет).
ну да, очень не аккуратный код.
это была в общем-то разовая акция, нужно было скачать книжку, которая
была картинками представлена, качальщиком сайтов оч. долго, поэтому решил накидать код.
Ответить с цитированием