Достойно ли приложение по качеству кода?

плеер на python? давно пора!

в целом, нужно. только бы не vlc

Twitter Author

ты автор твиттера? крууть

Если действительно не сарказм, то в планах на ближайший месяц, на gstreamer перейти. По поводу twitter, да, затупил с написанием ссылки ))

почему регэксп на регэкспе, а не апи? https://developers.google.com/youtube/v3/code_samples/ есть грабли какие-то?

апи с огр. реквестами, дальше платно...

ясно. коллеге подкину, он кроме тытрубы ничего не смотрит и не слушает.

да, буду даже рад)

What the hell is that?

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

Меня пугает хардкод. А вообще - круто. Поставил, взлетело, работает.

Ну, тут пока решения у меня нет, без хардкода, именно для данных нужд, но попробую что то придумать.
Отлично, это радует)

Завтра же поставлю

на gstreamer перейти

Почему бы просто не дёргать FFmpeg?

Может быть, но планируется расширение функционала воспроизведения аудио, по типу эквалайзера, например.

Значит, донат в битках ты осилил, а упомянуть помогавших тебе регистрантов, хотя бы в About?

Мне нравится.

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

Сразу скажу что на питоне с Qt никогда не писал, но возможно стоит обратить внимание на следующие заметки:

• Виджетам в main.ui стоит давать осмысленное название, а не «pushButton_2» или «label_2» (кажется, ты уже поправил «pushButton_2», но не перезалил main.ui: в main.py указано main2.ui).
• Неплохо бы разбавить код комментариями (например в encryption.py), комментарий же # arr button tracks напротив не вносит ясности.
• Что заlabel_2.setText('<img src="./img/volume.png" />'), неужели Qt не может отобразить картинку без HTML?
• Класс YAudio нарушает Single Responsibility Principle, вынеси из него класс плейлиста, например.
• В yaudio.py методы должны следовать единому стилю именования: updateProgress => update_progress, volumeChanged=>volume_changed.
• Класс Search напрямую обращается к объектам(ui.search_btn) из другого потока, замени на взаимодействие через сигналы.

Что заlabel_2.setText('<img src="./img/volume.png" />'), неужели Qt не может отобразить картинку без HTML?

Может. Там должен быть setPixmap.

Понял, учту и переделаю согласно всем стандартам

Питон и качество кода — понятия крайне ортогональные!

а что setup.py делает в src?

Затупил после последнего билда, поправлю

по качеству кода?

Едва-едва троечку. За константы внутри кода оторвать руки. Модульности ноль, всё связано со всем. Сплошные god class которые делают всё и ещё немножко сверху.

Общее впечатление - поделка студента на Visual Basic, вид сбоку.

Но ты не расстраивайся, 99% кода на питоне выглядит также.

тестирую под оффтопом.

>python src/yaudio.py
Traceback (most recent call last):
  File "src/yaudio.py", line 14, in <module>
    import helpers.search
  File "C:\dev\others\yaudio\src\helpers\search.py", line 3, in <module>
    import lxml.html
  File "C:\python\lib\site-packages\lxml\html\__init__.py", line 54, in <module>
    from .. import etree
ImportError: DLL load failed: Не найдена указанная процедура.

На форточках вечная проблема с lxml, погугли как Ее решить, я собирал на win

На вб никогда в жизни не писал, только на пыхе, ноде и луа. А так критику понял, буду подгонять все под стандарты общепринятые. Спасибо

В ффмпег есть эквалайзеры и дофига чего ещё.

Поставил звёздочку. Буду присылать иногда пулл-реквесты, если не против. :)

Только за, за этим развитие!
По поводу ffmpeg, прислушался.

Не вышло так и поставить( у меня arch и руки не из того места растут

Странно, на арче я его и писал, кинь логи

Было бы идеально если бы приложении было в aur

Скоро будет

Вернулся с отдыха, не совсем понял, по поводу вынесения плейлиста, тыкни пожалуйста, что именно и как правильнее вынести.

Точно так же, как ты вынес playback в класс Play, какие конкретно у плейлиста будут методы тебе виднее. Я бы убрал quene_tracks из класса YAudio и создал такой класс Playlist

Playlist:
add_track(track) #Добавить трек в список воспроизведения
insert_track(index, track) #Добавить трек в список воспроизведения на позицию
remove_track(index) #Удалить трек из списка воспроизведения
skip_next() #Перейти к следующему треку
skip_prev() #Перейти к предыдущему треку
current_track() -> track #Получить текущий трек
signal track_changed(newTrack, oldTrack) #Поменялся текущий трек

Понял, спасибо, займусь сегодня