Return Styles: Pseud0ch, Terminal, Valhalla, NES, Geocities, Blue Moon. Entire thread

some C++ question

Name: that nigger 2009-04-02 2:29

so /prog/rammers can you tell me why this only returns
x=1 y=1
I'm trying to have it return the values that equal 133745639

heres the code:

#include <cstdlib>
#include <iostream>

using namespace std;

int main(int argc, char* argv[])
{
unsigned long int x = 1;
unsigned long int y = 1;
unsigned long int z;


while ( z != 133745639)
{
z = x * y ;

if ( z = 133745639 )
{
cout << "x=" << x << "y=" << y << endl;
}
else
x++;
}
system("PAUSE");
return 0;

}

Name: Anonymous 2009-04-02 6:40

What is "133745639" how would another programmer reading this code understand what that value is there for? You should avoid the use of so called "magic constants" and declare them as a constant value with appropriate variable name earlier in the file.

Constants don't have variable names. That's why they're constant. They don't vary.

Newer Posts
Don't change these.
Name: Email:
Entire Thread Thread List