Need help on programming.

**BlazKaz** · 04-09-2019

So I'm a new member here and i don't know how to make things coding for the amount of total claim for the question.
Here is what i got so far

Code:

#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
int main(int argc, char*argv)
    {
    char clas,response;
    int dista;
    float amount, rate;
    printf("Enter Class of Vehicle: ");
    scanf("%c",&clas);
    Label1:
    printf("Enter Distance Travelled(km):");
    scanf("%d",&dista);
    switch(clas)
    {
    case'A':
    case'a':rate = 0.70;break;
    case'B':
    case'b':rate = 0.60;break;
    case'C':
    case'c':rate = 0.50;break;
    case'D':
    case'd':rate = 0.45;break;
    default:printf("Wrong Input");rate=0;
    }
    amount = rate*dista;
    printf("\n Amount of Claim 1 (RM): %.2f",amount);
    printf("\n Next mileage Claim(EnterY/N):");
    if(response 'Y')
    {
        goto Label2;
    }
    else(response 'N')
    {
        printf("End");
    }
    }

    Label2:
    printf("\n Enter Distance Travelled(km): ");
    scanf("%d",&dista);

    switch(clas)
    {
    case'A':
    case'a':rate = 0.70;break;
    case'B':
    case'b':rate = 0.60;break;
    case'C':
    case'c':rate = 0.50;break;
    case'D':
    case'd':rate = 0.45;break;
    default:printf("Wrong Input");rate=0;
    }
    amount = Label1 + Label2;
    printf("Total Amount of Claim: %.2f",amount);
    return(0);

What should i do to make it work? I've been trying to solve this for a few weeks now

Or I tried this one out but still stuck. Which method is better for having total amount of claim?

Code:

#include<stdio.h>
int main()
    {
    char clas, next, response, Y, N;
    int i, dista;
    float amount1,amount2, rate, grandtotal;    
    printf("Enter Class of Vehicle: ");
    scanf("%c", &clas);
    printf("Enter Distance Travelled(km):");
    scanf("%d", &dista);
    
    switch(clas)
    {
    case'A':
    case'a':rate = 0.70;break;
    case'B':
    case'b':rate = 0.60;break;
    case'C':
    case'c':rate = 0.50;break;
    case'D':
    case'd':rate = 0.45;break;
    default:printf("Wrong Input");rate=0;
    }
    amount1 = rate*dista;
    printf("\n Amount of Claim 1 (RM): %.2f",amount1);
    i++;
    printf("\n Next Mileage Claim (Enter Y/N)");
    scanf("%c",&next);
    if(response == Y);
    {
    printf("\n Enter Distance Travelled(km):");
    scanf("%d", &dista);
    }
    if else(response == N);
    {
        printf("End");    
    }
    amount2 = rate*dista;
    printf("\n Amount of Claim 2 (RM): %.2f",amount2);
        
    //This is the total amount i'm trying to make
    grandtotal = amount1 + amount2;
    printf("Total Amount of Claim = %.2f",grandtotal);
    
    return(0);
    }

**Click_here** · 04-09-2019

Be cautious using goto - It can make your program very difficult to debug and reduce the readability.

It is usually advised that you don't use 'goto' at all, unless you need to exit from a deeply nested loop.

Break your problem into section such as:
* Intro message
* Get Vehicle class and distance travelled
* Get rate
* Display result
* Implement do/while loop to go back to the start if the user enters 'y' or 'Y' when prompted
* Add grand total functionallity
* ...

Make a new programme. Start with the first section that you made before - Don't move on until it is 100% working.

**flp1969** · 04-09-2019

Originally Posted by Click_here

Be cautious using goto - It can make your program very difficult to debug and reduce the readability.

It is usually advised that you don't use 'goto' at all, unless you need to exit from a deeply nested loop.

Break your problem into section such as:
* Intro message
* Get Vehicle class and distance travelled
* Get rate
* Display result
* Implement do/while loop to go back to the start if the user enters 'y' or 'Y' when prompted
* Add grand total functionallity
* ...

Make a new programme. Start with the first section that you made before - Don't move on until it is 100% working.

Dijkstra would be proud...

**gaxio** · 04-11-2019

You can do yourself a favor and format your code correctly. This is not a nitpicky "because you're supposed to but no one really does" kind of thing. Your code is legitimately difficult to read. I can't be sure what the closing brace on line 37 does to. It looks like it closes the main function, which will cause the program to not compile.

