sentinel loop execution in c

Question

I have the following code and I would like it to break after entering the sentinel number only once but at the moment I have to input the sentinel number twice for the code to break. Any ideas?

#include <stdio.h>// Include a standard C Library
#define Sentinel_Numb -1
int main(){

  int ID[360];              
  float seconds[360];
  int i = 0;

 printf("please enter the  ID and how many seconds to calculate enter  -1 when done\n");

while (1)                           
{
    scanf_s("%d",&ID[i]);
    scanf_s("%f",&seconds[i]);

    if( ID[i] == Sentinel_Numb || seconds[i] == Sentinel_Numb){
       break;
         }
    i++;
}

return 0;
}

You should learn [proper c formatting](//prohackr112.tk/proper-c-formatting). It'll encourage people to help you. — MD XF, Dec 02 '16 at 01:39

score 1 · Answer 1 · answered Dec 02 '16 at 00:02

1

Change to:

scanf_s("%d",&ID[i]);
if (ID[i] == Sentinel_Numb)
    break;
scanf_s("%f",&seconds[i]);
if (seconds[i] == Sentinel_Numb)
    break;

answered Dec 02 '16 at 00:02

stark

12,615
3
33
50

score 0 · Accepted Answer · edited Dec 02 '16 at 02:24

0

while (1)                           
{
    scanf_s("%d",&ID[i]);

    if (ID[i] == Sentinel_Numb)
        break; 

    scanf_s("%f", &seconds[i]);

    if (seconds[i] == Sentinel_Numb)
        break;
    i++;
}

edited Dec 02 '16 at 02:24

MD XF

7,860
7
40
71

answered Dec 02 '16 at 00:05

Fady Nabil Yacoub

162
6

sentinel loop execution in c

2 Answers2