#155 Komentojen refaktoroiminen omiksi luokikseen:

M4R774 / bobweb2

Telegram botti piristämään elämää

MIT License

8 stars 1 forks source link

#155 Komentojen refaktoroiminen omiksi luokikseen: #156

Closed Latemus closed 2 years ago

Latemus commented 2 years ago

Lisätty AbstractCommand ja sen konkreettiset toteutukset HelpCommand ja LeetCommand
- Nämä kaksi lisätty ensimmäisinä toimivuuden ja rakenteen testaamisen vuoksi. Loput lisätään jos tämä ok
Nyt nämä komennot ovat täysin omissa moduuleissaan (luominen ja toiminnallisuus)
Muutettu message_handler toimimaan AbstractCommand toteuttavilla objekteilla
Demote ja Promote metodit siirretty help_command.py -> ranks.py

Latemus commented 2 years ago

@M4R774 Tässä ei mee testit läpi, kun vain 2 komentoa toteutettu. Tarkotus siis demota tätä rakenteellista muutosta ja kysyä siitä mielipidettä, että kannattaako viedä loputkin komennot omiin luokkiinsa.

Latemus commented 2 years ago

Ja tämä siis lähinnä välivaiheen ehdotus, ennenkö käytän enempää aikaa tän luokkatoteutuksen tekemiseen. Jos sun mielestä ok, niin suljen tän pull-requestin, teen loppujen komentojen siirrot ja hion toteutuksen, jonka jälkeen avaan sitten uudelleen kun valmiimpi lopullista tarkastelua varten.

Latemus commented 2 years ago

En saanut noita järkevästi paketoitua pakettihierarkiaan, kun importit lakkas toimimasta, eikä oikein löytynyt ratkasua siihen. Eli se pitää sitten selvitellä seuraavaksi.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
4 Code Smells

98.8% Coverage
0.0% Duplication

Latemus commented 2 years ago

@M4R774 Mä nyt poistin tuon Abstraktin luokan käsitteen ja lisäilin kommentteja ja noit attribuuttien nimien määrittelyjä constructor-kutsuihin.