Пользовательская функция возврата структуры застряла на входе?

Question

Я хотел сделать функцию со структурами для упрощения рациональных чисел,
один член является числителем (int), а другой - знаменателем (int), но программа застревает на вводе!! Я знаю, что scanf() немного рискованно, но я подумал, что это всего лишь пара целых чисел!

#include <stdio.h>
#include <stdlib.h>

typedef struct rational{
    int num;
    int den;
}rational;

rational makerational(int num,int den);
void printrational(rational r);
int main()
{
    int a,b;
    printf("\n\n Input integers for fraction:");
    scanf(" %d%d",&a,&b);
    printrational(makerational(a,b));
}
rational makerational(int a,int b){
    int live=1; 
    rational r;
    r.num=a;
    r.den=b;
  while(live){  
    if(!(r.num%2 && r.den%2)){
        r.num/=2;
        r.den/=2;
    }else if(!(r.num%3 && r.den%3)){
        r.num/=3;
        r.den/=3;
    }else if(!(r.num%5 && r.den%5)){
        r.num/=5;
        r.den/=5;
    }else if(!(r.num%7 && r.den%7)){
        r.num/=7;
        r.den/=7;
    }else live--;
  }
    return r;
}
void printrational(rational r){
    printf("\nFRACTION -> %d/%d\n",r.num,r.den);
}

Answer 1

Вы застряли в бесконечном цикле здесь:

while(live){  
    if(!(r.num%2 && r.den%2)){
        r.num/=2;
        r.den/=2;
    }

    …
}

Например, когда r.num == 1 а также r.den == 2рассмотрим, что происходит:

Итерация 1

r.num % 2 == 1 % 2 == 1
r.den % 2 == 2 % 2 == 0
!(1 && 0) == 1

r.num / 2 == 1 / 2 == 0
r.den / 2 == 2 / 2 == 1

Итерация 2

r.num % 2 == 0 % 2 == 0
r.den % 2 == 1 % 2 == 1
!(0 && 1) == 1

r.num / 2 == 0 / 2 == 0
r.den / 2 == 1 / 2 == 0

Итерация 3 до бесконечности и далее...

r.num % 2 == 0 % 2 == 0
r.den % 2 == 0 % 2 == 0
!(0 && 0) == 1

r.num / 2 == 0 / 2 == 0
r.den / 2 == 0 / 2 == 0

Измените выражения в операторах ветвления на что-то вроде этого, чтобы исправить вашу логику:

while(live)
{
    if((r.num % 2 == 0) && (r.den % 2 == 0))
    {
        r.num /= 2;
        r.den /= 2;
    }

    …
}

Answer 2

Проблема в том, что когда вы вводите 8 20, числа меняются, как пара;

4 10

2 5

1 2

0 1

0 0 <--- здесь бесконечный цикл

причина в том, что целое число округляет число до нуля, вы должны добавить контрольный оператор для случая, когда числа достигают 0