The value of the inventory of a particular book is the price of that book times the number of copies we have in stock. Write a function called inventory_value that takes a Book as its argument and returns the value of our inventory of that book. Then write a function called top_value that takes a list of Books as its argument and returns the Book object (from the list) that has the highest-vaue inventory. Finally, write a sequence of statements that prints a line in this form: The highest-value book is War and Peace by Tolstoy, Leo at a value of $ 595.00.
So far, I've written this:
from collections import namedtuple
Book = namedtuple('Book', 'author title genre year price instock')
# Book attributes: author, title of book, category of book, year of publication, and number of copies available
BSI = [
Book('George Orwell', 'Animal Farm', 'Fiction', 1945, 9.99, 21),
Book('J.K. Rowling', 'Harry Potter and the Deathly Hallows', 'Fantasy', 2007, 24.26, 32),
Book('J.R.R. Tolkien', 'The Fellowship of the Ring', 'Fantasy', 1954, 10.87, 26),
Book('Toni Morrison', 'The Bluest Eye', 'Fiction', 1970, 11.02, 13),
Book('Ernesto Galarza', 'Barrio Boy', 'Autobiography', 1971, 13.50, 23),
Book('Stephen Chbosky', 'The Perks of Being a Wallflower', 'Fiction', 1999, 8.01, 25)]
def inventory_value(b: Book)->str:
return (b.price * b.instock)
assert(inventory_value(BSI[1])) == 776.32
assert(inventory_value(BSI[2])) == 282.62
def top_value(b: Book)->str:
for top_value in BSI:
if (b.price * b.instock) == max(inventory_value):
return(top_value.title)
print(top_value(BSI))
It's my function of top_value that isn't working out for me, but I can't tell what I'm doing wrong. I'm new to this and I'd like for someone to explain how I can go about doing this part of the problem. Thanks!
In your code, there are a few issues with the implementation of the top_value function that need to be addressed.
1. Naming Conflict: You have used the same variable name 'top_value' for both the function and the loop variable. This can cause confusion and unexpected behavior.
2. Incorrect Logic: In the if statement condition, you're trying to calculate the maximum value of the inventory_value, which is not the correct approach. We need to compare the inventory value of each book and find the book with the highest inventory value.
Here's the corrected implementation for the top_value function:
```python
def top_value(books: List[Book]) -> str:
highest_value = 0
top_book = None
for book in books:
value = inventory_value(book)
if value > highest_value:
highest_value = value
top_book = book
return top_book
# Test the top_value function
highest_value_book = top_value(BSI)
print('The highest-value book is', highest_value_book.title, 'by', highest_value_book.author,
'at a value of $', highest_value_book.price * highest_value_book.instock)
# Output: The highest-value book is Harry Potter and the Deathly Hallows by J.K. Rowling at a value of $ 776.32
```
In this code:
1. The `top_value` function receives a list of books as an argument.
2. It initializes a variable `highest_value` to keep track of the highest inventory value found so far and a variable `top_book` to store the book with the highest inventory value.
3. It iterates through each book in the list and calculates the inventory value using the `inventory_value` function.
4. If the inventory value of the current book is higher than the previously highest value, it updates the `highest_value` and `top_book` variables accordingly.
5. Finally, it returns the `top_book`, which represents the book with the highest inventory value.
Hope this helps! Let me know if you have any further questions.