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'`.
-
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.struct bvalores acoes[QACOES];
-
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).int contador = 0;
-
Locales são um assunto bem avançado e não quero entrar em detalhes. Dito issosetlocale(LC_ALL, "Portuguese");
LC_ALL
é uma marreta poderosa (melhor evitada), ePortuguese
não é portável. Além disso, acredito que setar o locale uma única vez seja suficiente para sua aplicação. -
Tinha um errinho na conta que produzia variações negativas para ações em alta. Tive que inverter abertura e fechamento.acoes->variAcao = (acoes->valorFechamento - acoes->valorAbertura) / acoes->valorAbertura;
-
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//*(acoes + contador) = (struct bvalores*)malloc(1 * sizeof(struct bvalores)); registros(&acoes[contador]);
registros
. -
Aqui ocorreram várias pequenas modificações.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"); }
- 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.
- Troquei
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.