Can anyone help me with this??

Code:

#include<stdio.h>
void main (void)
{
	int num;
	float bbal,tchar,tcred,credl,nbal;
	
	printf("Enter account number (-1 to end):");
	scanf("%d",&num);
	while (num!=-1)
	{
	printf("Enter beginning balance: ");
	scanf("%f",&bbal);
	printf("Enter total charges: ");
	scanf("%f",&tchar);
	printf("Enter total credits: ");
	scanf("%f",&tcred);
	printf("Enter credit limit: ");
	scanf("%f",&credl);
	nbal=bbal+tchar-tcred;
	
	{if (nbal>credl)
		{	printf("Account:%d\n",num);
			printf("Credit limit:%.2f\n",credl);
			printf("Balance:%.2f\n",nbal);
			printf("Credit limit exceeded.\n\n");
			printf("\n\nEnter account number (-1 to end):");
			scanf("%d",&num);}
	else (nbal<credl)
		{	printf("\n\nEnter account number (-1 to end):");
			scanf("%d",&num);}
	}
	}
}

Problem is, wether nbal exceeds credl or not...it still follows the function of (nbal>credl), its ignoring else.... btw, this whole thing should repeat itself until account number is -1.

The result should come out just ask like its shown below if it exceeds or not. Thx! =)

check the else line...remove the condition after else it will work....
and you could also reduce the code by removing redundant input of account number..

Originally Posted by subhash.rao

check the else line...remove the condition after else it will work....

Not quite. Add the keyword "if" after the "else" keyword. (nbal < cred1) is not the opposite of (nbal > cred1).

And main() returns int, not void.

Originally Posted by grumpy

Add the keyword "if" after the "else" keyword. (nbal < cred1) is not the opposite of (nbal > cred1).

I said to remove condition because if u add if and make it else if(nbal < cred1) what will happen if for some input nbal is equal to cred1..for that u have write a seperate else again..correct me if I am wrong..

Originally Posted by subhash.rao

I said to remove condition because if u add if and make it else if(nbal < cred1) what will happen if for some input nbal is equal to cred1..for that u have write a seperate else again..correct me if I am wrong..

People rarely type in conditions after the else keyword unless they intend those conditions be tested, in which case the "if" keyword is necessary.

Among people who I've seen doing this type of thing in the past: over 90% chance they've forgotten the "if" keyword, less than 10% they need to remove the condition being tested.

That equates, across all people I've seen asking this sort of question, to over a 90% chance you are wrong. Either way, the OP will probably be slapping his or her forehead, because it is a simple and obvious error once pointed out.

I was just trying to point out u may get undesirable output for some inputs....

Originally Posted by grumpy

People rarely type in conditions after the else keyword unless they intend those conditions be tested, in which case the "if" keyword is necessary.

Anyway u r right..

Originally Posted by grumpy

Either way, the OP will probably be slapping his or her forehead, because it is a simple and obvious error once pointed out.

hehe..

And this is C, not C++.

thanks ppl! =) im new in this...