-
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?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
from livro import Livro | ||
|
||
class Biblioteca: | ||
def __init__(self): | ||
self.livros = [] | ||
|
||
def adicionar_livro(self, livro: Livro): | ||
if(not isinstance(livro, Livro)): | ||
raise TypeError(f"Esperado livro obtido valor {livro} do tipo {type(livro)}") | ||
self.livros.append(livro) | ||
|
||
def exibir_livros(self, livro: Livro): | ||
#if | ||
return self.livros | ||
|
||
def emprestar_livro(self, livro): | ||
if livro in self.livros: | ||
return True | ||
else: | ||
return False | ||
|
||
""" | ||
def buscar_livro(self, livro: Livro): | ||
if(not isinstance(livro, Livro)): | ||
raise TypeError("Esse livro não faz parte do nosso acervo") | ||
return self.livros | ||
|
||
|
||
|
||
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 commentThe 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 |
||
for livro in self.livros: | ||
if livro["nome"] == id_livro: | ||
print(f"O livro xxx de id xxx, está na rua xx, prateleira x.") | ||
return | ||
print("Não possuimos este livro em nossa biblioteca.") | ||
""" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
class Livro: | ||
def __init__(self, nome, autor): | ||
self.nome = nome | ||
self.autor = autor | ||
self.esta_emprestado = False |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
""" | ||
from Biblioteca import Biblioteca | ||
from Livro import Livro | ||
|
||
nome_livro = "O mito da beleza" | ||
autor_livro = "Naomi Wolf" | ||
livro_objeto = Livro(nome = nome_livro,autor = autor_livro) | ||
|
||
biblioteca_objeto = Biblioteca() | ||
|
||
|
||
print(biblioteca_objeto.livros) | ||
|
||
biblioteca_objeto.adicionar_livro(livro_objeto) | ||
|
||
for livro in biblioteca_objeto.livros: | ||
print(livro.nome) | ||
print(livro.autor) | ||
""" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
from unittest import TestCase | ||
from livro import Livro | ||
|
||
class Testlivro(TestCase): | ||
def test_init_deve_passar(self): | ||
# Arrange | ||
nome = "Calibã e a bruxa" | ||
autor = "Silvia Federici" | ||
|
||
# Act | ||
livro = Livro(nome, autor) | ||
|
||
# Assert | ||
self.assertEqual(nome, livro.nome) | ||
self.assertEqual(autor, livro.autor) | ||
self.assertEqual(False, livro.esta_emprestado) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,82 @@ | ||
from unittest import TestCase | ||
from biblioteca_pg2 import Biblioteca | ||
from livro import Livro | ||
|
||
class TestBiblioteca(TestCase): | ||
def setUp(self): | ||
self.biblioteca = Biblioteca() | ||
|
||
def test_init_deve_passar(self): | ||
# Arrange / Act | ||
|
||
# Assert | ||
self.assertIsInstance(self.biblioteca.livros, list) | ||
|
||
def test_adicionar_livro_deve_passar(self): | ||
|
||
#Arrange / organizar os valores p/serem testados | ||
nome_livro = "O mito da beleza" | ||
autor_livro = "Naomi Wolf" | ||
livro = Livro(nome_livro, autor_livro) | ||
|
||
#Act / chamar a função q qro testar | ||
self.biblioteca.adicionar_livro(livro) | ||
|
||
# Assert / comparar o resultado da função com o valor q eu vou informar | ||
self.assertEqual(1, len(self.biblioteca.livros)) | ||
|
||
|
||
def test_adicionar_livro_nao_deve_inserir_numero(self): | ||
#arrange | ||
|
||
livro = 1988 | ||
|
||
#assert/act | ||
with self.assertRaises(TypeError): | ||
self.biblioteca.adicionar_livro(livro) | ||
|
||
def test_exibir_livros_deve_passar(self): | ||
# Arrange | ||
self.biblioteca.livros = ["Jessica"] | ||
nome_livro = "Jessica" | ||
autor_livro = "yy" | ||
livro = Livro(nome_livro , autor_livro) | ||
|
||
# Act | ||
self.biblioteca.exibir_livros(livro) | ||
|
||
# Assert | ||
self.assertEqual(["Jessica"], self.biblioteca.livros) #comparação do livro com biblioteca | ||
|
||
def test_emprestar_livro_deve_passar_ou_nao(self): | ||
#Arrange / organizar os valores p/serem testados | ||
self.biblioteca.livros = ["xxx"] | ||
nome_livro = "xxx" | ||
autor_livro = "RJ" | ||
livro = Livro(nome_livro, autor_livro) | ||
|
||
# Act / chamar a função q qro testar | ||
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 commentThe reason will be displayed to describe this comment to others. Learn more. O que a gente quer validar aqui? Falta pouco para completar essa parte, dona Jessica There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Obrigada prof pelo feedback! |
||
self.assertFalse | ||
|
||
""" | ||
def test_buscar_livro_deve_passar_ou_nao(self): | ||
|
||
#Arrange / organizar os valores p/serem testados | ||
self.biblioteca.livros = ["WWW"] | ||
nome_livro = "WWW" | ||
autor_livro = "JR" | ||
livro = Livro(nome_livro, autor_livro) | ||
|
||
# Act / chamar a função q qro testar | ||
self.biblioteca.buscar_livro(self, livro, Livro) | ||
|
||
# Assert / comparar o resultado da função com o valor q eu vou informar | ||
self.assertIsInstance(self.biblioteca.livros, list) | ||
|
||
with self.assertRaises(TypeError): | ||
self.biblioteca.buscar_livro(livro) | ||
""" |
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