Форум программистов, компьютерный форум, киберфорум
Python: GUI, графика
Войти
Регистрация
Восстановить пароль
Карта форума Темы раздела Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/8: Рейтинг темы: голосов - 8, средняя оценка - 4.75
2 / 2 / 3
Регистрация: 19.09.2019
Сообщений: 103
1

Советы по оформлению кода

01.12.2019, 19:40. Показов 1484. Ответов 1

Author24 — интернет-сервис помощи студентам
Здравствуйте, у меня есть программа, она зашифровывает и расшифровывает шифр Цезаря(но это не очень важно).
Не могли бы вы посмотреть код и дать советы по его оформлению(например, если я использую какую-то функцию, которую не принято использовать, то скажите как это исправить).

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
from docx import *
from tkinter import *
from tkinter import filedialog as fd
from tkinter.ttk import Combobox
 
 
class Interface(Frame):                      #GUI класс
    def __init__(self, window):
        self.window = window
        Frame.__init__(self, window)
 
        self.window.title('Decoder')
        self.window.config(padx=3, bg='grey')
        self.window.geometry('900x600')
 
        self.menu_bar = Menu(self.window)
 
        self.file_menu = Menu(self.menu_bar, tearoff=0)
        self.file_menu.add_command(label='Open', command=self.open_file)
        self.file_menu.add_command(label='Save', command=self.save_file)
        self.file_menu.add_separator()
        self.file_menu.add_command(label='Exit', command=exit)
        self.menu_bar.add_cascade(label="File", menu=self.file_menu)
 
        self.window.config(menu=self.menu_bar)
 
        self.opt_lbl = Label(window, text='Options', font='Courier 14', bg='grey')
        self.opt_lbl.place(x=0, y=5)
 
        self.mode_lbl = Label(window, text='Mode', font='Courier 14', bg='grey')
        self.mode_lbl.place(x=0, y=35, width=50, height=30)
 
        self.mode = Combobox(window, font='Courier 14', width=10, values=['Decrypt', 'Encrypt'], state='readonly')
        self.mode.place(x=50, y=35, width=100, height=30)
 
        self.step_lbl = Label(window, text='Step', font='Courier 14', bg='grey')
        self.step_lbl.place(x=160, y=35, width=50, height=30)
 
        self.select_step = Entry(window, font='Courier 14', width=8, bd=0)
        self.select_step.place(x=210, y=35, width=50, height=30)
 
        self.lng_lbl = Label(window, text='Language', font='Courier 14', bg='grey')
        self.lng_lbl.place(x=270, y=35, width=90, height=30)
 
        self.lng = Combobox(window, font='Courier 14', width=10, values=['English', 'Russian'], state='readonly')
        self.lng.place(x=370, y=35, width=100, height=30)
 
        self.btn_for_decode = Button(window, text='Run', font='Courier 14', command=self.decode_btn)
        self.btn_for_decode.place(x=600, y=35, width=100, height=35)
 
        self.txt_lbl = Label(window, text='Text', font='Courier 14', bg='grey')
        self.txt_lbl.place(x=0, y=70, width=50, height=30)
 
        self.txt = Text(window, height=20, width=78, font='Courier 14', bd=0, wrap=WORD)
        self.txt.place(x=0, y=100, relwidth=0.99, relheight=0.8)
        self.scroll_for_txt = Scrollbar(window)
        self.scroll_for_txt.place(relx=0.99, y=100, relheight=0.8)
        self.txt.config(yscrollcommand=self.scroll_for_txt.set)
 
        self.rt_clc = Menu(self.txt, tearoff=0)
        self.rt_clc.add_command(label='Select all', accelerator="Alt+A",
                                command=lambda: self.txt.tag_add(SEL, '1.0', END))
        self.rt_clc.add_command(label='Cut', accelerator="Alt+X", command=lambda: self.txt.event_generate('<<Cut>>'))
        self.rt_clc.add_command(label='Copy', accelerator="Alt+C", command=lambda: self.txt.event_generate('<<Copy>>'))
        self.rt_clc.add_command(label='Paste', accelerator="Alt+V",
                                command=lambda: self.txt.event_generate('<<Paste>>'))
        self.rt_clc.add_command(label='Clear', command=lambda: self.txt.delete(1.0, END))
        self.txt.bind('<Button-3>', lambda event: self.rt_clc.post(event.x_root, event.y_root))
 
        self.mode.bind('<<ComboboxSelected>>', self.check_mode)
        self.txt.bind('<Key>', self.rus_ctrl)
 
    def decode_btn(self):                              #Кнопка для запуска
        if self.lng.get() == 'Russian':
            alphabet = 'абвгдеёжзийклмнопрстуфхцчшщъыьэюя'
        else:
            alphabet = 'abcdefghijklmnopqrstuvwxyz'
        if self.mode.get() == 'Decrypt':
            decode_txt = decoder(self.txt.get(1.0, END), key_finder(frequency_finder(self.txt.get(1.0, END), alphabet),
                                                                    alphabet), alphabet)
        else:
            decode_txt = decoder(self.txt.get(1.0, END), -int(self.select_step.get()), alphabet)
        self.txt.delete(1.0, END)
        self.txt.insert(1.0, decode_txt)
 
    def check_mode(self, window):                               #Метод который делает step неактивнымпри расшифровке
        if self.mode.get() == 'Decrypt':
            self.select_step.config(state='disabled')
        else:
            self.select_step.config(state='normal')
 
    def open_file(self):                #Открытие файла
        f_name = fd.askopenfilename(title='Select file', filetypes=(('MS WORD file', '*.docx'), ('Text File', '*.txt')))
 
        if '.docx' in f_name:
            f = Document(f_name)
            f_txt = ''
            for paragraph in f.paragraphs:
                f_txt += paragraph.text + '\n'
        else:
            f = open(f_name, 'r', encoding='utf-8')
            f_txt = f.read()
            f.close()
        self.txt.delete(1.0, END)
        self.txt.insert(1.0, f_txt)
 
    def save_file(self):            #Сохранение файла
        f_name = fd.asksaveasfilename(title='Select file', defaultextension='*.docx',
                                      filetypes=(('MS WORD file', '*.docx'), ('Text File', '*.txt')))
        saved_txt = self.txt.get(1.0, END)
        if '.docx' in f_name:
            f = Document()
            f.add_paragraph(saved_txt)
            f.save(f_name)
        elif '.txt' in f_name:
            f = open(f_name, 'w', encoding='utf-8')
            f.write(saved_txt)
            f.close()
 
    def rus_ctrl(self, event):                  #Нужно для того чтобы работали ctrl+v и тд в русской раскладке
        ctrl = (event.state & 0x4) != 0
        if event.keycode == 88 and ctrl and event.keysym.lower() != 'x':
            event.widget.event_generate('<<Cut>>')
 
        if event.keycode == 86 and ctrl and event.keysym.lower() != 'v':
            event.widget.event_generate('<<Paste>>')
 
        if event.keycode == 67 and ctrl and event.keysym.lower() != 'c':
            event.widget.event_generate('<<Copy>>')
 
        if event.keycode == 65 and ctrl and event.keysym.lower() != 'a':
            self.txt.delete(1.0, END)
 
 
