Python-сообщество

• Заметки
  
• Таймер
  
• Мониторинг Monecraft серверов
  
• Часы
  
• Зашифрованая заметка
  
• Мониторинг процессора
  
• Мониторинг ОЗУ и подкачки
  

• Не всегда используются конструкторы. Например, для QLabel - я устанавливаю текст через setText.
  
• Возможно, было бы правильней во многих местах применить QGridLayout. Но мне больше нравится играть с горизонтальными и вертикальными.
  
• В некоторых местах названия переменных не информативны, а в некоторых могут вовсе запутать. Не считая мелочей, вроде коротких циклов, где содержание переменной очевидно.
  
• Есть подозрение, что можно было бы составить более эффективную архитектуру кода виджетов. Например, рисовать UI только непосредственно перед отображением, чтобы не добавленные на стол виджеты ели меньше ресурсов.
  

InterVi
мнение о коде

 def edit_mode(self, mode):
        try:
            for widget in self.widgets:
                if mode:
                    widget.setWindowFlags(Qt.WindowMinimizeButtonHint |
                                          Qt.WindowStaysOnBottomHint | Qt.Tool)
                    widget.show()
                else:
                    widget.setWindowFlags(Qt.CustomizeWindowHint |
                                          Qt.WindowStaysOnBottomHint | Qt.Tool)
                    widget.show()
            self.save_conf()
        except:
            print(traceback.format_exc())

> От вас хочется узнать мнение о коде, возможно я что-то делаю не так.

Я посмотрел на это чудо и вот мои первые впечатления:
1) Используй дизайнер.
2) Наследование диалогов от QWidget.
3) Перевод сообщений на иностранные языки не надо хранить как конф. файл. Используй gettext или стандартную систему перевода Qt.
4) Кнопки которые не могут быть использованы должны быть не активными.
5) Выджеты нельзя перемещать.
6)

         self.NAME = 'CPU Info'
        self.DESCRIPTION = self.lang['description']
        self.HELP = self.lang['help']
        self.AUTHOR = 'InterVi'
        self.EMAIL = 'intervionly@gmail.com'
        self.URL = 'https://github.com/InterVi/DeWidgets'

 def _show_error(names=()):
    mbox = QMessageBox(parent)
    mbox.setIcon(QMessageBox.Critical)
    mbox.setWindowIcon(QIcon(ERROR))
    mbox.setWindowTitle(lang['ADD_NEW']['error_title'])
    mbox.setText(lang['ADD_NEW']['error_text'])
    mbox.setStandardButtons(QMessageBox.Ok)
    ok = mbox.button(QMessageBox.Ok)
    ok.setText(lang['ADD_NEW']['error_ok_button'])
    ok.setToolTip(lang['ADD_NEW']['error_ok_button_tt'])
    if names:
        mbox.setInformativeText(get_str_list(names))
    mbox.exec()
def _show_success(names=()):
    mbox = QMessageBox(parent)
    mbox.setWindowIcon(QIcon(SUCCESS))
    mbox.setIcon(QMessageBox.Information)
    mbox.setWindowTitle(lang['ADD_NEW']['success_title'])
    mbox.setText(lang['ADD_NEW']['success_text'])
    mbox.setStandardButtons(QMessageBox.Ok)
    ok = mbox.button(QMessageBox.Ok)
    ok.setText(lang['ADD_NEW']['success_ok_button'])
    ok.setToolTip(lang['ADD_NEW']['success_ok_button_tt'])
    if names:
        mbox.setInformativeText(get_str_list(names))
    mbox.exec()

> Объясните смысл происходящего.

Это у него так логи работают

Да, это для логов, stdout и stderr пишутся в файл. Если не принтить, будет тишина и никакого дебага. Если не обрабатывать, упадёт всё приложение от мелкого бага в виджете.

Rodegast
1) Используй дизайнер.

Rodegast
2) Наследование диалогов от QWidget.

Rodegast
3) Перевод сообщений на иностранные языки не надо хранить как конф. файл. Используй gettext или стандартную систему перевода Qt.

Rodegast
4) Кнопки которые не могут быть использованы должны быть не активными.

Rodegast
5) Выджеты нельзя перемещать.

Rodegast
ИХМО эти атрибуты не должны быть частью объекта окна.

Rodegast
7) Повальное дублирование кода. Например вот это вполне можно было-бы объединить:

Rodegast
Ну и так далее…

> Да, это для логов, stdout и stderr пишутся в файл

Для логирования есть соответствующий модуль. И научись пользоваться декораторами.

> его код отвратителен

А тебе какая разница? Ты его в ручную править не будешь.

> А что в этом не так?

Диалоги нужно наследовать от QDialog по моему это очевидно.

> Вот про это можно по подробнее?

Нагугли gettext. В чём проблема?

> Как это нельзя? Можно - кнопочка в главном окне открывает меню.

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

> Он хранится в памяти для доступа из разных частей и эти атрибуты нужны. Где тогда их хранить?

http://qaru.site/questions/19712/what-is-the-common-header-format-of-python-files

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

Ты сначала текущие косяки исправь.

Отредактировано Rodegast (Янв. 13, 2018 13:21:27)

Rodegast
Для логирования есть соответствующий модуль.

Rodegast
И научись пользоваться декораторами.

Rodegast
А тебе какая разница? Ты его в ручную править не будешь.

Rodegast
Диалоги нужно наследовать от QDialog по моему это очевидно.

Rodegast
Нагугли gettext. В чём проблема?

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

Rodegast
http://qaru.site/questions/19712/what-is-the-common-header-format-of-python-files

Rodegast
Диалоги нужно наследовать от QDialog по моему это очевидно.

A dialog is always a top-level widget, but if it has a parent, its default location is centered on top of the parent. It will also share the parent's taskbar entry.

Отредактировано InterVi (Янв. 13, 2018 16:32:30)

> Зачем менять хороший код на плохой? Вот этого я совсем не понимаю.

Я у тебя “хорошего кода” ещё не видел.

> QMessageBox от QDialog, что за бред?

Где я писал про QMessageBox? Вот что имелось в виду:

 class Help(QWidget):
    """help window"""
    def __init__(self, lang):

         self.NAME = 'Digital Time'
        self.DESCRIPTION = self.lang['description']
        self.HELP = self.lang['help']
        self.AUTHOR = 'InterVi'
        self.EMAIL = 'intervionly@gmail.com'
        self.URL = 'https://github.com/InterVi/DeWidgets'

Rodegast
Где я писал про QMessageBox? Вот что имелось в виду:

Rodegast
Это технологический стандарт.

Rodegast
Я эту галочку включаю и ничего такого не происходит.

Rodegast
Например возьмём виджет digital_time. Каким образом будут использоваться вот эти атрибуты?