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

Add info about media_type #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xeniia-nikole
Copy link
Contributor

Added information that the received data can be not only an image

Added information that the received data can be not only an image
@neee
Copy link
Collaborator

neee commented Jun 19, 2021

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

@xeniia-nikole
Copy link
Contributor Author

Там может быть не только видео, но так же аудио)

@neee
Copy link
Collaborator

neee commented Jun 21, 2021

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

@xeniia-nikole
Copy link
Contributor Author

xeniia-nikole commented Jun 21, 2021

Проблема в том, что сейчас программа получает данные, но, если не выводить их в консоль, то невозможно понять, сработало ли вообще. Либо же необходимо прописывать программу и для видео, и для аудио, и для изображения, что сложнее.
Вместо этого можно предложить проверить тип получаемого файла и если это не изображение выводить в консоль уведомление об этом :
что-то вроде "Вы успешно получили данные, однако медиа файл не является изображением и не может быть сохранен в рамках текущей программы. "

Можно так же оставить возможность создать отдельные методы для видео, и аудио, но на усмотрение студента)

@neee
Copy link
Collaborator

neee commented Jun 21, 2021

А можешь показать свой код или написать мне в слаке? Насклько я понимаю мы скачиваем файл с удаленного сервера, этот файл по факту массив байт, нам нужно сохранить этот массив байт с нужным разрешением файла, разрешение файла указывется в имени файла или я что-то упускаю?

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

Successfully merging this pull request may close these issues.

2 participants