If statments [C++]

[Sidoh]

But it's a bad habit, and something that somebody learning C should be doing right away. 

I think wasting time is more of a bad habbit, but whatever floats your boat. ^^

[AntiVirus]

When I put  } elseif(change == "kilobyte") {

Then I get this error.

error C2065: 'elseif' : undeclared identifier

Here is my code again:

Code Select Expand

#include <iostream>
using namespace std;
int main()
{

char filetype;
char change[10];
float byte = 8;
float megabyte = 1048576;
float gigabyte = 1072741824;

cout <<"This program is designed to convert byte, kilobyte, megabyte, and gigabyte into different file sizes." << endl;
cout <<endl;
cout <<"What file size are you wanting to be converted?"<<endl;
cin  >>filetype;
if ( filetype == byte) {
cout <<"What do you want to change it into?"<<endl;
cin >> change;
}
elseif(change == "kilobyte");  {
cout << "1024 bytes in a Kilobyte"<<endl;
}


return 0;



}


I am confused. :(

[Blaze]

There isn't an elseif in C++, that I know of..

[AntiVirus]

That's what I thought, but Joe seems to think there is.

1) You spelled "a lot" wrong.
2) See Sidoh's comment about logic/assignment.
3) Tab your code buddy!
4) Heres the code, tabbed and fixed (read comments for more stuff)

cout << "What is your favorite icecream?" << endl; // ew at not using spaces there =p
cin >> icecream;
if (icecream == "strawberry") { // You want that to be in quotes, because your using a string constant instead of a defined constant
cout << "Really, how many servings do you eat per day?" << endl;
cin >> servings;
if (servings == "alot") {
// Always use brackets. They keep stuff in line and solve a lot of problems =)
// Also, see my above comment about string constants
cout >> "Your fat" >> endl;
} elseif(servings == "small") {
// You could simply change this to an else if you wanted, but this allows you to add others, such as medium.
// Again, see my comment about string constants.
cout >> "Good" >> endl;
}
}
return 0;
}

[Sidoh]

That's what I thought, but Joe seems to think there is.

Joe is a VB nub (jk joe)

[Blaze]

Try using Select instead.

[Sidoh]

Try using Select instead.

That wouldn't be logical in this situation.  Besides, C doesn't have Select, it has switch.  :)

[Blaze]

Ugh... shh!

[AntiVirus]

There is a "else if (blah = blah) thing.  Just elseif can't be together.

[Sidoh]

Ugh... shh!

lol

There is a "else if (blah = blah) thing. Just elseif can't be together.

That's not really a special operator, though.  doing:

Code Select Expand

if(0) { ... } else if(1) { ... } is the same thing as:

Code Select Expand


if(0) {
   ...
} else {
   if(1) {
      ...
   }
}

[Joe]

Quit being mean to me because I don't know the difference between C and C++. =p

Anyhow, I almost fixed it again. I tested it too.

Code Select Expand

#include <iostream>
using namespace std;

int main() { //I always put the start bracket on the same line, oh well. =)
  char filetype;
  char change[10];
 
  float byte = 8;
  float megabyte = 1048576;
  float gigabyte = 1072741824;

  cout << "This program is designed to convert byte, kilobyte, megabyte, and gigabyte into different file sizes." << endl;
  cout << endl;
  cout << "What file size are you wanting to be converted?" << endl;
  cin  >> filetype;
  switch(filetype) { //use switch (vb's select case) instead of if, because if == ugly
    case "byte": { //again, your using a string constant, not a variable, so it needs to be in quotes
      cout << "What do you want to change it into?"<<endl;
      cin >> change;
      switch(change) {
        case "kilobyte": {
          cout << "1024 bytes in a Kilobyte" << endl;
        } //end byte>>kilobyte case
        default: {
          cout << "Type not recongnized." << endl;
        } //end byte>?? case
      } //end switch for byte>>type
    } //end byte case
    default: {
      cout << "Type not recongnized." << endl;
    } // end ?? case
  } //end switch for type
  return 0xDEADBEEF; //ok, thats just leet..
}

Code Select Expand

joe@JoeMomma:~/dev/cpp/helloworld $ gcc Main.cpp
Main.cpp: In function `int main()':
Main.cpp:17: error: case label does not reduce to an integer constant
Main.cpp:20: error: switch quantity not an integer

[iago]

Using a switch there is a pretty bad use for a switch statement. 

"else if" is want you want. 

This forum probably doesn't have the best people to ask about C, I've been noticing, so take whatever you learn here with a grain of salt. 

[AntiVirus]

Lol, thanks Ron for the advise. :)  And thanks Joe, Sidoh, and Blaze for the help. :D

[Eric]

Switch can't make string-based comparisons.

[Newby]

Switch can't make string-based comparisons.

You figured "not an integer" would clue him in. Guess not!