Originally Posted by anon
i know it's somewhat frowned upon, but from the dirth of gross errors, it seemed pretty likely to me that he was too lost to even understand how to ask questions educatedly.
and i don't know about you, but i learn a lot more from the examples in the documentation than the verbose english discussions.
if he doesn't learn anything, well then he'll fail then class anyway.
Hrmmm...
Ubuntu Desktop
GCC/G++
Geany (for quick projects)
Anjuta (for larger things)
Proper form is either "edjumucated" or "edjucated-like". The latter can also be adorned with the adverb "real".
lol i was originally going to say "intelligently" but thought that sounded a bit harsh.
i cant get it to store females ages over 21 in the variable adult_Females same for adultMale
help!!! please!!
Code:#include<iostream> int main() { int age, males=0, females=0, adult_Males=0, adult_Females=0, minors=0,minorage_Sum=0, code, count=0,max; while(count++<max && code !=0) { max=code+age; std::cout<< "Please enter the gender code 1= Female, 2= Male, 0= Exit" <<std::endl; std::cin>>code; std::cout<< "Please enter person age" <<std::endl; std::cin>>age; } if( age<21) { males=males+1 females=females+1 } if(males, age>=21) { adult_Males++; } if(age>=21); { adult_Females++; } { std::cout<< "Statistics:" <<std::endl; std::cout<< "Number of Females over 21"<<" "<<adult_Females<<std::endl; std::cout<< "Number of Males over 21"<<" "<<adult_Males<<std::endl; std::cout<< "Total People"<<" "<<minors+adult_Males+adult_Females<<std::endl; } if (minors>0) { minors=males+females; std::cout<< "Average of Minors:"<<(float)minorage_Sum/(float)males+females<<std::endl; } system("PAUSE"); return 0; }
I too, continue to try to solve the problem of finding and then keeping adult females. I try to put them in my closet but they keep breaking out and running off into the woods screaming "kidnapper!"
i thought as long as i dont reset the variable to 0 that it would continue to have input stored in it.
To get a better response, learn to format your code properly.
This is what it should look like (had to add two semicolon to make the formatter work properly):
Here's the warnings it produces:Code:#include<iostream> int main() { int age, males=0, females=0, adult_Males=0, adult_Females=0, minors=0,minorage_Sum=0, code, count=0,max; while (count++<max && code !=0) { max=code+age; std::cout<< "Please enter the gender code 1= Female, 2= Male, 0= Exit" <<std::endl; std::cin>>code; std::cout<< "Please enter person age" <<std::endl; std::cin>>age; } if ( age<21) { males=males+1; females=females+1; } if (males, age>=21) { adult_Males++; } if (age>=21); { adult_Females++; } { std::cout<< "Statistics:" <<std::endl; std::cout<< "Number of Females over 21"<<" "<<adult_Females<<std::endl; std::cout<< "Number of Males over 21"<<" "<<adult_Males<<std::endl; std::cout<< "Total People"<<" "<<minors+adult_Males+adult_Females<<std::endl; } if (minors>0) { minors=males+females; std::cout<< "Average of Minors:"<<(float)minorage_Sum/(float)males+females<<std::endl; } system("PAUSE"); return 0; }
All are important. If you use uninitialized variables then your program could do anything.test.cpp(26) : warning C4390: ';' : empty controlled statement found; is this the intent?
test.cpp(7) : warning C4700: uninitialized local variable 'max' used
test.cpp(7) : warning C4700: uninitialized local variable 'code' used
test.cpp(8) : warning C4700: uninitialized local variable 'age' used
The largest problem is that processing input happens after the loop that gets input. All this while I've been entering data and the program doesn't do anything with it!
I might be wrong.
Quoted more than 1000 times (I hope).Thank you, anon. You sure know how to recognize different types of trees from quite a long way away.
this is syntactically and logically wrong.Code:if(age,males) { adult_males++; }
your statements with max and count do not appear to be doing anything useful to you.
you need to think through the structure of your program before you start writing it.
No, it's syntactically fine but logically wrong.
odd usage. never seen it before. i don't know why you'd ever want to do that instead of explicitly using a logical and.
anyhow, @ OP:
you have a misconception about how boolean logic works.
things like ints can be evaluated in if statements , but the expression is evaluated as true if the argument (the int you're passing in) is >0, and false otherwise.
you seem to think the application intuitively knows that if code == 2, then males = true. it doesn't.
the expression you entered there is equivalent to:
which is not what you want to do.Code:if(males>0 && age>=21) { adult_males++; }
your program needs to differentiate between which adult_ variable to increment based on the value of code
It's syntactically fine in the sense that it will compile, but it's not doing anything near what you want it to do. Using the comma like that will invoke the comma operator, which will evaluate the first term, then the second term, then return the second term, ignoring the result of the first (except side effects).
You should be using combinations of || (or) and && (and) in your if statements.
But the bigger problem is that the variable code, which reads the gender from the user is not actually interpreted that way by the program.
It is too clear and so it is hard to see.
A dunce once searched for fire with a lighted lantern.
Had he known what fire was,
He could have cooked his rice much sooner.
It's not equivalent to logical AND, it just evaluates to the value of the rightmost operand.
so it's useless? no wonder i've never seen it used.
