-
Notifications
You must be signed in to change notification settings - Fork 1
ciao Mocha #17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ciao Mocha #17
Conversation
LinkedList completata e classi errori + modifiche minime classi michele
Modifica test e main node
Array + linked list
|
Here's the translation to Italian: 🧙 Sourcery ha completato la revisione della tua pull request! Suggerimenti e comandiInteragire con Sourcery
Personalizzare la tua esperienzaAccedi al tuo dashboard per:
Ottenere aiuto
Original review guide in English🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ehi @JustGuardian - Ho esaminato le tue modifiche - ecco alcuni feedback:
Commenti Generali:
- Considera di standardizzare l'uso dello namespace std in tutto il codebase - usa prefissi std:: espliciti ovunque o un singolo 'using namespace std' all'inizio
- Correggi gli errori di battitura nei nomi dei dispositivi di test (ad es., 'Frigrifero', 'Lavtrice', 'Televisre') per garantire test affidabili
Ecco cosa ho esaminato durante la revisione
- 🟡 Problemi generali: 1 problema trovato
- 🟢 Sicurezza: tutto a posto
- 🟢 Test: tutto a posto
- 🟢 Complessità: tutto a posto
- 🟢 Documentazione: tutto a posto
Il tuo periodo di prova scade l'11 gennaio 2025. Per favore aggiorna per continuare a utilizzare Sourcery ✨
Aiutami a essere più utile! Per favore clicca 👍 o 👎 su ogni commento e userò il feedback per migliorare le tue revisioni.Original comment in English
Hey @JustGuardian - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider standardizing the use of namespace std throughout the codebase - either use explicit std:: prefixes everywhere or a single 'using namespace std' at the top
- Fix typos in test device names (e.g., 'Frigrifero', 'Lavtrice', 'Televisre') to ensure reliable testing
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Your trial expires on January 11, 2025. Please upgrade to continue using Sourcery ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.|
|
||
| Node* current = searchDispositivoName(nome); | ||
|
|
||
| if(current == head) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue: Gestione mancante per il caso di un singolo nodo nei metodi di rimozione
Quando si rimuove l'unico nodo (head == tail), il codice andrà in crash cercando di accedere a head->prev. Aggiungi un controllo per questo caso.
Original comment in English
issue: Missing handling for single-node case in remove methods
When removing the only node (head == tail), the code will crash trying to access head->prev. Add a check for this case.
Sommario di Sourcery
Implementazione della classe LinkedList per la gestione dei dispositivi, inclusi metodi per l'inserimento, rimozione per nome o ID, rimozione di dispositivi spenti, gestione dei timer e visualizzazione del consumo dei dispositivi.
Nuove Funzionalità:
Test:
Original summary in English
Summary by Sourcery
Implement the LinkedList class for managing devices, including methods for insertion, removal by name or ID, removing devices that are off, managing timers, and displaying device consumption.
New Features:
Tests: