Update #21
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,54 +1,45 @@ | ||
| #include "../include/LinkedList.h" | ||
| #include <iostream> | ||
| #include <string> | ||
| #include <vector> | ||
|
|
||
| int main() { | ||
| try { | ||
| // Creazione della lista collegata | ||
| LinkedList lista; | ||
|
|
||
| // Creazione dispositivi di test | ||
| Dispositivo d1("Lampada", 1, 100, 5, 10); | ||
| Dispositivo d2("PC", 2, 200, 8, 12); | ||
| Dispositivo d3("TV", 3, 150, 6, 9); | ||
|
|
||
| // Inserimento dispositivi | ||
| lista.insert(d1); | ||
| lista.insert(d2); | ||
| lista.insert(d3); | ||
|
|
||
| // Visualizzazione iniziale della lista | ||
| std::cout << "Lista iniziale:\n" << lista.showAll() << std::endl; | ||
|
|
||
| // Rimozione per nome | ||
| lista.removeDispositivoName("Lampada"); | ||
| std::cout << "Lista dopo rimozione del dispositivo 'Lampada':\n" << lista.showAll() << std::endl; | ||
|
|
||
| // Rimozione per ID | ||
| lista.removeDispositivoId(2); | ||
| std::cout << "Lista dopo rimozione del dispositivo con ID 2:\n" << lista.showAll() << std::endl; | ||
|
|
||
| // Rimozione dispositivi spenti | ||
| auto dispositiviSpenti = lista.removeAllDispositiviOff(10); | ||
| std::cout << "Dispositivi rimossi (spenti):" << std::endl; | ||
| for (const auto& dispositivo : dispositiviSpenti) { | ||
| std::cout << dispositivo->getNome() << std::endl; | ||
| } | ||
|
|
||
| // Visualizzazione finale della lista | ||
| std::cout << "Lista finale:\n" << lista.showAll() << std::endl; | ||
| } catch (const std::exception& e) { | ||
|
There was a problem hiding this comment. issue (bug_risk): La gestione degli errori dovrebbe propagare lo stato di errore attraverso il valore di ritorno Attualmente l'errore viene registrato ma il programma restituisce 0 (successo). Questo potrebbe mascherare gli errori dai processi chiamanti. Considera di restituire un valore diverso da zero per indicare un fallimento. Original comment in Englishissue (bug_risk): Error handling should propagate failure status through return value Currently the error is logged but the program returns 0 (success). This could mask failures from calling processes. Consider returning a non-zero value to indicate failure. |
||
| std::cerr << "Errore: " << e.what() << std::endl; | ||
| } | ||
| return 0; | ||
| } | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 (security): Chiarisci la semantica di proprietà dei puntatori restituiti da removeAllDispositiviOff
La funzione restituisce puntatori raw senza semantica di proprietà chiara. Considera l'uso di smart pointer o documenta il trasferimento di proprietà per prevenire memory leak.
Original comment in English
🚨 issue (security): Clarify ownership semantics of returned pointers from removeAllDispositiviOff
The function returns raw pointers without clear ownership semantics. Consider using smart pointers or documenting the ownership transfer to prevent memory leaks.