Bug em programa alocação dinâmica de vetores

[ESMA 0]

Pessoal, vocês podem me ajudar a corrigir o bug?

"Faça um programa em C++ que leia números do teclado e os armazene em um vetor alocado dinamicamente. O usuário irá digitar uma sequência de números, sem limite de quantidade. Os números serão digitados um a um e, sendo que caso ele deseje encerrar a entrada de dados, ele irá digitar o número ZERO. Os dados devem ser armazenados na memoria deste modo."

Eu fiz esse código, mas ele buga quando digito 0:

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

int main () {

    int *vet;
    int j, n = 0;

    vet = (int *)malloc(1 * sizeof(int));

    while (vet != 0) {
        printf("\nDigite um número: ");
        scanf("%i",&vet);
        n++;
    }

    for (j = 1; j < n; j++) {
        printf("%i ",vet[j]);
    }
    printf("\n");
    return 0;
    free(vet);
}

[Nefisto 0]

Amigo, esta muito errado.

1- vet é um endereço de memoria, comparar ele com o decimal zero, seria como comparar ele com NULL e nao é o que você quer, substitua.

- vet != 0 por *vet != 0

Desrefencie antes de comparar.

2-Voce esta alocando 1*sizeof(int), ou seja, 4 bytes para armazer um inteiro, e você coloca isso dentro de um laço onde a condição de parada é dada pelo usuario, você precisa realocar o espaço dentro do loop, oq eu te recomendo é o seguinte, por malloc ser uma operação mto custoza, aloca inicialmente um espaço maior como - 100*sizeof(int)-, e se o seu n passar de 99 você da um realloc em mais 100 espaços.

3- Por vet ser um ponteiro, e scanf esperar um ponteiro para guardar o valor inserido pelo usuario, a intrução "...&vet" nao faz sentido, ja q o & serve para pegar o endereço de algo na memoria, voce esta pegando o endereço de um endereço, vai gerar erros sutis, se você quer usar o & entao faça "...&vet[n]" ou para nao usar o & você usa o vet solto, "...vet);" mas nesse ultima caso você precisa incrementar o ponteiro com vet++, e nao se eskeca de salvar a posição inicial do ponteiro se for seguir por este ultimo caminho.

4- você usa um while(*vet != 0), sem inicializar vet, entao se por um acaso o lixo de memoria q esta la quando você alocou o espaço for igual a 0, o laço nunca vai acontecer, use ou um do/while, ou inicialize valores para o endereço para evitar este erro.

5-Nao tem pq j começar em 1, e se você optar por usar um do/while é necessario q seja usado j<n-1, ja q ele vai passar pelo while uma vez a mais no 0

6- O seu free esta abaixo do return, é um erro comum, o return do main encerra o programa, logo sua memoria nunca sera liberada, coloque-a uma linha acima

Acho q é isso, segue o codigo corrigido,

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

int main () {

    int *vet;
    int size = 1;
    int j, n = 0;

    vet = (int*)malloc(size * 100 * sizeof(int));

    do
    {
        printf("Digite um numero:\n");
        scanf("%d", &vet[n]);
	n++;
		
	if(n / 100 == size)
	{
	   size++;
	   realloc(vet, size * 100 * sizeof(int));
	}
    }while(vet[n-1]);

    for (j = 0; j < n-1; j++) 
    {
        printf("%d ", vet[j]);
    }
	
    printf("\n");
    free(vet);
	
    return 0;
}

[Wilkyn Fernandes Taborda 0]

Os dois programas acima estão usando a biblioteca string.h mesmo não fazendo nenhuma chamada de função à ela. É desnecessário. A linha de código #include<string.h> pode ser removida por que linhas de código desnecessárias como essa apenas fazem com que o código-fonte fique sujo e poluído. Isso que eu digo é chamado de "limpar o código". É tirar o que não serve e deixar somente o que o programa necessita. Abaixo o programa funcionando: 

#include<iostream.h>

struct no_num{
  int num;
  no_num *prox;
}; 

int main(){
  no_num *primeiro=new(no_num);
  no_num *ultimo=primeiro;
  int num=1;
  while(num!=0){    
    cout << "Digite algum numero. 0 para parar:"; 
    cin >> num;
    ultimo->num=num;
    ultimo->prox=new(no_num);
    ultimo=ultimo->prox;
    ultimo->prox=NULL;
  }
  cout << endl << "Numeros digitados:";   
  for(no_num *temp=primeiro;temp->prox->prox!=NULL;temp=temp->prox)
    cout << endl << temp->num;
  
  system("pause");
}