Skip to content
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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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