Skip to main content
adicionou 409 caracteres ao conteúdo
Fonte Link
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109
#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", &acoesacoes->codigoAcao);
    printf("Digite a área de atuação da empresa: ");
    scanf("%s", &acoesacoes->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;
}
  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'.

  2. 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.

  3. 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).

  4. 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.

  5.  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);
    

    Em C vetores decaem em ponteiros, ou seja, algo como acoes->variAcao = (acoes->valorFechamento - acoes->valorAbertura) /char acoes->valorAbertura;codigoAcao[6] é visto como um char* apontando para o primeiro elemento do vetor. Dessa maneira, ao ler uma string não é necessário usar &.

  6.   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.

  7.  //*(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.

  8.  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 ações não inicializadas ou fora dos limites do vetor.
    • A lógica para acessar cada ação também foi modificada conforme itens 2 e 67.
    • Várias pequenas mudanças nas strings de formatação.
#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;
}
  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'.

  2. 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.

  3. 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).

  4. 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.

  5. 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.

  6.  //*(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.

  7.  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 ações não inicializadas ou fora dos limites do vetor.
    • 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.
#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;
}
  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'.

  2. 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.

  3. 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).

  4. 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.

  5.  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);
    

    Em C vetores decaem em ponteiros, ou seja, algo como char codigoAcao[6] é visto como um char* apontando para o primeiro elemento do vetor. Dessa maneira, ao ler uma string não é necessário usar &.

  6.   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.

  7.  //*(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.

  8.  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 ações não inicializadas ou fora dos limites do vetor.
    • A lógica para acessar cada ação também foi modificada conforme itens 2 e 7.
    • Várias pequenas mudanças nas strings de formatação.
removeu 2 caracteres do conteúdo
Fonte Link
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109

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'. 2. 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.
  1.   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).

  2. 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.
  1.  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.
  1.  //*(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`.
  1.  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 ações não inicializadas ou fora dos limites do vetor.
* 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.
  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'.

  2. 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.

  3. 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).

  4. 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.

  5. 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.

  6.  //*(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.

  7.  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 ações não inicializadas ou fora dos limites do vetor.
    • 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.

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'. 2. 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.
  1.   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).

  2. 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.
  1.  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.
  1.  //*(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`.
  1.  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 ações não inicializadas ou fora dos limites do vetor.
* 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.
  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'.

  2. 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.

  3. 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).

  4. 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.

  5. 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.

  6.  //*(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.

  7.  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 ações não inicializadas ou fora dos limites do vetor.
    • 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.
removeu 2 caracteres do conteúdo
Fonte Link
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109

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'. 2. struct bvalores acoes[QACOES];

Você precisanão sempreprecisa deixarde um espaço extra paravetor ode terminadorponteiros depara *string*ações. e.g.,Como ao *string*número `"PETR4"`de éações representadaé pelosfixo, caracteresum `'P'`,simples `'E'`,vetor `'T'`,de `'R'`,ações `'4'`,é `'\0'`suficiente.
  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 ações não inicializadas ou fora dos limites do vetor.
    • 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.
  1.   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).

  2. 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.
  1.  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.
  1.  //*(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`.
  1.  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 ações não inicializadas ou fora dos limites do vetor.
* 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.

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 ações não inicializadas ou fora dos limites do vetor.
    • 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.

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'. 2. 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.
  1.   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).

  2. 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.
  1.  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.
  1.  //*(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`.
  1.  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 ações não inicializadas ou fora dos limites do vetor.
* 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.
adicionou 27 caracteres ao conteúdo
Fonte Link
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109
Carregando
Fonte Link
Anthony Accioly
  • 21,2mil
  • 2
  • 54
  • 109
Carregando