Skip to main content
1 de 5
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109

Fiz várias pequenas modificações para fazer seu código rodar.

#include <stdio.h>
#include <stdlib.h>
#include <locale.h>
#include <string.h>

#define MAX 100
#define QACOES 5

struct Data 
{
    int dia;
    int mes;
    int ano;
};

struct bvalores
{
    char codigoAcao[6];
    char areaAtuacao[MAX];
    struct Data data;
    float valorAbertura = 0;
    float valorFechamento = 0;
    double variAcao;
};

struct bvalores acoes[QACOES];
int contador;

//recebe os dados digitados pelo usuário
void registros(struct bvalores* acoes)
{
    setlocale(LC_ALL, "Portuguese");

    printf("Digite o código da ação: ");
    scanf("%s", &acoes->codigoAcao);
    printf("Digite a área de atuação da empresa: ");
    scanf("%s", &acoes->areaAtuacao);
    printf("Qual o valor de aberta: ");
    scanf("%f", &acoes->valorAbertura);
    printf("Qual o valor de fechamento: ");
    scanf("%f", &acoes->valorFechamento);

    acoes->variAcao = (acoes->valorFechamento - acoes->valorAbertura) / acoes->valorAbertura;
    acoes->variAcao *= 100;

    printf("Data de lançamento: ");
    scanf("%d/%d/%d", &acoes->data.dia, &acoes->data.mes, &acoes->data.ano);
}

//registra cada ação com seus respectivos dados em uma posição da memória
void novoregistro()
{
    setlocale(LC_ALL, "Portuguese");

    if (contador < QACOES)
    {
        //*(acoes + contador) = (struct bvalores*)malloc(1 * sizeof(struct bvalores));
        registros(&acoes[contador]);
        contador++;
    }
    else
    {
        printf("Só é possível 5 registros. Limite excedido");
    }
}

char menu()
{
    setlocale(LC_ALL, "Portuguese");

    printf("\n");
    printf("Digite [I] para incluir um novo registro: \n");
    printf("Digite [R] para visualizar a variação das ações registradas: \n");
    printf("Digite [S] para sair do programa: \n");

    printf("\nQual a opção desejada: ");

    int opcao = getchar();
    int c;

    while ((c = getchar()) != '\n' && c != EOF)
    {
        opcao = c;
    }
    return opcao;   
}

//função que monta um relatório dos registro das açoes
void relatorio()
{
    setlocale(LC_ALL, "Portuguese");
    struct bvalores* acao;

    for (int i = 0; i < contador; i++)
    {
        acao = &acoes[i];
        printf("Ação da bolsa: %s  -  Area de atuação: %s\n", acao->codigoAcao, acao->areaAtuacao);
        printf("Data da operação: %02d/%02d/%04d\n", acao->data.dia, acao->data.mes, acao->data.ano);
        printf("\n");
        printf("Valor de abertura: R$ %.2f  -  Valor de fechamento: R$ %.2f\n", acao->valorAbertura, acao->valorFechamento);
        printf("Variãção do dia: %.2f%%", acao->variAcao);
        printf("\n");
    }
}

int main()
{
    char sopcao;

    do 
    {
        sopcao = menu();
        switch (sopcao)
        {
        case 'i':
        case 'I':
            novoregistro();
            break;

        case 'R':
        case 'r':
            relatorio();
            break;

        default:
            printf("Opção invalida");
            break;
        }
    } while (sopcao != 'S' && sopcao != 's');   

    //system("pause");
    return 0;
}

Veja funcionando no ideone.com

Tentei mexer o mínimo possível no seu código, mesmo quando ele fere algumas boas práticas (veja explicações abaixo):

1. char codigoAcao[6];

Você precisa sempre deixar um espaço extra para o terminador de *string*. e.g., a *string* `"PETR4"` é representada pelos caracteres `'P'`, `'E'`, `'T'`, `'R'`, `'4'`, `'\0'`.
  1.  struct bvalores acoes[QACOES];
    
    Você não precisa de um vetor de ponteiros para ações. Como o número de ações é fixo, um simples vetor de ações é suficiente.
  2.   int contador = 0;
    
    Enquanto não mexi na sua implementação, pois variáveis globais e estatísticas são inicializadas "zeradas", é boa prática sempre inicializar todas as variáveis (até para não esquecer de inicializar variáveis locais).
  3. setlocale(LC_ALL, "Portuguese");
    
    Locales são um assunto bem avançado e não quero entrar em detalhes. Dito isso LC_ALL é uma marreta poderosa (melhor evitada), e Portuguese não é portável. Além disso, acredito que setar o locale uma única vez seja suficiente para sua aplicação.
  4.  acoes->variAcao = (acoes->valorFechamento - acoes->valorAbertura) / acoes->valorAbertura;
    
    Tinha um errinho na conta que produzia variações negativas para ações em alta. Tive que inverter abertura e fechamento.
  5.  //*(acoes + contador) = (struct bvalores*)malloc(1 * sizeof(struct bvalores));
     registros(&acoes[contador]);
    
    Devido a mudança no item 2 não é mais necessário alocar memória dinamicamente. Em compensação se torna necessário passar o endereço da ação para a função registros.
  6.  for (int i = 0; i < contador; i++)
     {
         acao = &acoes[i];
         printf("Ação da bolsa: %s  -  Area de atuação: %s\n", acao->codigoAcao, acao->areaAtuacao);
         printf("Data da operação: %02d/%02d/%04d\n", acao->data.dia, acao->data.mes, acao->data.ano);
         printf("\n");
         printf("Valor de abertura: R$ %.2f  -  Valor de fechamento: R$ %.2f\n", acao->valorAbertura, acao->valorFechamento);
         printf("Variãção do dia: %.2f%%", acao->variAcao);
         printf("\n");
     }
    
    Aqui ocorreram várias pequenas modificações.
    • Troquei <= por < para evitar acessar índices não inicializadas.
    • A lógica para acessar cada ação também foi modificada conforme itens 2 e 6.
    • Várias pequenas mudanças nas strings de formatação.

OBS: Não usei printf_s e scanf_s pois eles não estão disponíveis na glibc (na verdade, nunca vi essas funções sendo usadas fora do mundo Microsoft). Pesquisando sobre o assunto, me parece que há certa controvérsia em relação a TR 24731-1, dito isso, se portabilidade não for prioridade fica a seu critério usar as versões safe.

system("pause"); também só funciona no Windows e não faz muito sentido para um programa que já pausa para ler do teclado em outros pontos.

Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109