Expected primary expression before ';',"int" comes because...?

[yoyo22]

#include<cstdio>
#include<cstdiolib>
#include<iostream>
using namespace std;
int main(int nNumberofArgs, char* pszArgs[])
 
{
    cout<<"Enter your account type";
    int FD;
    if ("FD")
    {
         int p=10000000;
         float r=10;
         int t;
         cin>>t;         
         cout<<p*r*t/100;
 
    }
    int RD;
    else ("RD")    {
         int p=1000000;
         float r=10;
         int t;
         cin>>t;     
         cout<<p*r*t/100;
    }
    system("PAUSE");
    return 0;
}

[weaknessforcats]

It's probably right here:

 cout<<int si=p*r*t/100;

You can't create a variable this way. You calling an operator<< function. This one has an ostream& on the left and some argument on the right. You will need to review which types are available as the right hand argument.

If all you want to see is p*r*t/100 then use that and remove the int si=.

BTW: Please use CODE tags on future posts.

[yoyo22]

Thank you. I will use the code tags in my future posts.

[divideby0]

this doesn't look like either; I'd think it would lead to illegal else without if or such.

what is being tested if("FD")? I've not seen that before.

if("FD") {
   ...
} 
 
int RD;
 
else("RD") {
   ...
}

if the result of p * r * t / 100 isn't what you'd expect, use parenthesis around the operators.

[weaknessforcats]

"FD" is a string literal. What is being tested is the address of te literal. This will always be true since the literal exists.

I expect "FD" is a bug since there is an int named FD and probably should have ben coded:

if (FD)
{
etc...

In this case you are testing the contents of the variable FD to be zero or not. If zero, the if will be false.

On the other hand, nothing is ever put into FD so the whole program makes no sense.

[divideby0]

Thank you.

I couldn't figure out what if("FD") was supposed to be; is that ever useful? The int was more of what I'd expect... that is, testing a declared type.

[weaknessforcats]

It may be useful but it's bad practice to have hard-coded values in your code. A better choice is to use a singleton object. The object is created before the program starts and stays around until the program completes. The creation process could read a disc file to get the "FD". That way you could change the value by changing the disc file rather than recompiling the code and sending copies to everyone that has the program.

Singleton obj;  //reads disc file
int main()
{
   string data;
etc.....
 
   if (data == obj.data())
   {
      //data equal to the disc file value here
   }
etc...
}

As you see, the code is now independent of the value "FD".

A not as good approach is to use a macro. Here you define the macro as "FD". This approach keeps the "FD" in one location but to change it you need to recompile the code:

#define  VALUE  "FD"

This is put in a header file, say macro.h.

Then you:

#include <macro.h>
 
int main()
{
   string data;
etc.....
 
   if (data == VALUE)
   {
      //data equal to the macro definition here
   }
etc...
 

[johny10151981]

Interestingly the grammatical error you have made is

 if(condition)
 {
  statements;
 }
  statements; // this is the line where you have made your grammatical error
 else
 {
  statements;
 }
 

in your original code line 19

Now first you ask account type:
but you didn't write any statement to read the account type

instruction:

char account_type[4]
cin>>account_type;
 

then you defined integer type FD. FD is a variable without any assigned value.

You don't need FD or RD

What you need is in if condition compare whether the account is FD or RD.
Instruction:

if(strcomp(account_type,"FD")==0)
{
  Statements;//
}
else if(strcomp(account_type,"RD")==0)
{
 Statements;
}
else 
{
 cout<<"Invalid Account Type Inputted"<<endl;
}