Merge pull request #1 from PierBover/PierBover-patch-array-tutorial

Clarification of array example
pull/2491/head
Pier Bover 7 years ago committed by GitHub
commit 428c737873
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -1,8 +1,8 @@
<script>
let numbers = [1, 2, 3, 4];
function addNumber() {
numbers.push(numbers.length + 1);
function addNumber(value) {
numbers.push(value);
}
$: sum = numbers.reduce((t, n) => t + n, 0);
@ -12,4 +12,4 @@
<button on:click={addNumber}>
Add a number
</button>
</button>

@ -1,8 +1,8 @@
<script>
let numbers = [1, 2, 3, 4];
function addNumber() {
numbers = [...numbers, numbers.length + 1];
function addNumber(value) {
numbers = [...numbers, value];
}
$: sum = numbers.reduce((t, n) => t + n, 0);
@ -12,4 +12,4 @@
<button on:click={addNumber}>
Add a number
</button>
</button>

@ -7,8 +7,8 @@ Because Svelte's reactivity is triggered by assignments, using array methods lik
One way to fix that is to add an assignment that would otherwise be redundant:
```js
function addNumber() {
numbers.push(numbers.length + 1);
function addNumber(value) {
numbers.push(value);
numbers = numbers;
}
```
@ -16,8 +16,8 @@ function addNumber() {
But there's a more *idiomatic* solution:
```js
function addNumber() {
numbers = [...numbers, numbers.length + 1];
function addNumber(value) {
numbers = [...numbers, value];
}
```

Loading…
Cancel
Save