[Not a question] logic problem in c programming

Question

0.00/5 (No votes)

See more:

I'm beginner in C language.I have a problem with logic.For example, the availability of the car is 1,but when i insert the amount of the car i want to rent is more than 1, the system is still running and does not enter the if else statement.Thanks !

C#

int car;
    int amountcarSAGA;
    int amountcarVIOS;

        printf(" Which car you prefer\n\t1 PROTON SAGA %d",availabilitySAGA);
        printf("\n\t2 TOYOTA VIOS\n\t");
        scanf("%d", &car );

        switch (car)
        {
          case 1 : if(availabilitySAGA<=0)
                   {
                       printf("No car available");
                       break;
                   }
                   else
                   {


                   printf("How many days you want to rent\n\t");
                   scanf("%d", &day);
                   printf("How many cars you want to rent\n\t");
                   scanf("%d",&amountcarSAGA);

                   if(availabilitySAGA>0)
                   {
                      calculation_carSAGA(day,amountcarSAGA);
                   }
                   else
                   {
                    printf("Amount of car exceed the availability");
                   }
                   availabilitySAGA=availabilitySAGA-amountcarSAGA;
                   break;
                   }

         case 2 : printf("How many days you want to rent\n\t");
                  scanf("%d", &day);
                  printf("How many cars you want to rent\n\t");
                   scanf("%d",&amountcarVIOS);
                   calculation_carVIOS(day,amountcarVIOS);
                   break;
         default: printf("\nEnter only one from the above");

        }

Posted 21-Dec-13 7:57am

arave0521

Updated 21-Dec-13 17:02pm

Sergey Alexandrovich Kryukov

v2

Add a Solution

Comments

ZurdoDev 21-Dec-13 22:21pm

Debug it and find out what is happening.

arave0521 21-Dec-13 22:31pm

User can insert the amount of car more than the availability of car.
However,for second input user cannot enter the case because the amount car is less or equal than 0.

ZurdoDev 21-Dec-13 22:38pm

Why? Just put a breakpoint and see what is happening.

arave0521 22-Dec-13 0:26am

problem solved
thanks !!! :)

1 solution

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)

Er. Tushar Srivastava · Answer 1 · 2013-12-21T19:21:00

My Friend,
This is an obvious logic that user can enter 100 cars even if you only have 3 cars. The reason is explained below... You may have to read :P

1. The first if condition as per given by you is :

C

if (availabilitySAGA > 0) { // implies that SAGA is avialable even if one the condition is true
    // Rent the car to user
} else { // Now this condition is already checked by you previously before entring this if else condition am I right?
}

This condition thereby is completely wrong. The Logic is wrong.

2. Now, the logic should be this
-> Check if AvailabilitySAGA == 0 'cause you are not going to go in minus can you?
-> Compare the AvaialabilitySAGA >= Entered amount; This is the correct logic.
-> Do whatever you want based on the positivity of the condition.

3. You generally should try to do a paper-pen simulation i.e. making flow charts and pseudo codes, before writing any logic. It will reduce considerable red eyes due to excessive debugging by you all night :P

I tried to put it in a fun way... Hope that it helped although I had read your comment already that you solved the problem yourself. It's really good. :)

With Regards
Tushar Srivastva