Question

I am putting together a program that will calculate the area of either a triangle or a square and then prompt the user whether they wish to calculate another. I’ve got the code working to the point that it will calculate the area of either shape, but then doesn’t continue with the rest of the code. For example is square is selected, the area is calculated and then goes back to the prompt for the square’s side. I am assuming that it is again the while looping forever, but don’t know how to stop the loop from going on endlessly.

Here is my code:

 #include<stdio.h>
 #include<math.h>
 
 int main(void)
 
 {
     float sq_side, tri_base, tri_height, Area;
     char shape, cont = 'Y';
     
     printf("Please select the type of shape you desire to calculate the area for:\n");
     printf("                                                                     \n");
     printf("   Square = S                             Triangle = T               \n");
     printf("   -------                                    x                      \n");
     printf("   :     :                                   x x                     \n");
     printf("   :     :                                  x   x                    \n");
     printf("   -------                                 xxxxxxx                   \n");
     printf("                                                                     \n");
     printf("Please select either S or T:");
     scanf("%c", &shape);
     while (cont != 'n' && cont != 'N')
     
        if (shape == 'S' || shape == 's')
        {
            printf("What is the length of the sides of the square?:\n");
            scanf("%f", &sq_side);
         
            Area = pow(sq_side,2);
         
            printf("The area of the square is %.2f.\n", Area);
        }   
         
        else if (shape == 'T' || shape == 't')
        {
            printf("What is the length of the base of the triangle?:\n");
            scanf("%f", &tri_base);
            printf("What is the height of the triangle?:\n");
            scanf("%f", &tri_height);
            
            Area = 0.5 * tri_base * tri_height;
            
            printf("The area of the triangle is %.2f.\n", Area);
        }   
        
        else 
        {
        printf("Error:  You have select an incorrect option.");
        }
        
        printf("Do you wish to calculate a new shape?");
        fflush(stdin);
        scanf("%c", &cont);
        
     return(0);
     
 }
 

Answer 1

You’re missing curly braces. The result was that only the if statement (which includes the chain of else ifs) was actually in the loop body. The printf (and later) is not part of this compound statement.

 while (cont != 'n' && cont != 'N')
 {
    if (shape == 'S' || shape == 's')
    {
        printf("What is the length of the sides of the square?:\n");
        scanf("%f", &sq_side);

        Area = pow(sq_side,2);

        printf("The area of the square is %.2f.\n", Area);
    }   

    else if (shape == 'T' || shape == 't')
    {
        printf("What is the length of the base of the triangle?:\n");
        scanf("%f", &tri_base);
        printf("What is the height of the triangle?:\n");
        scanf("%f", &tri_height);

        Area = 0.5 * tri_base * tri_height;

        printf("The area of the triangle is %.2f.\n", Area);
    }   

    else 
    {
    printf("Error:  You have select an incorrect option.");
    }

    printf("Do you wish to calculate a new shape?");
    fflush(stdin);
    scanf("%c", &cont);
}