Skip to content

Commit 28e1a56

Browse files
longo-andreapasor1
andauthored
Apply suggestions from code review
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
1 parent f03d375 commit 28e1a56

File tree

5 files changed

+13
-15
lines changed

5 files changed

+13
-15
lines changed

1-js/05-data-types/05-array-methods/1-camelcase/task.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,4 @@ camelize("list-style-image") == 'listStyleImage';
1616
camelize("-webkit-transition") == 'WebkitTransition';
1717
```
1818

19-
P.S. Suggerimento: usate `split` per divider una stringa in un array, trasformatela e infinite riunite tutto con `join`.
19+
P.S. Suggerimento: usate `split` per dividere una stringa in un array, trasformatela e infinite riunite tutto con `join`.

1-js/05-data-types/05-array-methods/10-average-age/task.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ importance: 4
44

55
# Ottenere l'età media+
66

7-
Scrivete una funzione `getAverageAge(users)` che accetta un array di oggetti con la proprietà `age` e ritorni l'età media.
7+
Scrivete una funzione `getAverageAge(users)` che accetti un array di oggetti con la proprietà `age` e ritorni l'età media.
88

99
La formula della media è: `(age1 + age2 + ... + ageN) / N`.
1010

1-js/05-data-types/05-array-methods/11-array-unique/task.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ importance: 4
66

77
Abbiamo un array `arr`.
88

9-
Create una funzione `unique(arr)` che ritornari un array con elementi unici.
9+
Create una funzione `unique(arr)` che ritorni un array con elementi unici.
1010

1111
Ad esempio:
1212

1-js/05-data-types/05-array-methods/4-sort-back/task.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ importance: 4
22

33
---
44

5-
# Riordinate in ordine descresente
5+
# Riordinare in ordine decrescente
66

77
```js
88
let arr = [5, 2, 1, -10, 8];
@@ -11,4 +11,3 @@ let arr = [5, 2, 1, -10, 8];
1111

1212
alert( arr ); // 8, 5, 2, 1, -10
1313
```
14-

1-js/05-data-types/05-array-methods/article.md

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ arr.slice([start], [end])
124124

125125
Ritorna un nuovo array contente tutti gli elementi a partire da `"start"` fino ad `"end"` (`"end"` escluso). Sia `start` che `end` possono essere negativi; in tal caso si inizierà a contare dalla coda dell'array.
126126

127-
Funziona come `str.slice`, ma crea dei sotto-array piuttosto di sotto-stringhe.
127+
Funziona come `str.slice`, ma crea dei sotto-array piuttosto che sotto-stringhe.
128128

129129
Ad esempio:
130130

@@ -136,7 +136,7 @@ alert( arr.slice(1, 3) ); // e,s (copia da 1 a 3)
136136
alert( arr.slice(-2) ); // s,t (copia da -2 fino alla fine)
137137
```
138138

139-
Possiamo anche utilizzarlo senza argomenti: `arr.slice()` crea una copia di `arr`. Questo tipo di chiamata è spesso utilizzata per una copia con cui poter liberamente lavorare senza modificare l'array originale.
139+
Possiamo anche utilizzarlo senza argomenti: `arr.slice()` crea una copia di `arr`. Questo tipo di chiamata è spesso utilizzata per creare una copia con cui poter liberamente lavorare senza modificare l'array originale.
140140

141141
### concat
142142

