Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

Create a dictionary as a friend group #11

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

Conversation

kyrlazari
Copy link

Created a group of people using dictionary as a structure. Issue 8: UCL-RITS/rse-classwork-2020#8

group.py Outdated
my_group =
my_group = {"Kyriazis":{"age":25,
"job":"Msc Student",
"friends":["Mathew","Michael"]
Copy link

Choose a reason for hiding this comment

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

Nice simple structure, what it you wanted to add a different type of relationship apart from friends?

Choose a reason for hiding this comment

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

Nice and neat

Copy link

@immyharrison immyharrison left a comment

Choose a reason for hiding this comment

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

this limits the relationship to only friend, could have enter that was connections/person and inside that have the option to add friend/landlord/family etc. may make it more widely applicable.

Very clearly laid out and easy to read.

Add print statement at end, makes output very clear.

LuweiCai123 pushed a commit to Ebony59/friend-group-2020 that referenced this pull request Oct 28, 2020
@kyrlazari kyrlazari mentioned this pull request Oct 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants