Skip to content

Junior - lab functions done#16

Open
linharesjunior wants to merge 1 commit intoIronhack-data-bcn-oct-2023:mainfrom
linharesjunior:main
Open

Junior - lab functions done#16
linharesjunior wants to merge 1 commit intoIronhack-data-bcn-oct-2023:mainfrom
linharesjunior:main

Conversation

@linharesjunior
Copy link
Copy Markdown

No description provided.

@sh-ih
Copy link
Copy Markdown

sh-ih commented Oct 16, 2023

Junior,

Awesome job on this lab! ⭐

Keep using such clear names to name your variables, because that makes it easier for anyone to understand and reuse your code.

Just a comment, on question 5, you could simplify your code by calling the functions you created on the previous two questions:

def oper_all(arr, oper):
    if oper == "+":
        return sum_all(arr)
    elif oper == "*":
        return mult_all(arr)

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