-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update calculadora.py #36
base: master
Are you sure you want to change the base?
Update calculadora.py #36
Conversation
Versão atualizada da Calculadora-TK. Com as funções de logaritmo, percentual, fatorial, seno, cosseno, tangente e conversões para números binários, hexadecimais e octais.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obrigado pela sua contribuição @FernandoBertoldi12 😄
Bem, creio que houve pequeno desvio aqui. A ideia é manter apenas uma quantidade de operações matemáticas conforme os botões disponíveis, que no momento são 2. Isso é para forçar que seja uma calculadora simples (não tenho a intenção de transformar em uma calculadora científica no momento, talvez no futuro) e não necessite de mudanças no layout.
Portanto, peço que mantenha apenas as operações de % e log conforme a issue #6 e remova as demais. Você pode fazer isso, por favor?
Além disso, tem algumas pequenas revisões gerais abaixo. Peço que as resolva também.
@@ -1,10 +1,10 @@ | |||
# -*- coding: utf-8 -*- | |||
|
|||
import math |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Siga o mesmo padrão nas importações. Mova essa linha para junto dos imports Builtins
abaixo.
self._BTN_PORCENT = tk.Button(master, text='%', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_LOG = tk.Button(master, text='log', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_FAT = tk.Button(master, text='!', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_SIN = tk.Button(master, text='sin', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_COS = tk.Button(master, text='cos', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_TAN = tk.Button(master, text='tan', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_BIN = tk.Button(master, text='bin', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_OCT = tk.Button(master, text='oct', cnf=self.theme['BTN_OPERADOR']) | ||
self._BTN_HEX = tk.Button(master, text='hex', cnf=self.theme['BTN_OPERADOR']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Há uma repetição de código aqui. Os botões já foram atribuídos algumas linhas acima. Você pode remover esse trecho?
Versão atualizada da Calculadora-TK.
Com as funções de logaritmo, percentual, fatorial, seno, cosseno, tangente e conversões para números binários, hexadecimais e octais.