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

acrescentando novo extractor para interpretar campos sem a expecificação de decimal #4

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

rodrwankenobi
Copy link
Contributor

Acrescentei um novo extractor, que consegue interpretar linhas no formato como o exemplo abaixo:
03 VALOR PIC S9(10)V USAGE COMP-3.
Neste exemplo, ele consegue obter o tamanho 10 e seta o valor 0 para a propriedade decimals.

A2

class FieldSignalNumeric1Decimals3(AbstractFieldExtractor):
def extract(self, book_item: BookItem):
m = re.search(r"^S9(([0-9]+))V$", book_item.format)
if m:
book_item.type = "NUMERIC"
book_item.size = int(m.group(1))
book_item.decimals = 0
return book_item
else:
return super().extract(book_item)

…ção do decimal, por exemplo: PIC S9(10)V USAGE COMP-3. Neste exemplo ele consegue obter o tamanho 10 e 0 na propriedade decimals
@jrperin
Copy link
Owner

jrperin commented Aug 21, 2023

Rodrigo, bacana as alteracoes que feitas no projeto.
Dei uma olhada e fiz algumas alteracoes apenas para manter a documentacao do projeto.
Seguindo a ordem de encadeiamento das classes. Como a nova classe eh uma classe do tipo SignedDecimals, reordenei para que ela ficasse agrupada.

Criei um MR para a sua branch.
#6

@jrperin jrperin merged commit 28a7eef into jrperin:master Aug 21, 2023
jrperin added a commit that referenced this pull request Aug 21, 2023
acrescentando novo extractor para interpretar campos sem a expecificação de decimal
rodrwankenobi pushed a commit to rodrwankenobi/cobol-copybook.jsonifier that referenced this pull request Aug 22, 2023
acrescentando novo extractor para interpretar campos sem a expecificação de decimal
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