Skip to content
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

Constante com declaracao tipos #555

Conversation

Aristidescosta
Copy link
Contributor

Atualização: Constante com definição de tipos

Descrição

Esta PR vem dar solução a Issue Criar constante com definição de tipos #453
Atualização do código para remover a verificação de tipos de forma "manual" escrita.
Para isso, adicionei dois arquivos com os nome de comum e lista em fontes\tipos-de-dados

Capturas de ecrã

Capture

Exemplo de Utilização

import tiposDeDados from '../tipos-de-dados/comum'

if (typeof valorMinimo !== tiposDeDados.NUMERO || typeof valorMaximo !== tiposDeDados.NUMERO) {
   return Promise.reject(
      new ErroEmTempoDeExecucao(this.simbolo, 'Os dois parâmetros devem ser do tipo número.')
   );
}

@samuelrvg
Copy link
Member

@Aristidescosta Consegue já implementar o uso desses tipos nos pontos onde esta manual a definição?

@Aristidescosta
Copy link
Contributor Author

@Aristidescosta Consegue já implementar o uso desses tipos nos pontos onde esta manual a definição?

Sim, consigo.

@Aristidescosta
Copy link
Contributor Author

Aristidescosta commented Nov 23, 2023

@samuelrvg
O mesmo deve ser aplicado neste exemplo??

['vetor', 'qualquer[]', 'inteiro[]', 'texto[]']

@samuelrvg
Copy link
Member

@samuelrvg O mesmo deve ser aplicado neste exemplo??

['vetor', 'qualquer[]', 'inteiro[]', 'texto[]']

Sim.

@Aristidescosta
Copy link
Contributor Author

@samuelrvg A implementação do uso dos tipos criado será em todos os arquivos ou em arquivos específicos?

@samuelrvg
Copy link
Member

samuelrvg commented Nov 23, 2023

@samuelrvg A implementação do uso dos tipos criado será em todos os arquivos ou em arquivos específicos?

@Aristidescosta A princípio faz somente para esse arquivo aqui https://github.com/DesignLiquido/delegua/blob/principal/fontes/interpretador/interpretador-base.ts

@samuelrvg samuelrvg merged commit 60505aa into DesignLiquido:principal Nov 23, 2023
1 check 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