wapmorgan » 2016.03.25 21:57

Приветствую.

Прошу протестировать и оценить библиотеку
- composer: wapmorgsn/media-encoder
- или с гитхаба https://github.com/wapmorgan/MediaConverter и обновить зависимости - composer update

Проверьте этот код (нужно иметь видео файл filename.avi:

Код:

1
2
$conv = new wapmorgan\MediaConverter\MefiaConverter('filename.avi');
$conv->convertTo3gp('filename.3gp');
Gemorroj » 2016.03.25 22:30

какие люди)
сорян, но где PSR? В качестве прослойки над ffmpeg полюбому лучше https://github.com/PHP-FFMpeg/PHP-FFMpeg, если писать свою, то хотя бы использовать прослойку над консолью (https://github.com/symfony/process, например).

Код:

1
if (($bin = find_existing_file(['/usr/bin/ffmpeg', '/usr/local/bin/ffmpeg', '/bin/ffmpeg'])) !== false)

ненастраиваемый путь к ffmpeg - гарантированная проблема. На вапинете например, кастомная сборка и лежит не по стандартному пути.
Добавлено спустя    55 секунд:
и да, напиши тесты и используй для сборки и запуска тестов travis.

Nu3oN » 2016.03.26 10:58

Какой же я динозавр и для меня дико применения символа слеш в вызове класса. Это типа класс в классе?

wapmorgan\MediaConverter\MefiaConverter

Gemorroj » 2016.03.26 12:43

Nu3oN, эти типа неймспесы... Ты все еще на php 5.2 пишешь..?

wapmorgan » 2016.03.26 19:41

Gemorroj написал:

какие люди)
сорян, но где PSR? В качестве прослойки над ffmpeg полюбому лучше https://github.com/PHP-FFMpeg/PHP-FFMpeg, если писать свою, то хотя бы использовать прослойку над консолью (https://github.com/symfony/process, например).

Код:

1
if (($bin = find_existing_file(['/usr/bin/ffmpeg', '/usr/local/bin/ffmpeg', '/bin/ffmpeg'])) !== false)

ненастраиваемый путь к ffmpeg - гарантированная проблема. На вапинете например, кастомная сборка и лежит не по стандартному пути.
Добавлено спустя    55 секунд:
и да, напиши тесты и используй для сборки и запуска тестов travis.

1. Какой именно psr? По стилю кодирования? Ну, не все из того psr я соблюдаю.
2. process, это конечно хорошо (тем более у меня тоже есть - subprocess), но для начала можно и так.
3. Как бы не самонадеянно звучали мои слова, но в большинстве случаев ffmpeg будет установлен самый обычный через системный менеджер пакетов.
4. Тесты будут, возможно. Но пока у меня просто нет времени их написать. 2 часа в день с телефоном - это немного, на самом деле.

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

Gemorroj » 2016.03.26 23:09

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

Nu3oN » 2016.03.28 09:17

Gemorroj написал:

Nu3oN, эти типа неймспесы... Ты все еще на php 5.2 пишешь..?

вообще не пишу)