golang / tour

[mirror] A Tour of Go
BSD 3-Clause "New" or "Revised" License
1.53k stars 521 forks source link

vague description for Exercise: Equivalent Binary Trees #654

Open BubbleBear opened 5 years ago

BubbleBear commented 5 years ago

Context: https://tour.golang.org/concurrency/8

it doesn't specify the order to walk the tree, while which implies the inorder traversal is required.

plz add this to the description to make it more clear. :) tx

jimbobmcgee commented 4 years ago

Completely agree; this exercise doesn't test anyone's ability to understand channels, more the prior knowledge of how a binary tree is supposed to be walked, and so detracts from the overall goal of teaching about channels.

Really, any indicator for the non-CS among us, that you have to walk the left, then emit the current value, then walk the right would be a bonus...