A ideia toda está errada, melhorei algumas coisas e deixei outras de lado.
Se existe uma entidade conta bancária deveria criar um tipo para lidar com isto, não deve lidar com os membros disto isoladamente como se fossem coisas isoladas.
Usei nomes mais significativos e dentro do padrão normal de nomenclatura, mantendo tudo em minúsculo e usando o vetor no plural para deixar claro que ali tem vários elementos.
Inicializei o vetor na main()
e liberei a memória no final, ainda que neste caso não precise porque ele está acabando mesmo.
Fiz um controle do tamanho de forma mais simples, evite passar coisas por referência sem necessidade. E usei o tamanho como gatilho para encerrar o cadastro, no código original era infinito.
Usei o realloc()
para mudar a alocação de local. Isto não é o ideal, mas é melhor que ir alocando novos vetores e abandonar sujando a memória o que criaria problemas futuros. Eu mantive acrescendo de um em um, mas isto não é correto, deveria alocar um número de itens mínimo inicial e ir pelo menos dobrando de tamanho a cada vez que faz a relocação, mas precisaria fazer a relocação só quando a capacidade estiver toda preenchida, senão poderia ser até pior. Como já tem tamanho e capacidade o mais correto seria ter um tipo que abstraísse o vetor de contas com esses elementos, aliás isso é o que se faz em código real, eu só não quis mudar demais o código.
Quero lembrar que float
não é adequado para guardar valor monetário, mas deixei por ser um exercício e iriam complicar fazer certo. Mas tenha isso em mente quando for fazer algo real.
Dei uma limpada no código. Não é o ideal, para para exercício acho que está bom pra ter uma ideia.
O código provavelmente não deveria ser digitado e sim pego de uma fonte atômica.
#include <stdio.h>
#include <stdlib.h>
typedef struct {
int codigo;
float saldo;
} Conta;
int ler_dados() {
int cod;
printf("Digite o codigo:");
scanf("%d", &cod);
return cod;
}
int inserir(Conta *contas, int tamanho) {
int codigo = ler_dados();
if (codigo == 0) return tamanho;
contas = realloc(contas, sizeof(int) * (tamanho + 1));
contas[tamanho].codigo = codigo;
contas[tamanho].saldo = 0;
return ++tamanho;
}
int main (void) {
Conta *contas = malloc(sizeof(int));
int tamanho = 0;
while (1) {
int retorno = inserir(contas, tamanho);
if (retorno == tamanho) break;
tamanho = retorno;
}
printf("Vetor\n");
for (int i = 0; i < tamanho; i++) printf("%d\n", contas[i].codigo);
free(contas);
}
Coloquei no GitHub para referência futura.
Há um problema neste código, veja mais em Problema com alocação dinâmica - realloc().