Пет-проект

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

Прикрутил:

• ползунок громкости в 1 клик, а не ПКМ-ЛКМ-прокрутку как в стандартном микшере
• настройку хоткея на почти любой (Shift+PgDown по умолчанию)
• безрамочность, чтобы alt-tab меню не загружал
• окей windows-дизайн

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

screenshot

github

P.S Про ООП можно не писать, в коде ужасы местами, модулей нет

Gentleleg сказал(а):↑

Подскажите, что можно изменить в лучшую сторону или добавить?

Нажмите, чтобы раскрыть...

Gentleleg сказал(а):↑

Про ООП можно не писать, в коде ужасы местами, модулей нет

Нажмите, чтобы раскрыть...

Ты же сам ответил на свой вопрос

Не пихать исходный код на гитхабе в зипфайл?

GinSilverSoul сказал(а):↑

Не пихать исходный код на гитхабе в зипфайл?

Нажмите, чтобы раскрыть...

Ссылка ведет на релизы, там код автоматически в зипе. А в корне репозитория не в зипе.

Gentleleg сказал(а):↑

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

Прикрутил:

• ползунок громкости в 1 клик, а не ПКМ-ЛКМ-прокрутку как в стандартном микшере
• настройку хоткея на почти любой (Shift+PgDown по умолчанию)
• безрамочность, чтобы alt-tab меню не загружал
• окей windows-дизайн

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

screenshot

github

P.S Про ООП можно не писать, в коде ужасы местами, модулей нет

Нажмите, чтобы раскрыть...

Для начала сделай так, чтобы тебе самому код казался идеальным, потом спрашивай что можно улучшить

74-82: происходит копипаста, подумай как исправить.

103-104: зачем повторяемся? если уже есть cancel_pressed?

В самом cancel_pressed и много где еще используется dynamic scoping. Dynamic scoping, IMHO, надо избегать, потому что ты переименуешь завтра w в window для читабельности, и тебе придётся переименовывать всюду это по коду, не говоря уже про тяжесть чтения такого кода, когда откуда ни возьмись появляются сущности: w, event и прочие, которые заданы даже не сверху файла, а где-то в низах в if __name__ == ...

В общем, глобальные переменные — плохо.

258: Почему оно static? Почему оно названо x?

К приложению можно например добавить поиск процесса, а лучше job’а.

Если собираешься в портфолио пихать этот проект, оформи readme получше. Добавь более подробное описание, картинки/гифки. Мб описание на RU/EN.
Код в проекте реализуй настолько чисто, насколько можешь на текущем твоем уровне. Постарайся построить красивую архитектуру и структуру проекта.

Gentleleg сказал(а):↑

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

Прикрутил:

• ползунок громкости в 1 клик, а не ПКМ-ЛКМ-прокрутку как в стандартном микшере
• настройку хоткея на почти любой (Shift+PgDown по умолчанию)
• безрамочность, чтобы alt-tab меню не загружал
• окей windows-дизайн

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

screenshot

github

P.S Про ООП можно не писать, в коде ужасы местами, модулей нет

Нажмите, чтобы раскрыть...

А зачем кнопка обновления рядом с 'chrome.exe'?

добавь .gitignore. Конфигурация твоей идеи(папка .idea) никому не нужна, поэтому ее не нужно пушить в remote репозиторий.
Код тупо в одном файле - это не читабельно, очень сомневаюсь, что кто-то будет тратить свое время на такое. Ты сам это понимаешь, поэтому первое что нужно сделать - оформить его нормально. Особенно, если собираешься вставлять ссылку на него в резюме

xzka сказал(а):↑

А зачем кнопка обновления рядом с 'chrome.exe'?

Нажмите, чтобы раскрыть...

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

Gentleleg сказал(а):↑

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

Нажмите, чтобы раскрыть...

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