Hello - First post on this forum

[EssiJoon]

Hello everyone...

I am new to C and I am working for an assignment for school and I was hoping I could get some help if possible.

for some reason this do while loop doesn't work the way I want it to. Even when I enter "A" or "P" it keeps going over and over.

Can someone please help me with this?

Thank you very much.

  do{ 
  printf("Please enter A (for AM) or P (for PM): ");
  ampm = getchar();
  }while( (ampm = getchar() ) !='A' ||  (ampm = getchar() ) !='P' );

[EssiJoon]

Oh... I tried using a scanf("%c", &ampm);

still no luck

[EssiJoon]

and string also

[stahta01]

Try adding a leading space that eats the newlines before the %c

scanf(" %c", &ampm);

}while( (ampm = getchar() ) !='A' ||  (ampm = getchar() ) !='P' );

Likely you want instead; see next post.

[ZuK]

too many getchar()'s
try

do{ 
      printf("Please enter A (for AM) or P (for PM): ");
      ampm = getchar();
  }while( ampm !='A'  &&  ampm  !='P' );

think about it. or doesnt work here. the input cannot be 'A' and 'P' at the same time

Kurt

[EssiJoon]

Thank you Kurt... I greatly appreciate your help. That worked great. My followup question now is when I run that this is what i see...

Please enter A (for AM) or P (for PM): Please enter A (for AM) or P (for PM):

I would like to only ask for the input once. Any tips?

[ZuK]

Thank you Kurt... I greatly appreciate your help. That worked great. My followup question now is when I run that this is what i see...

Please enter A (for AM) or P (for PM): Please enter A (for AM) or P (for PM):

I would like to only ask for the input once. Any tips?

This has to do with what you do before that code.
using
scanf(" %c", &ampm);
instead might help. like stahta01 suggested.
Kurt

[EssiJoon]

Thank you to both of you... You are all amazing. That worked great.

First impressions are so important and this forum really made a wonderful first impression on me.

Thank you to both of you.