Skip to content

Conversation

@JehFerreiraPG
Copy link

No description provided.

Copy link
Collaborator

@mayaracsferreira mayaracsferreira left a 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
Copy link
Collaborator

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:")
Copy link
Collaborator

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
Copy link
Collaborator

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obrigada prof pelo feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants