Thread: Error in printf line,,,what's wrong?

> printf ("H = %lf W T2 = %lf K" ,heat_rate, temp2) ;
> printf ("k = %lf W/m-K T1 = %lf K", coef, temp1) ;
> printf ("A = %lf m^2 x = %lf m" , area, thickness) ;
Perhaps these lines too need a \n at the end of them.

Or call
fflush(stdout);
when you're done.

Nothing big problem here.
Actually you have closed the main() function before printf statement.
So correct that one.
It will work correctly.

Problem in your code

Code:

}


  printf ("                kA (T2-T1)   \n") ;
  printf ("           H = ------------  \n") ;
  printf ("                    x        \n") ;

  printf ("H = %lf W       T2 = %lf K" ,heat_rate, temp2) ;
  printf ("k = %lf W/m-K   T1 = %lf K", coef, temp1) ;
  printf ("A = %lf m^2     x = %lf m" , area, thickness) ;
 
  
  system ("PAUSE") ;
  return 0 ;
}

Remove the curly bracket which is above to the statement printf (" kA (T2-T1) \n") ;

Originally Posted by mft_ika

Also, I don't know whether I use variable "counter" the right way. The idea is to count the input from the user. If it's not 5, that means one must be 0 (the one with the question mark inputted), and thus will go to the respective function. But if it's 5, the user must input again the data. At this point I don't understand how to loop it back to the first 6 steps (inquiring input from user).

No, you're not using it correctly. That scanf would attempt to get more input from the user (they'd need to type everything in over again which defeats the purpose of asking it the first time around).

Your overall I/O would need work if you truly expect to process question mark '?' characters by using scanf to read into doubles with %lf. On encountering a question mark, the scanf would fail (returning 0) and the stream would go into an error state. Before proceeding to the next input you'd have to clear the error state and flush the current contents of the buffer (otherwise the data that caused the error state would still be there polluting the next attempt at input). You would need to do this error checking each time you try and process user's input.

Your if/else if block of code also assumes that an unsuccessful input attempt into an uninitialized variable magically sets the variable's contents to 0...

...it does not BTW.

Thank you all for your help!
I've revised the code. This time it works, but I think the logic is still wrong. because each time I input a question mark, the program will terminate and skip the steps of inquiring other data from the user. This is the code.

Code:

/* This program is used to solve simple conduction problems using various forms of the formula*/
#include <stdio.h> /* define printf and scanf */
#include <math.h> /* define mathematical operations */

double calc_h (double coef, double thickness, double area, double temp1, double temp2) ; /* define function named calc_k */
double calc_k (double heat_rate, double thickness, double area, double temp1, double temp2) ; /* define function named calc_k */
double calc_a (double heat_rate, double thickness, double coef, double temp1, double temp2) ; /* define function named calc_a */
double calc_x (double heat_rate, double coef, double area, double temp1, double temp2) ; /* define function named calc_x */
double calc_t2 (double heat_rate, double thickness, double area, double temp1, double coef) ; /* define function named calc_t1 */
double calc_t1 (double heat_rate, double thickness, double area, double coef, double temp2) ; /* define function named calc_t2 */
void check (double var, double *temporary) ;

main() 
{
  /* define variables that will be used */     
  double heat_rate  ; 
  double coef ;
  double area ; 
  double temp1 ;
  double temp2 ;
  double thickness ;
  double counter ;
  counter = 1 ;
      
  /* mention the background and the purpose of the program */
  printf ("Respond to the the prompts with the data known, \n") ;
  printf ("Unknown quantity, enter a question mark (?). \n") ;
  
  printf ("Rate of heat transfer (unit = watts) ---> ") ;
  scanf ("%lf",&heat_rate) ;
  check (heat_rate, &counter) ;

  printf ("Coefficient of thermal conductivity (unit = W/m-K) ---> ") ;
  scanf ("%lf",&coef) ;
  check (coef, &counter) ;

  printf ("Cross sectional area of conductor (unit = m^2) ---> ") ;
  scanf ("%lf",&area) ;
  check (area, &counter) ;

  printf ("Temperature on the first side (unit = Kelvin) ---> ") ;
  scanf ("%lf",&temp1) ;
  check (temp1, &counter) ;

  printf ("Temperature on the second side (unit = Kelvin) ---> ") ;
  scanf ("%lf",&temp2) ;
  check (temp2, &counter) ;

  printf ("Thickness of conductor (unit = m) ---> ") ;
  scanf ("%lf",&thickness) ;
  check (thickness, &counter) ;
  
      
  if (counter == 0)
  {
  if(heat_rate == 0)
  {
        calc_h (coef, thickness, area, temp1, temp2)      ; 
  }
  else if (coef == 0)
  {
        calc_k (heat_rate, thickness, area, temp1, temp2) ;
  }
  else if (area == 0)
  {
        calc_a (heat_rate, thickness, coef, temp1, temp2)  ;
  }
  else if (temp1 == 0)
  {
        calc_t1 (heat_rate, thickness, area, coef, temp2) ;
  }
  else if (temp2 == 0)
  {
        calc_t2 (heat_rate, thickness, area, temp1, coef) ;
  }
  else
  {
        calc_x (heat_rate, coef, area, temp1, temp2) ;
  }
  }else
  {
      printf ("Invalid input! You must input a question mark for the unknown variable. \n") ;
  }
  


  printf ("                kA (T2-T1)   \n") ;
  printf ("           H = ------------  \n") ;
  printf ("                    x        \n") ;

  printf ("H = %lf W       T2 = %lf K \n" ,heat_rate, temp2) ;
  printf ("k = %lf W/m-K   T1 = %lf K \n", coef, temp1) ;
  printf ("A = %lf m^2     x = %lf m \n" , area, thickness) ;
 
  
  system ("PAUSE") ;
  return 0 ;
}



/* function named calc_k, used to calculate the coefficient of thermal conductivity */
double calc_k (double heat_rate, double thickness, double area, double temp1, double temp2)
{
       return ((heat_rate*thickness)/(area*(temp2-temp1))) ;
}

/* function named calc_a, used to calculate the cross sectional area */
double calc_a (double heat_rate, double thickness, double coef, double temp1, double temp2)
{
       return ((heat_rate*thickness)/(coef*(temp2-temp1))) ;
}

/* function named calc_x, used to calculate the thickness of the conductor */
double calc_x (double heat_rate, double coef, double area, double temp1, double temp2)
{
       return ((coef*area*(temp2-temp1))/heat_rate) ;
}

/* function named calc_h, used to calculate the rate of heat transfer */
double calc_h (double coef, double thickness, double area, double temp1, double temp2)
{
       return ((coef*area*(temp2-temp1))/thickness) ;
}

/* function named calc_t2, used to calculate the temperature on the second or furthest side of the conductor */
double calc_t2 (double heat_rate, double thickness, double area, double temp1, double coef)
{
       return (((heat_rate*thickness)/(coef*area))+temp1) ;
}

/* function named calc_t1, used to calculate the temperature on the first or nearest side of the conductor */
double calc_t1 (double heat_rate, double thickness, double area, double coef, double temp2)
{
       return (temp2-((heat_rate*thickness)/(coef*area))) ;
}

void check (double var, double *temporary) 
{
if (var == 0)
*temporary = *temporary - 1 ; 
else 
printf ("\n") ; 
}

could anyone help me with this? thank you so much again!