Понятно ли будет использование класса для работы с API ВКонтакте?

Question

[Владислав Старцев]

Написал собственный класс для работы с API ВКонтакте, думаю писать о нём на хабре было бы слишком, решил для начала поделится здесь.

Репозиторий класса https://github.com/esvlad/VkQuery
Сам класс в папке Models/VkQuery.php

И собственно вопрос, понятно ли описание работы класса, и как сам, код класса?

Comments

[Антон Середний]

спасибо, мне понравилось, будем юзать

[Эдуард]

files_get_content, а так же цепочка if elseif elseif elseif elseif вероятно не самое лучшее решение (да простят меня разработчики backend)

[Владислав Старцев]

Эдуард: да мне тоже не очень нравится тот момент, можно использовать switch, только я и в нём смысла не вижу, лишь по красивше немного будет, а смысл тот же, и до добавления методов маркета, была лишь конструкция if else потому сделал elseif

[Эдуард]

Владислав Старцев: не думаю, что этот класс является продуктом который вы продаете, поэтому, его будут смотреть при трудоустройстве, а именно на такие моменты обратит внимание опытный разработчик и будет решать насчет вашей стоимости)

[Владислав Старцев]

Эдуард: как всё же хорошо, что опубликовал здесь, думаю всё же заменю на switch и понял что ещё одна конструкция лишняя))

[Эдуард]

Владислав Старцев: кстати по коду оставляют дельные комменты, с форматированием беда, но на самом деле большинство пишет вообще жесть так что норм) остальное уже зависит от того что требуется, красота или скорость.

[riot26]

читать "Чистый код" Мартина

[Владислав Старцев]

riot26: принял во внимание

[Евгений]

Владислав Старцев Скажите а метод для загрузки товаров есть ?

[Владислав Старцев]

Евгений: товары добавляются все теми же get запросами т.е. используется getData однако нужно помнить, что сначала нужно загрузить картинку, а потом добавлять товар, пример:

//Для картинки товара
$img = 'картинка_товара';
$group_id = 'ид_группы';
$params['main_photo'] = 1; //1 для обложки, 0 для доп. фотографии (лучше отдельным запросом)

$main_photo = $vk->uploadsImage($img, 'market', $group_id, $params);

//Для товара
$market_params = array(
'owner_id' => -$group_id,
'name' => 'название товара',
'description' => 'Описание товара'
'category_id' => 'ид категории',
'price' => 'цена товара',
'main_photo_id' => $main_photo->response[0]->id
);

$vk->getData('market.add', $market_params, true);
//Всё товар должен быть загружен

Answer 1

[Mikhail Osher]

Извините, но код ужасен.
Почитайте про PSR, паттерны, DRY/SOLID/KISS.

Comments

[Дмитрий]

Владислав Старцев: 1. Нет наймеспайсов. 2. Вместо file_get_contents curl. 3 Не обрабатываем исключения. 4. Composer. Про всякие PSR уже сказали Вот пример отличной библиотеки https://github.com/vladkens/VK

[Mikhail Osher]

Дмитрий: она тоже не отличная. :)

[Дмитрий]

Mikhail Osher: По крайне менее лучше для ВК я не видел

Answer 2

[Александр]

В принципе мне понравилось. Ман не плох, понятен. Думаю, сохраню себе ;-)

Answer 3

[Александр N++]

Передача параметров в конструктора отсуствует
1 В вашем случаи нужно постоянно менять, код в самом классе чтобы заменить конфиг
2 PSR4 отсутствует, композера нет. Так для сайтов на коленке норм.
3 Аля плюшкики типа загрузить много фоток, пост на стену, лучше вынести отдельно в другой класс. например назвать HelperVk

4 много лишних теле движений с if () {
return
}else{}
можно так

if( условие ) {
   return 
}
// тут код else но уже без скобок
return

5 В ошибках ошибки "добавленно" пишется с одной Н итд
6 Лучше если метод будет возвращать true/false

косяков много исправляйте

Comments

[Владислав Старцев]

спасибо за дельный ответ