INDAPlus21 / INDAPlus21/antonul-sorting
Pass
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- HTML
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
**Well done Anton!**
_SORRII Anton. I'm stepping back from my initial harsh comment below. I've come to realise that my assignment instructions are open for missinterpretation. Since you made a good job of completing parts of the assignment, you gain a Pass._
Not only did you give me pesky programming for depricated estetare, you pushed substandard formatting and poor inline documentation; not good; not worthy. And where is your implementation of selection sort and merge sort?
~~Lets comprimise; I'll give you _Pass_ if you simply submit an implementation of selection sort and merge sort, without the necessary visualisation logic for them.~~
Lastly, nice pancake sort.
#### Notes on your code:
I notice that you do not make use of the full extent of javascript for-loops.
_Standard for-loop_:
```js
for (let i = 0; i < array.length; i++) {/*...*/}
```
_for-loop over index range_:
```js
for (let i in array) {/*...*/}
```
_for-loop over elements_:
```js
for (let elem of array) {/*...*/}
```
Have you considered using arithmetic values in arithmetic expressions to avoid code douplication?
_Your code_:
```js
if (x%2 == 0) {
for (let i = 0;i <= ofiInsert[x]; i++) {
fillNumberRed(actionsInsert[x][i],i);
}
}
else {
for (let i = 0;i <= ofiInsert[x-1]; i++) {
fillNumberRed(actionsInsert[x][i],i);
}
}
```
_No douplications_:
```js
for (let i = 0;i <= ofiInsert[x - (x % 2)]; i++) {
fillNumberRed(actionsInsert[x][i],i);
}
```
Nice! I actually did not know of this syntax, `[array[start], array[i]] = [array[i], array[start]];`.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
El issue no nombra archivos, pruebas ni puntos de entrada; consiste en comentarios sobre la tarea y ejemplos de JavaScript. La solicitud de selection-sort y merge-sort está tachada, y el issue concede un Pass, por lo que actualmente no hay ningún objetivo de finalización que verificar.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- frontend, web-dev
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 15/100