@@ -300,15 +300,15 @@ let user = users.find(item => item.id == 1);
300300
alert(user.name); // John
301301
```
302302

303-
Gli array di oggetti sono molto comuno, quindi il metodo `find` risulta molto utile.
303+
Gli array di oggetti sono molto comuni, quindi il metodo `find` risulta molto utile.
304304

305305
Da notare che nell'esempio noi forniamo a `find` un singolo argomento `item => item.id == 1`. Gli altri parametri di `find` sono raramente utilizzati.
306306

307307
Il metodo [arr.findIndex](mdn:js/Array/findIndex) fa essenzialmente la stessa cosa, ma ritorna l'indice in cui è stata trovata la corrispondenza piuttosto di ritornare l'oggetto stesso; se l'oggetto non viene trovato ritorna `-1`.
308308

309309
### filter
310310

311-
Il metodo `find` cerca un singola occorrenza dell'elemento, la prima, che passata a una funzione ritorna `true`.
311+
Il metodo `find` cerca un singola occorrenza dell'elemento, la prima, e se trovata ritorna `true`.
312312

313313
Se vogliamo cercare più occorrenze, possiamo utilizzare [arr.filter(fn)](mdn:js/Array/filter).
314314

@@ -448,7 +448,7 @@ alert(arr); // *!*1, 2, 15*/!*
448448
````
449449

450450
````smart header="Le arrow functions sono le migliori"
451-
Ricordate le [arrow functions](info:function-expressions-arrows#arrow-functions)? Possiamo utilizzarle per un più breve ordinamento:
451+
Ricordate le [arrow functions](info:function-expressions-arrows#arrow-functions)? Possiamo utilizzarle per più conciso il codice di ordinamento:
452452
453453
```js
454454
arr.sort( (a, b) => a - b );
@@ -490,7 +490,7 @@ Inoltre ritorna `arr` dopo averlo invertito.
490490
491491
### split e join
492492
493-
Vediamo una situazione realistica. Stiamo scrivendo un'applicazione di messaggistica, e l'utente inserisce una lista di destinatari: `John, Pete, Mary`. Per noi sarebbe più comodo avere un array di nomi piuttosto di una singola stringa. Come possiamo ottenerlo?
493+
Vediamo una situazione reale. Stiamo scrivendo un'applicazione di messaggistica, e l'utente inserisce una lista di destinatari: `John, Pete, Mary`. Per noi sarebbe più comodo avere un array di nomi piuttosto di una singola stringa. Come possiamo ottenerlo?
494494
495495
Il metodo [str.split(delim)](mdn:js/String/split) fa esattamente questo. Divide la stringa in un array utilizzando il delimitatore `delim`.
496496
@@ -524,7 +524,7 @@ alert( str.split('') ); // t,e,s,t
524524
```
525525
````
526526

527-
La chiamata ad [arr.join("collante")](mdn:js/Array/join) fa esattamente l'inverso di `split`. Crea una stringa con gli elementi di `arr` incollati tra loro dal `collante`.
527+
La chiamata ad [arr.join(separatore)](mdn:js/Array/join) fa esattamente l'inverso di `split`. Crea una stringa con gli elementi di `arr` incollati tra loro dal `separatore`.
528528

529529
Ad esempio:
530530

@@ -653,9 +653,9 @@ alert(Array.isArray({})); // false
653653
alert(Array.isArray([])); // true
654654
```
655655

656-
## I più dei metodi accettano "thisArg"
656+
## la maggior parte dei metodi accetta "thisArg"
657657

658-
Quasi tutti i metodi dedicati degli array che richiedono una funzione -- come `find`, `filter`, `map`, fatta eccezione per `sort`, accettano un parametro opzionale `thisArg`.
658+
Quasi tutti i metodi degli array che richiedono una funzione -- come `find`, `filter`, `map`, fatta eccezione per `sort`, accettano un parametro opzionale `thisArg`.
659659

660660
Questo parametro non è stato spiegato nella sezione sopra, perché viene raramente utilizzato. Studiamolo per completezza.
661661

@@ -701,7 +701,6 @@ alert(soldiers[1].age); // 23
701701

702702
Nella chiamata sopra, utilizziamo `user.younger` come filtro e forniamo `user` come contesto. Se non avessimo fornito il contesto, `users.filter(user.younger)` avrebbe chiamato `user.younger` come funzione a sé stante, con `this=undefined`. Che avrebbe provocato un errore.
703703

704-
A call to users.filter(army.canJoin, army) can be replaced with users.filter(user => army.canJoin(user)), that does the same. The latter is used more often, as it’s a bit easier to understand for most people.
705704

706705
Una chiamata a `user.filter(army.canJoin, army)` può essere sostituita da
707706
`users.filter(user => army.canJoin(user))`, che fa lo stesso. L'ultima versione viene utilizzata più spesso e per molte persone è più semplice da capire.

0 commit comments

Comments
 (0)