def frequency_finder(text, used_alphabet):          #Ищет частоту букв
    text = text.lower()
    text_letter_frequency = list()
    for alphabet_letter in used_alphabet:
        letter_counter = 0
        for text_letter in text:
            if alphabet_letter == text_letter:
                letter_counter += 1
        text_letter_frequency.append(letter_counter / len(text))
    return text_letter_frequency
 
 
def key_finder(frequency_array, used_alphabet):         #Ищет ключ для расшифровки
    if used_alphabet[0] == 'a':  # EN 'a'
        alphabet_letter_frequency = (0.0817, 0.0149, 0.0278, 0.0425, 0.127, 0.0223, 0.0202, 0.0609, 0.0697, 0.0015,
                                     0.0077, 0.0403, 0.0241, 0.0675, 0.0751, 0.0193, 0.001, 0.0599, 0.0633, 0.0906,
                                     0.0276, 0.0098, 0.0236, 0.0015, 0.0197, 0.0007)
    else:
        alphabet_letter_frequency = (0.07998, 0.01592, 0.04533, 0.01687, 0.02977, 0.08483, 0.00013, 0.0094, 0.01641,
                                     0.07367, 0.01208, 0.03486, 0.04343, 0.03203, 0.067, 0.10983, 0.02804, 0.04746,
                                     0.05473, 0.06318, 0.02615, 0.00267, 0.00966, 0.00486, 0.0145, 0.00718, 0.00361,
                                     0.00037, 0.01898, 0.01735, 0.00331, 0.00639, 0.02001)
    key, difference = 0, 100
    for i in range(len(used_alphabet)):
        difference_for_i = 0
        for j in range(len(used_alphabet)):
            difference_for_i += abs(alphabet_letter_frequency[j - i] - frequency_array[j])
        if difference_for_i < difference:
            difference, key = difference_for_i, i
    return key
 
 
