Skip to content

Completed Project 2 - #15

Open
sokuno222 wants to merge 1 commit into
sd17fall:masterfrom
sokuno222:master
Open

sokuno222 wants to merge 1 commit into
sd17fall:masterfrom
sokuno222:master

Conversation

@sokuno222

Copy link
Copy Markdown

No description provided.

Comment thread recursive_art.py
l1.append(l2)
return l1

print("ran func: " + (str)(build_random_function(3,4)))

@mightydeveloper mightydeveloper Oct 11, 2017

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • You can remove to TODO comment at the top
  • If you use l = lambda : build_random_function(min_depth-1, max_depth-1), and use l() whenever you need them, you don't need to repeat l2, l3 codes and write out similar code for each if statement.
    for example, the code would look like
elif myInt == 2:
    return ['avg', l(), l()]
  • I'm not sure what the print code is doing... shouldn't it be str(build_random_function(3,4)) ??
  • Overall, great job for the concise code!

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