This is bad:

Code:

int main(int argc, char *argv[])
  {
  for(int x = 0; x < 10; x++)
  {
  for(int y = 0; y < 10; x++)
  {
  // Stuff
  }
  }

It's not too bad here, but as the code gets longer, it gets worse and word. This is better:

Code:

int main(int argc, char *argv[])
{
  for(int x = 0; x < 10; x++)
  {
    for(int y = 0; y < 10; x++)
    {
      // Stuff
    }
  }

Form the indentation, it's very clear what the structure of the program is.

Next, don't use goto. It's rarely appropriate in C and it's from an era where programming languages didn't have very good looping features. I would suggest not using it at all until you learn where it's appropriate. And using goto to things like "label1" and "label2" is even worse. Name your labels something meaningful.

I was going to give your program a crack, but honestly I can't make heads or tails of it. Someone else suggested that you start over, start small and build the program iteratively and I think that's a great idea.

Also, what book are you using to learn C? Because I really recommend the K. N. King book. If you really read that book and do the exercises (yes, all of them), you'll be having a much better time of this than struggling with this one program for weeks.

**BlazKaz** · 04-17-2019

Originally Posted by Click_here

Be cautious using goto - It can make your program very difficult to debug and reduce the readability.

It is usually advised that you don't use 'goto' at all, unless you need to exit from a deeply nested loop.

Break your problem into section such as:
* Intro message
* Get Vehicle class and distance travelled
* Get rate
* Display result
* Implement do/while loop to go back to the start if the user enters 'y' or 'Y' when prompted
* Add grand total functionallity
* ...

Make a new programme. Start with the first section that you made before - Don't move on until it is 100% working.

got it, ill rebuild again soon

**BlazKaz** · 04-17-2019

Originally Posted by gaxio

You can do yourself a favor and format your code correctly. This is not a nitpicky "because you're supposed to but no one really does" kind of thing. Your code is legitimately difficult to read. I can't be sure what the closing brace on line 37 does to. It looks like it closes the main function, which will cause the program to not compile.

This is bad:

Code:

int main(int argc, char *argv[])
  {
  for(int x = 0; x < 10; x++)
  {
  for(int y = 0; y < 10; x++)
  {
  // Stuff
  }
  }

It's not too bad here, but as the code gets longer, it gets worse and word. This is better:

Code:

int main(int argc, char *argv[])
{
  for(int x = 0; x < 10; x++)
  {
    for(int y = 0; y < 10; x++)
    {
      // Stuff
    }
  }

Form the indentation, it's very clear what the structure of the program is.

Next, don't use goto. It's rarely appropriate in C and it's from an era where programming languages didn't have very good looping features. I would suggest not using it at all until you learn where it's appropriate. And using goto to things like "label1" and "label2" is even worse. Name your labels something meaningful.

I was going to give your program a crack, but honestly I can't make heads or tails of it. Someone else suggested that you start over, start small and build the program iteratively and I think that's a great idea.

Also, what book are you using to learn C? Because I really recommend the K. N. King book. If you really read that book and do the exercises (yes, all of them), you'll be having a much better time of this than struggling with this one program for weeks.

Oh, i just started programming so im having a hard time with it on my own with just youtube and internet but thanks for helping me out. i learned the C programming on internet so there is no books here that relates to coding. but thanks for your suggestion on the book. ill look it up for now

**gaxio** · 04-18-2019

I would get away from videos and random stuff you find on the internet for the time being. Some of it is very low quality, but the real issue is that it's inconsistent. You need a good book that introduces things in the correct (or as correct as can be) order, all explained well with plenty of exercises to get you to put it into practice.

Trust me when I say that learning from bad sources is a big mistake. I learned C from a pocket reference guide and a text file someone had written. This was before I had the internet so I couldn't ask questions, either. I got so much stuff wrong and honestly it was just a waste of time. Get a good book however you can and work it from cover to cover, and ask questions when you get stuck. You have these two advantages I didn't have, use them.

Thread: Need help on programming.

Thread Tools

Search Thread

Display

Need help on programming.

Similar Threads

C Programming Tips for Advanced Platform Independent (Portability) Programming - 2

C Programming Tips for Advanced Platform Independent (Portability) Programming - 1

Notepad++ Game Programming -- Working with Directx and Win32 Programming.

Total newb to programming here... Question about the many programming languages. Ty!

Tags for this Thread