Skip to main content
incluí sugestão de solução
Fonte Link
marcus
  • 2,2mil
  • 1
  • 12
  • 16

Veja a linha:

cin >> n;

A variável n é do tipo char*, então o operador >> pensa que há um array alocado ali, e lê coisas que não cabem no buffer ("-1" tem 2 chars). Nem há buffer, porque você alocou dinamicamente apenas 1 char (para treinar, OK, num programa de verdade, parece sem propósito).

Sugiro que troque então por:

cin >> *n;

ou

scanf("%c", n);

Se digitar -1, a linha acima (qualquer uma) lerá apenas o "-", sem gravar fora do espaço alocado.

Outra coisa: &*dinheiro é meio inútil. Os operadores & e * se cancelam (na ausência de operator overloading), e seria o mesmo que escrever apenas dinheiro.

E uma importante lição no mundo do C e C++: o fato do compilador aceitar está muito vagamente relacionado com o fato do programa estar correto.

Veja a linha:

cin >> n;

A variável n é do tipo char*, então o operador >> pensa que há um array alocado ali, e lê coisas que não cabem no buffer ("-1" tem 2 chars). Nem há buffer, porque você alocou dinamicamente apenas 1 char (para treinar, OK, num programa de verdade, parece sem propósito).

Outra coisa: &*dinheiro é meio inútil. Os operadores & e * se cancelam (na ausência de operator overloading), e seria o mesmo que escrever apenas dinheiro.

E uma importante lição no mundo do C e C++: o fato do compilador aceitar está muito vagamente relacionado com o fato do programa estar correto.

Veja a linha:

cin >> n;

A variável n é do tipo char*, então o operador >> pensa que há um array alocado ali, e lê coisas que não cabem no buffer ("-1" tem 2 chars). Nem há buffer, porque você alocou dinamicamente apenas 1 char (para treinar, OK, num programa de verdade, parece sem propósito).

Sugiro que troque então por:

cin >> *n;

ou

scanf("%c", n);

Se digitar -1, a linha acima (qualquer uma) lerá apenas o "-", sem gravar fora do espaço alocado.

Outra coisa: &*dinheiro é meio inútil. Os operadores & e * se cancelam (na ausência de operator overloading), e seria o mesmo que escrever apenas dinheiro.

E uma importante lição no mundo do C e C++: o fato do compilador aceitar está muito vagamente relacionado com o fato do programa estar correto.

Fonte Link
marcus
  • 2,2mil
  • 1
  • 12
  • 16

Veja a linha:

cin >> n;

A variável n é do tipo char*, então o operador >> pensa que há um array alocado ali, e lê coisas que não cabem no buffer ("-1" tem 2 chars). Nem há buffer, porque você alocou dinamicamente apenas 1 char (para treinar, OK, num programa de verdade, parece sem propósito).

Outra coisa: &*dinheiro é meio inútil. Os operadores & e * se cancelam (na ausência de operator overloading), e seria o mesmo que escrever apenas dinheiro.

E uma importante lição no mundo do C e C++: o fato do compilador aceitar está muito vagamente relacionado com o fato do programa estar correto.