scanf and c++

[Xavier Serrand]

With GCC version 3.4.4 (mingw special) , following code gives stange results
:


void encapsulation_c(void)
{
double x, y;
std:rintf("Calcul de moyenne\n");
std:rintf("Entrez le premier nombre : ");
std::scanf("%f", &x);
std:rintf("\nEntrez le deuxieme nombre : ");
std::scanf("%f", &y);
//y = 3;
std:rintf("\n\nLa valeur moyenne de %8.2f et de %8.2f est %8.2f\n",
x, y, ((x+y)/2));
}

output is :

Calcul de moyenne
Entrez le premier nombre : 1.0

Entrez le deuxieme nombre : 2.0


La valeur moyenne de 0.00 et de 0.00 est 0.00


Some idea ?

 

[Robert Bauck Hamar]

With GCC version 3.4.4 (mingw special) , following code gives stange
results
:


void encapsulation_c(void)
{
double x, y;
std:rintf("Calcul de moyenne\n");
std:rintf("Entrez le premier nombre : ");
std::scanf("%f", &x);

std::scanf("%lf", &x);

std:rintf("\nEntrez le deuxieme nombre : ");
std::scanf("%f", &y);

std::scanf("%lf", &y);

//y = 3;
std:rintf("\n\nLa valeur moyenne de %8.2f et de %8.2f est %8.2f\n",
x, y, ((x+y)/2));
}

output is :

Calcul de moyenne
Entrez le premier nombre : 1.0

Entrez le deuxieme nombre : 2.0


La valeur moyenne de 0.00 et de 0.00 est 0.00


Some idea ?

Read the manual when using printf/scanf. Googling for scanf says exactly
what's needed. Also, the warning

warning: format '%f' expects type 'float*', but argument 2 has
type 'double*'

is also useful.

 

[James Kanze]

With GCC version 3.4.4 (mingw special) , following code gives stange results
:

void encapsulation_c(void)
{
double x, y;
std:rintf("Calcul de moyenne\n");
std:rintf("Entrez le premier nombre : ");
std::scanf("%f", &x);
std:rintf("\nEntrez le deuxieme nombre : ");
std::scanf("%f", &y);
//y = 3;
std:rintf("\n\nLa valeur moyenne de %8.2f et de %8.2f est %8.2f\n",
x, y, ((x+y)/2));
}

output is :

Calcul de moyenne
Entrez le premier nombre : 1.0

Entrez le deuxieme nombre : 2.0

La valeur moyenne de 0.00 et de 0.00 est 0.00

Some idea ?

The code has undefined behavior, so nothing can really be
considered strange.

Forget about scanf. It's very difficult to use correctly to
begin with, and very, very fragile, leading to absolutely
unmaintainable code. Something like:

double x, y;
std::cout << "Calcul de moyenne" << std::endl ;
std::cout << "Entrez le premier nombre : " ;
std::cin >> x ;
std::cout << "\nEntrez le deuxieme nombre : " ;
std::cin >> y ;

Does the trick nicely (except that it lacks error handling), and
is robust in face of change. (It works equally well, regardless
of whether x and y are double, float or even int.)