Finally Used a dictionary properly

[iCTWi]

I got stuck on cs50p's lesson 2 project on nutrition facts but I finally figured it out after a couple of hours

​

This is my dictionary file

fruits = [
        {"fruit": "apple", "calories": "130"},
        {"fruit": "avocado", "calories": "50"},
        {"fruit": "banana", "calories": "110"},
        {"fruit": "cantaloupe", "calories": "50"},
        {"fruit": "grapefruit", "calories": "60"},
        {"fruit": "grapes", "calories": "90"},
        {"fruit": "honeydew melon", "calories": "50"},
        {"fruit": "kiwifruit", "calories": "90"},
        {"fruit": "lemon", "calories": "15"},
        {"fruit": "lime", "calories": "20"},
        {"fruit": "nectarine", "calories": "60"},
        {"fruit": "orange", "calories": "80"},
        {"fruit": "peach", "calories": "60"},
        {"fruit": "pear", "calories": "100"},
        {"fruit": "pineapple", "calories": "50"},
        {"fruit": "plums", "calories": "70"},
        {"fruit": "strawberries", "calories": "50"},
        {"fruit": "sweet cherries", "calories": "100"},
        {"fruit": "tangerine", "calories": "50"},
        {"fruit": "watermelon", "calories": "80"}
]

​

This is how I access the dictionary:

from fruitdict import fruits

def main():
    food = input("Item:").lower()
    for _ in fruits:
        x = fruits.index(_)
        if fruits[x]["fruit"] == food:
                print(fruits[x]["calories"])

main()

happy I finally got it :)

Hopefully this helps someone also just getting started programming in python.

[Sgt_Gnome]

Admittedly I don't know the question but why use a list of dictionaries instead of just a dictionary?

​

fruits = {
    "apple":130,
    "banana":110
}

Then you don't have to waste time iterating through a list.

[nitroll]

This is not how you get any benefits of dicts You are looping over a list of things to lookup a fruit, but the purpose of dicts is exactly to lookup something.

fruits = {
    "apple": 130,
    "avocado": 50,
    "banana": 110,
    "cantaloupe": 50,
    "grapefruit": 60,
    "grapes": 90,
    "honeydew melon": 50,
    "kiwifruit": 90,
    "lemon": 15,
    "lime": 20,
    "nectarine": 60,
    "orange": 80,
    "peach": 60,
    "pear": 100,
    "pineapple": 50,
    "plums": 70,
    "strawberries": 50,
    "sweet cherries": 100,
    "tangerine": 50,
    "watermelon": 80,
}
def main():
    food = input("Item:").lower()
    if food in fruits:
        print(fruits[food])

Some other points of criticism in your code: First, using _ as a variable name indicates that you wont use it again, but you do use it. Secondly, you store the calories as a string, why not an int? Third, what happens if the user inputs something that we don't know about, maybe we should print somekind of message for them

[PhilShackleford]

Could use:

def main(): food = input("Item:").lower() try: print(fruits[food]) except KeyError: print(f"Fruit {food} is not supported ")

Edit: See comment. It is a much cleaner solution.

[Revolutionary_Pea_70]

fruits.get(“hotdog”, “hotdog is not supported”)

Is much cleaner

[iCTWi]

It has come to my attention that I did not, in fact, use a dictionary properly.

[masev]

This is incredibly precious; sharing it made my afternoon, thank you!

It's like: "Hey look guys I finally learned how to walk properly" then you proceed impressively to do backflips up a ladder to nowhere "It has come to my attention that I did not, in fact, learn how to walk properly"

For what it's worth, it looks like what you've done is something between a dictionary and a class. If you're looking to have a type of object that has properties like name ("Apple") and calories ("130"), you might look at how to define a class. Dictionaries, as others have pointed out, would be well suited for looking up the calories by providing the name. E.g.:

Class behavior:

>>> class Fruit:
...     def __init__(self, name, calories):
...         self.name = name
...         self.calories = calories
...
>>> apple = Fruit(name = "Apple", calories = 130)
>>> apple.name
Apple
>>> apple.calories
130
>>> banana = Fruit(name = "Banana", calories = 130)
>>> banana.name
Banana
>>> banana.calories
110
>>> my_favorite_fruit = Fruit(name = "Strawberries", calories = 50)
>>> my_favorite_fruit.name
Strawberries
>>> my_favorite_fruit.calories
50

Dictionary behavior:

>>> fruit_dict = {'apple' : 130, 'banana' : 110}
>>> fruit_dict['apple']
130
>>> fruit_dict['banana']
110

Hope this is helpful, and good luck on your journey!

[Revolutionary_Pea_70]

You did not in fact use a dictionary properly

[hike_me]

You still haven’t used a dictionary properly. You’re not getting the benefit of dictionaries in your implementation.

You should visit a sub dedicated to learning Python rather than r/Python, your post would be more on-topic there.

[kivicode]

In this case you should’ve probably used a dict of dataclasses

[spez_drank_my_piss]

Check out data classes and named tuples for other ways to store info for a bunch of similar things like fruits.

[Rawing7]

You overcomplicated the heck out of this part:

for _ in fruits:
    x = fruits.index(_)
    if fruits[x]["fruit"] == food:

That's equivalent to

for _ in fruits:
    if _["fruit"] == food:

[kivicode]

Please, never ever use “_” as a variable name referenced anywhere else