Command "goto" creating infinite loop in "for" loop

Question

I want to simulate a school database that collects name, math and physics grades and averages both for 5 students (defined in a string of struct).

Any grade above 10 would be allowed, without any lock for it. Students would easily have grades above 10. So I thought I’d make a condition with if-else and used the goto, positioning its return at the beginning of the note request.

See the code below:

#include<stdio.h>
#include<locale.h>

int main(void){

setlocale(LC_ALL,"Portuguese");

struct Aluno{
    char nome[10];
    float notaMath, notaPhysics, media;
};

struct Aluno aluno[5];

int contador;

//Entrada de dados via teclado
for(contador = 0; contador < 5; contador++){
    printf("Nome do aluno %d: ", contador+1);
    scanf("%s",&aluno[contador].nome);

    RETORNO1:
    printf("Nota de matemática: ");
    scanf("%f",&aluno[contador].notaMath);
    if(aluno[contador].notaMath > 10.0){
        printf("Apenas notas até 10.\n");
        goto RETORNO1;
    }
    else{
        goto POINT1;
    }

    POINT1:
    RETORNO2:
    printf("Nota de física: ");
    scanf("%f",&aluno[contador].notaPhysics);
    if(aluno[contador].notaPhysics > 10.0){
        printf("Apenas notas até 10.\n");
        goto RETORNO2;
    }
    else{
        goto POINT2;
    }

    POINT2:
    aluno[contador].media = (aluno[contador].notaMath + aluno[contador].notaPhysics)/2;

    printf("\n");
}

printf("\n\n");

printf("------------Informações dos Alunos------------\n");
for(contador = 0; contador < 5; contador++){
    printf("Nome do aluno %d: %s\n", contador+1, aluno[contador].nome);
    printf("Nota de matemática: %.1f\n", aluno[contador].notaMath);
    printf("Nota de física: %.1f\n", aluno[contador].notaPhysics);
    printf("Média das notas: %.1f\n", aluno[contador].media);
    printf("\n\n");
}
}

In this case, by placing any floating point values greater than 10, it prints "only notes up to 10" and actually returns to the set point, all ok. But if I put some floating point value, like 10.5, it goes into loop infinite.

Without the placement of goto and the POINTs and RETORNOs he simply skips most of the following instructions and goes there forward in the loop for.

Example of execution (data copied from .exe):

"Nome do aluno 1: Jonathan
 Nota de matemática: 10.5
 Nota de física:
 Nome do aluno 2: Nota de matemática:"

Answer 1

The code had some bugs that I fixed. I improved the style a bit too.

goto is almost never suitable. It can almost always be replaced by a if or a while, what is the case. What you clearly want is a repeat loop until the person type the correct one, so make a loop, it’s much simpler. Only exit it when the entered value is valid.

#include <stdio.h>
#include <locale.h>

int main(void) {
    setlocale(LC_ALL,"Portuguese");
    struct Aluno {
        char nome[10];
        float notaMath, notaPhysics, media;
    } aluno[5];
    for (int contador = 0; contador < 5; contador++) {
        printf("Nome do aluno %d: ", contador + 1);
        scanf("%s", aluno[contador].nome);
        while (1) {
            printf("Nota de matemática: ");
            scanf("%f", &aluno[contador].notaMath);
            if (aluno[contador].notaMath > 10.0) printf("Apenas notas até 10.\n");
            else break;
        }
        while (1) {
            printf("Nota de física: ");
            scanf("%f", &aluno[contador].notaPhysics);
            if (aluno[contador].notaPhysics > 10.0) printf("Apenas notas até 10.\n");
            else break;
        }
        aluno[contador].media = (aluno[contador].notaMath + aluno[contador].notaPhysics) / 2;
        printf("\n");
    }
    printf("\n\n------------Informações dos Alunos------------\n");
    for (int contador = 0; contador < 5; contador++) {
        printf("Nome do aluno %d: %s\n", contador+1, aluno[contador].nome);
        printf("Nota de matemática: %.1f\n", aluno[contador].notaMath);
        printf("Nota de física: %.1f\n", aluno[contador].notaPhysics);
        printf("Média das notas: %.1f\n", aluno[contador].media);
        printf("\n\n");
    }
}

Behold working in the ideone. And in the repl it.. Also put on the Github for future reference.

I left the contador, but I prefer i, I find it very verbose, everyone knows what is the i, I find it even more readable. I would have other tips for writing better code, but it escapes the scope of the question.

Answer 2

Good as you want to make the user rewrite in the same house, if the note > 10, you can do the following, in this code snippet you can decrease your counter, this way it will go back to the same address:

scanf("%f %f",&aluno[contador].notaPhysics, &aluno[contador].notaMath);
if(aluno[contador].notaPhysics > 10.0 || aluno[contador].notaMath > 10.0){
    printf("Apenas notas até 10.\n");
    contador--;
}

But if you want to use Goto, I suggest you remove your RETORNO1 and RETORNO2 and also remove the Else.