Closed hjalm8 closed 3 years ago
Also check out my version of the exact same feature that I worked on with no clue about this one, at the exact same time. lol https://github.com/andesyv/lesebot/commit/5c6188387723459d5b6eee4d61cd7a2fd055737a
Tok en kjapp test på den nå:
MAT220
og STAT101
ble opprettet i riktig kategoriINF207
eller INF143A
ble plassert i noen kategori, de bare havnet frittståendeDAT251
siden den ikke er i eksamensinfofila, så den default
-branchen i switchen
hvor du finner kategori funker ikkeSorry, no course with the code found... try again
som feil, tror det er fordi du gjør if (subject == undefined)
, om du gjør if (subject === "")
tror jeg den blir riktig. Har samme problem i f.eks. insult
men skjønner ikke hvorfor den er ei tom streng og ikke undefined... Er kanskje mer riktig å gjøre if (!subject)
i steden så man sjekker for begge :thinking: * Verken `INF207` eller `INF143A` ble plassert i noen kategori, de bare havnet frittstående
category.name har funky case, burde egentlig bruke cache.find(cat => cat.id == "$ID")
* Det funket ikke å lage `DAT251` siden den ikke er i eksamensinfofila, så den `default`-branchen i `switchen` hvor du finner kategori funker ikke
Den funksjonen er kun for å hente ut hvilken kategori, i discord serveren, de skal plasseres i.
Sorry, no course with the code DAT251 found... try again
Betyr at den ikke fant faget i eksamensfila. User-input stringen inputSubject brukes ikke videre, og faget DAT251 vil aldri komme inn i switch casen.
Velger å ikke stole på user input. Den bruker ikke input til noe annet en å finne faget i egen database over fag.
* Når jeg prøvde uten et argument fikk jeg `Sorry, no course with the code found... try again` som feil, tror det er fordi du gjør `if (subject == undefined)`, om du gjør `if (subject === "")` tror jeg den blir riktig. Har samme problem i f.eks. `insult` men skjønner ikke hvorfor den er ei tom streng og ikke undefined... Er kanskje mer riktig å gjøre `if (!subject)` i steden så man sjekker for begge 🤔
Hadde noen problemer med at den prøvde å kalle undefined.toUpperCase().trim().
Foreslår:
if (subject === undefined || subject.trim() === "")
, fordi
a = undefined;
> undefined
a === ""
> false
a == ""
> false
Var ikke helt sikker på om === var nødvendig i TS, men når jeg ser litt mer på det, så er jeg enig i at det er best å unngå ==.
Added command for requesting text channels, based of course codes in exam.json