def decoder(text, step, usd_alp):                   #Расшифровывает
    dcr_text = ''
    for symbol in text:
        if symbol in usd_alp:
            if usd_alp.index(symbol) - (step % len(usd_alp)) < len(usd_alp):
                dcr_text += usd_alp[usd_alp.index(symbol) - (step % len(usd_alp))]
            else:
                dcr_text += usd_alp[usd_alp.index(symbol) - (step % len(usd_alp)) - len(usd_alp)]
        elif symbol in usd_alp.upper():
            if usd_alp.index(symbol.lower()) - (step % len(usd_alp)) < len(usd_alp):
                dcr_text += usd_alp[usd_alp.index(symbol.lower()) - (step % len(usd_alp))].upper()
            else:
                dcr_text += usd_alp[usd_alp.index(symbol.lower()) - (step % len(usd_alp)) - len(usd_alp)].upper()
        else:
            dcr_text += symbol
    return dcr_text
 
 
root = Tk()
 
app = Interface(root)
 
root.mainloop()
0
Лучшие ответы (1)
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
01.12.2019, 19:40
Ответы с готовыми решениями:

Дайте советы по улучшению кода
Как можно улучшить код чтобы сократить время его выполнения до минимума? Можно как нибудь...

Соглашения по оформлению кода
Всем привет! Вот какой вопрос есть... где можно найти и существует ли документ, описывающий...

Совет по оформлению кода
Есть программа, работает нормально. Обрабатывает из файла двумерный массив и выводит измененный...

Подскажите книги по оформлению кода
Всем привет! Подскажите книги по оформлению кода т.е. по правилам написания годного, читаемого кода...

1
814 / 526 / 214
Регистрация: 22.12.2017
Сообщений: 1,495
01.12.2019, 20:49 2
Лучший ответ Сообщение было отмечено nebantepls как решение

Решение

да программа как программа, просто в pycharm закинь и пусть он отформатирует как надо по стандарту, если ещё не делал, над читаемостью разве что можно поработать, документацию написать более грамотно, над именами получше подумать, askopenfilename например на мой взгляд не лучшим образом выглядит

Добавлено через 1 минуту
можешь почитать если не лень
1
01.12.2019, 20:49
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
01.12.2019, 20:49
Помогаю со студенческими работами здесь

Рекомендации по оформлению PHP кода
Часто читая темы новичков я встречаю очень ужасного вида код, это пугает и отбивает желание...

Нужна конструктивная критика по оформлению и читабельности кода (Game Trainer C++)
Относительно недавно начал изучать C++. Для обучения решил написать трейнер попутно изучая...

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

Рекомендации по оформлению кода WPF (и не только WPF)
Тема создана разделением темы https://www.cyberforum.ru/wpf-silverlight/thread408924.html:...

Советы по сокращению кода
Я только начал изучать c++ скажите пожалуйста, есть ли в коде лишние движения? Функция кода...

Советы по оптимизации кода
Написал парсер данных. Подскажите как можно было сделать проще? Стрелка должна быть в отдельной...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru