Add documentation for lambdas. #498
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kasperl
approved these changes
Aug 23, 2024
docs/language/blocks-and-lambdas.mdx
Outdated
|
|
||
| ``` | ||
| ## Introduction to Blocks and Lambdas |
There was a problem hiding this comment.
Suggested change
| ## Introduction to Blocks and Lambdas | |
| ## Introduction to blocks and lambdas |
docs/language/blocks-and-lambdas.mdx
Outdated
| [1, 2, 3].do: printer.call it | ||
| ``` | ||
|
|
||
| ## Key Differences |
There was a problem hiding this comment.
Suggested change
| ## Key Differences | |
| ## Key differences |
docs/language/blocks-and-lambdas.mdx
Outdated
| 4. **Flexibility**: Lambdas can be stored in variables, fields, and collections, and returned from functions. Blocks cannot. | ||
| 5. **Efficiency**: Blocks are significantly more efficient than lambdas. | ||
|
|
||
| ## Blocks in Detail |
There was a problem hiding this comment.
Suggested change
| ## Blocks in Detail | |
| ## Blocks in detail |
docs/language/blocks-and-lambdas.mdx
Outdated
|
|
||
| ## Blocks in Detail | ||
|
|
||
| ### Efficiency of Blocks |
There was a problem hiding this comment.
Suggested change
| ### Efficiency of Blocks | |
| ### Efficiency of blocks |
docs/language/blocks-and-lambdas.mdx
Outdated
|
|
||
| - [Lambdas](../tasks) cannot capture blocks. | ||
| ## Lambdas in Detail |
There was a problem hiding this comment.
Suggested change
| ## Lambdas in Detail | |
| ## Lambdas in detail |
docs/language/blocks-and-lambdas.mdx
Outdated
| callbacks_.do: it.call new-value | ||
|
|
||
| add-printer box/WatchedBox: | ||
| box.watch:: print "Was changed to $it." |
There was a problem hiding this comment.
Suggested change
| box.watch:: print "Was changed to $it." | |
| box.watch:: print "was changed to $it." |
There was a problem hiding this comment.
I think I would drop the trailing period here (and in the comments). It just seems misplaced and we don't typically have that in debug prints.
docs/language/blocks-and-lambdas.mdx
Outdated
| This example demonstrates how lambdas can be stored in a list | ||
| (`callbacks_`) and called later. | ||
|
|
||
| ### Lambda Arguments |
There was a problem hiding this comment.
Suggested change
| ### Lambda Arguments | |
| ### Lambda arguments |
| main: | ||
| // Both of these will print 42. | ||
| my-function:: print it | ||
| my-function: print it |
There was a problem hiding this comment.
It feels more logical to start with the block-variant call.
docs/language/blocks-and-lambdas.mdx
Outdated
| - Consider providing both block and lambda versions if you need to support both | ||
| use cases. The lambda version can always redirect to the block version: | ||
| ``` | ||
| my-function callback: |
There was a problem hiding this comment.
Maybe flip the order of these two functions?
docs/language/index.mdx
Outdated
| A block can return a value each time it is run. This is used for example in the `filter` method on lists. | ||
| ``` | ||
| // Returns a function that adds n to its argument. | ||
| add-n n: |
There was a problem hiding this comment.
Suggested change
| add-n n: | |
| add-n n -> Lambda: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.