Страница 1 из 2

Изменение плеера vk.com

Добавлено: 10 ноя 2012, 21:48
mephius
У вконтакте поменялся путь видео, из-за этого перестали работать некоторые сервисы (hdserials, hdcinema и т.п.).
Заставить все работать можно поменяв в tools/video.php в методе Video::getVideoVkontakte(); строчку:

Код: Выделить всё

$videos["MP4 $res"] = $host . "u$uid/video/$vtag.$res.mp4";
на

Код: Выделить всё

$videos["MP4 $res"] = $host . "u$uid/videos/$vtag.$res.mp4";
Т.е. там только video на videos поменялось.

Re: Изменение плеера vk.com

Добавлено: 10 ноя 2012, 21:51
agent_wowa
обновись, уже всё исправленно.

Re: Изменение плеера vk.com

Добавлено: 10 ноя 2012, 22:03
mephius
Поздно, я уже давно сам. Вас ждать долго было.

Re: Изменение плеера vk.com

Добавлено: 10 ноя 2012, 22:25
mephius
Кстати, не сочтите за наезд, но код у вас -- это ай-яй-яй. Чего стоит только использование глобалсов и жуткая лапша в шаблонах. Парсинг html-а тоже волшебен. Подробнее не смотрел.
А сервис сам по себе неплох.
Еще раз, не сочтите за наезд, просто там есть над чем поработать.

Re: Изменение плеера vk.com

Добавлено: 10 ноя 2012, 22:32
speedy
Можешь лучше - сделай. Всякий программер, читая чужой код считает, что лучше перписать с нуля. У всякого свой стиль программирования - можешь лучше - сделай!

Re: Изменение плеера vk.com

Добавлено: 10 ноя 2012, 22:44
mephius
Я ж именно поэтому и написал, что без наезда и указал конкретные причины, почему код, хм, не очень. И дело там вовсе не в стиле программинга.
Переписывание с нуля без острейшей необходимости -- зло. Постепенный рефакторинг при работе над кодом с целью повышения производительности, читабельности и аккуратности -- это гораздо правильнее и логичнее.
Мне писать с нуля тот же самый сервис нет ни малейшего смысла, сами понимаете.
А вот результатами моей работы вы почти наверняка пользовались, поэтому дело вовсе не в тщеславии или желании кго-то погнобить =)

Re: Изменение плеера vk.com

Добавлено: 11 ноя 2012, 01:47
consros
Давай, расскажи нам, как убрав глобалсы мы повысим производительность. Всем будет очень интересно.

Заодно не сочти за наезд, огласи сколько тебе лет, сколько опыт работы в целом и сколько с PHP.
Без обид.

Re: Изменение плеера vk.com

Добавлено: 11 ноя 2012, 12:30
mephius
:)
1. Убрав глобалсы вы уменьшите хрупкость своего кода и повысите читаемость. По производительности можно довольно сильно разогнать, например, ваш парсинг html, и это имело бы значительный смысл, если бы было больше одного пользователя
2. 32 года
3. опыта работы больше 10 лет, и, как я уже говорил, почти наверняка вы пользовались результатами моей работы

Я понимаю, что критика очень редко воспринимается конструктивно. Все это пишу я ровно для того, чтобы вы, позлившись и попостив яду в этот типик, подумали о том, как сделать ваш код лучше.

Re: Изменение плеера vk.com

Добавлено: 11 ноя 2012, 12:35
Мак-Гуру
mephius на форуме достаточно тем, где пользователи просят включить какой либо ресурс в сервис.
Попробуйте сами вписать любой ресурс и дать ссылку на его инсталл, если это поддержит руководство.
Получится проект от вас, а пользователи посмотрят на разницу.

Re: Изменение плеера vk.com

Добавлено: 11 ноя 2012, 16:07
consros
OK. Принимается. Просто со вчерашним студентом мы бы друг друга не поняли, в разговоре не было бы смысла.
Я нумерацию изменю.

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

В остальном общий уровень инкапсуляции вполне достаточный. Нужно хорошо понимать, что "не применять goto" и прочие привычные вещи - это рекомендации. И если на обслуживание рекомендаций тратятся излишние силы, только ради того, чтоб всё было "правильно" - то стоит сильно подумать, нужны ли они конкретно в этом месте конкретно этого проекта. У меня получилось - что не нужны.

В конечном итоге всегда создаётся некий компромис. Всегда приходится решать, где делать переход от шаблонов программирования и абстракций к конкретной предметной области. Мнения двух людей совпадут только в очевидных случаях. Это и ни что иное послужило причиной моего выхода из проекта TVonTop. Там Stalker тоже слишком тяготел к книжности. В результате конечный код был медленнее и гораздо сложнее.

Так что обсуждение тут может быть ровно на одном уровне. Ты берёшь конкретный участок и говоришь как, ты считаешь, его следует улучшить/переделать. Не забывая аргументировать. Иначе это - пустая болтовня.

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

3. Приведи примеры результатов твоей работы, которыми мы возможно пользовались? Это интересно.

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