Skip to content

Corrige divisões por zero em cálculos de preços#47

Merged
albertomandlate merged 1 commit intomainfrom
develop
Aug 20, 2025
Merged

Corrige divisões por zero em cálculos de preços#47
albertomandlate merged 1 commit intomainfrom
develop

Conversation

@albertomandlate
Copy link
Copy Markdown
Contributor

A lógica de cálculo da propriedade PercentagemDesconto foi ajustada para evitar divisão por zero, retornando 0m quando PrecoTotalComImpostos é zero.

As propriedades PrecoUnitarioComImpostos e PrecoUnitarioSemImpostos também foram modificadas para retornar 0m quando Quantidade é zero, prevenindo assim exceções de divisão por zero.

A lógica de cálculo da propriedade `PercentagemDesconto` foi ajustada para evitar divisão por zero, retornando 0m quando `PrecoTotalComImpostos` é zero.

As propriedades `PrecoUnitarioComImpostos` e `PrecoUnitarioSemImpostos` também foram modificadas para retornar 0m quando `Quantidade` é zero, prevenindo assim exceções de divisão por zero.
@albertomandlate albertomandlate self-assigned this Aug 20, 2025
Copilot AI review requested due to automatic review settings August 20, 2025 07:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fixes division by zero errors in price calculation properties by adding guard conditions to prevent runtime exceptions when dividing by zero values.

  • Added zero-check guards to prevent division by zero in percentage and unit price calculations
  • Modified PercentagemDesconto to return 0 when PrecoTotalComImpostos is zero
  • Updated PrecoUnitarioComImpostos and PrecoUnitarioSemImpostos to return 0 when Quantidade is zero

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/SAF-T.Mozambique/Models/FicheiroSAFT.cs
@albertomandlate albertomandlate merged commit 7bf4baf into main Aug 20, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants