Что не так с кодом?

Question

[Liuonid]

Было у меня такое ТЗ.

Необходимо разработать приложение, которое использует API YouTube и по запросу получает данные
о роликах.

Первый экран:
Строка для запроса и кнопка "искать".
После нажатия на кнопку "искать" необходимо:
- получить список из 20 (максимум) самых новых
видеороликов, которые соответствуют запросу.
- список должен быть отсортирован по популярности (количество просмотров) от большего к
меньшему
- каждый элемент списка содержит информацию: название ролика, автор, дата публикации
- при нажатии на названии ролика, открывается проигрыватель (эффект аккордеон)
- над списком должен быть текст: "результат поиска по запросу: {запрос}"

Ответ:

здесь

Что в этом может не понравится?
Знаю что не так с тегом label.
На чем стоит поработать?

Answer 1

[Северное Сияние]

На чем стоит поработать?

научиться писать красивый, опрятный код.

Сейчас это - говнокод. А наиважнейшая задача программиста писать код, понятный в первую очередь, ЧЕЛОВЕКУ.

Открой любой популярный opensource-фреймворк и посмотри, как элегантен там каждый файл кода.

Comments

[Liuonid]

писал на скорую руку. Да согласен нужны комменты и именование переменных не очень. Но код тут слишком маленький.

[Liuonid]

Я это скопипастил от гугла там так же.

[Liuonid]

$youtube->search->listSearch('snippet', array('maxResults' => 20, 'q' => $_GET['q'], 'type'=>'video', 'order'=>'date')); 'этот запрос получает 20 первых видео.
$youtube->videos->listVideos("snippet,statistics",
array('id' => $searchResult['id']['videoId'])); - это более детальное инфу о видео. там количестов просмотров лайков, дизлайков, комменты ...(инфа в гугле)

[Liuonid]

Хорошо критика на счет элегантенности кода принят.

[morricone85]

DarthWazer, ого:) это, какой фреймворк?

Answer 2

[Антон Шаманов]

отделить данные от их представления т.е. создать view, это как ты сам видишь и тебе мешает.

Comments

[Liuonid]

Сорри, я скопипастил с гугла. Там так же.