-
Notifications
You must be signed in to change notification settings - Fork 38
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
Projeto Guiado II- Jessica Ferreira #19
base: main
Are you sure you want to change the base?
Conversation
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.
Dona Jéssica,
Muito bem! Fez os exercícios propostos, tem alguns pontos de correção mas está indo no caminho certo. VI que começou a implementar até um extra, a senhora está de parabéns!
|
||
def emprestar_livro(self, livro): | ||
if livro in self.livros: | ||
return True |
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.
Esse método está indo no caminho certo porém aqui não basta retornar o True para o teste, o importante seria alterar a propriedade do livro de dentro da lista para emprestado = True
por exemplo:
self.livros[x].esta_emprestado = True
x sendo a posição do livro que quer alterar de fato. Vc pode dar uma olhada na função index() para pegar a posição do item na lista
|
||
|
||
def buscar_livros(self): | ||
id_livro = input("Digite o nome do livro desejado:") |
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.
Vi que essa implementação do método buscar_livros está bem promissora!
Só um comentário que não é necessário ter esse input do teclado aqui dentro da classe, acho quer não chegaram nessa parte da explicação de Orientação a objetos mas existe um conceito de responsabilidades da classe, esse método é responsável apenas por buscar o item na lista, aqui ele está lendo um input do teclado também. Isso pode ser feito por quem for chamar a classe biblioteca e o método de buscar_livros
Mas achei muito bacana que está trabalhando nessa parte
self.biblioteca.emprestar_livro(Livro) | ||
|
||
# Assert / comparar o resultado da função com o valor q eu vou informar | ||
self.assertTrue |
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.
O que a gente quer validar aqui?
o que o assertTrue e assertFalse precisa receber como parametro?
Falta pouco para completar essa parte, dona Jessica
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.
Obrigada prof pelo feedback!
No description provided.