INDAPlus21 / INDAPlus21/antonul-sorting

Pass

Offen
#1 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
HTML
Sterne
0
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

**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]];`.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Das Issue nennt keine Dateien, Tests oder Einstiegspunkte; es besteht aus Feedback zur Aufgabe und JavaScript-Beispielen. Die Anforderung für selection-sort und merge-sort ist durchgestrichen, und das Issue vergibt ein Pass, daher gibt es derzeit kein Abschlussziel, das überprüft werden kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
frontend, web-dev
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
15/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.