Simple IF Statements

Question

0.00/5 (No votes)

See more:

Hello Developers. !!
My code isn't working as the way i want, , ,
why the COUT statement is not executed. . .

Any Suggestions. ??

C++

#include <iostream>

using namespace std;


int main ()
{
	char isEmployer = 'E';

	char hasLocation = 'L';

	int hasAge = 0;

	int hasSalary = 0;

	cout << "enter details to get loan" << endl;

	cout << "Employer?" << endl;

	cin >> isEmployer;

	cout << "Location?" << endl;

	cin >> hasLocation;

	cout << "Salary" << endl;

	cin >> hasSalary;

	if (isEmployer == 'E')

		if (hasLocation=='L')

			if(hasSalary > 50000)

            cout << "You Are Allowed To Get The Loan" << endl;

		else
			cout << "Not Allowed To Get The Loan" << endl;

		return 0;
}

Posted 20-Nov-12 2:28am

Usman Hunjra

Add a Solution

Comments

Usman Hunjra 20-Nov-12 8:30am

i think there should be curly brackets within if . . .
But I want some more explanation . . .

MT_ 20-Nov-12 8:40am

Yes, you need curly bracket and leave last line out. See below

Richard MacCutchan 20-Nov-12 8:38am

Works fine for me when I put the correct values in. What are you typing in answer to the questions?

4 solutions

Solution 4

In the code it is taken Salary=0 but the condition checked is Salary>5000.So control is going to the statement
cout<<"Not Allowed To Get The Loan".

Posted 21-Nov-12 1:50am

sachinkale20

Comments

fjdiewornncalwe 21-Nov-12 10:11am

My 1. You've missed the "cin >> hasSalary;" statement, so your answer in this case is wrong. Sorry.

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Krunal Rohit · Accepted Answer · 2012-11-20T02:30:00

Solution 1

C++

if (isEmployer == 'E' && hasLocation=='L' && hasSalary > 50000)
             cout << "You Are Allowed To Get The Loan" << endl;

Posted 20-Nov-12 2:30am

Krunal Rohit

Comments

Usman Hunjra 20-Nov-12 8:34am

yes it works but what I'm doing wrong in my code ???
i don't want AND operator too ...

MT_ 20-Nov-12 8:36am

Check below in my code.

Philippe Mori 21-Nov-12 23:08pm

else only apply to previous if... so if either first 2 conditions are not verified, it will display nothing.

MT_ · Accepted Answer · 2012-11-20T02:35:00

I am not in C++, but the logic is coded wrongly.

Here if isEmployer is E then only it will check hasLocation, Then if HasLocation is L then only it will check HasSalary. Now if HasSalary is > 50000 you will get first string and else it will return second string. But for any thing other then E or L in first two condition it will not go in the third if only
It should have been

C++

if (isEmployer == 'E')
{
   if (hasLocation=='L')
      if(hasSalary > 50000)
         cout << "You Are Allowed To Get The Loan" << endl;
 }
else
   cout << "Not Allowed To Get The Loan" << endl;

Or You can put logical AND

C++

if (((isEmployer == 'E') && (hasLocation=='L') && (hasSalary > 50000))
  cout << "You Are Allowed To Get The Loan" << endl;
else
  cout << "Not Allowed To Get The Loan" << endl;

PS: I am not sure about symbol for logical AND condition in C++

Thanks
Milind

sja63 · Accepted Answer · 2012-11-20T21:27:00

Solution 3

If you really dislike the && operator try this:

C++

bool bOK = false;

if (isEmployer == 'E')
    if (hasLocation == 'L')
        if(hasSalary > 50000)
            bOK = true;

if (bOK) cout << "OK..." << endl;
else cout << "Bad..." << endl;

I hope this helps.

Posted 20-Nov-12 21:27pm

sja63

Comments

Philippe Mori 21-Nov-12 23:11pm

In a simple case like this one, I would prefer to uses &&. But in real life code you might have a lot of code in each block and it might make sense to uses a flag to know what to do later.

Simple IF Statements

4 solutions

Solution 1

Solution 2

Solution 3

Solution 4

Add your solution here

Preview 0

Existing Members

...or Join us