Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Destructors #25

Merged
merged 2 commits into from
Oct 26, 2023
Merged

Destructors #25

merged 2 commits into from
Oct 26, 2023

Conversation

DuarteSAssuncao
Copy link
Collaborator

Created destructors for the application classes that save the updated content in the .csv files.

Created destructors for the application classes that save the updated content in the .csv files
Copy link
Owner

@jvdcf jvdcf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

É importante fazermos a segunda parte do parser, obrigado por já teres adiantado isso!
Agora, temos de perguntar primeiro se queremos salvar apenas antes de fechar o programa ou a partir de um comando save() ou algo do género. No segundo caso, como queremos que ainda esteja em memória, é melhor serem métodos da classe em vez de desconstrutores.

@DuarteSAssuncao
Copy link
Collaborator Author

DuarteSAssuncao commented Oct 25, 2023

É importante fazermos a segunda parte do parser, obrigado por já teres adiantado isso! Agora, temos de perguntar primeiro se queremos salvar apenas antes de fechar o programa ou a partir de um comando save() ou algo do género. No segundo caso, como queremos que ainda esteja em memória, é melhor serem métodos da classe em vez de desconstrutores.

Estive a falar com o Gui sobre isso e acho que os dois concordamos que se guardaria no final, mas, para ser sincero, não me lembro a 100% da conclusão a que chegamos.
De qualquer maneira, acho que convém ter destrutores assim para quando fechar o programa, guardar automáticamente.
Se acharem que é melhor ter a opção de guardar enquanto o programa corre, podemos implementar uma função assim também.

@jvdcf
Copy link
Owner

jvdcf commented Oct 26, 2023

É importante fazermos a segunda parte do parser, obrigado por já teres adiantado isso! Agora, temos de perguntar primeiro se queremos salvar apenas antes de fechar o programa ou a partir de um comando save() ou algo do género. No segundo caso, como queremos que ainda esteja em memória, é melhor serem métodos da classe em vez de desconstrutores.

Estive a falar com o Gui sobre isso e acho que os dois concordamos que se guardaria no final, mas, para ser sincero, não me lembro a 100% da conclusão a que chegamos.
De qualquer maneira, acho que convém ter destrutores assim para quando fechar o programa, guardar automáticamente.
Se acharem que é melhor ter a opção de guardar enquanto o programa corre, podemos implementar uma função assim também.

Se já houve essa discussão, então perfeito! 🥰
Por mim pode ficar assim agora, e depois, se sentirmos necessidade, passamos o código para uma função e o desconstrutor chama essa função.

@jvdcf jvdcf merged commit e093055 into main Oct 26, 2023
@jvdcf jvdcf deleted the destructors branch October 26, 2023 06:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants