Der Code wird nicht ausgegeben, aber er läuft. [c]

Question

Code Funktion: Es ist ein russischer Bauer Multiplikation Rechner.
Code:

#include <stdio.h> 
#include <math.h> 

main(){ 
    int ier, product = 0, check, i = 1; //var to store numbers 
    float cand; //Var to store decimal numbers 

    while(i != 0){ 
    printf("\nInput only whole numbers!"); //Ask user for number to be multiplied 
    printf("\nMultiplier: "); 
    scanf("%d", &ier); 
    printf("Multiplicand: "); 
    scanf("%d", &cand); 

    while(i != 0){ 
     if((ier == 0 && cand > 0) || (ier > 0 && cand == 0) || (ier == 0 && cand < 0) || (ier < 0 && cand == 0)){//Regulations for the code 
     printf("Product: 0"); 
     printf("\n\nInput only whole numbers!"); 
     printf("\nMultiplier: "); 
     scanf("%d", &ier); 
     printf("Multiplicand: "); 
     scanf("%d", &cand); 
     } 
     if((ier > 0 && cand > 0) || (ier < 0 && cand > 0) || (ier > 0 && cand < 0) || (ier < 0 && cand < 0)){ 
     break; 
     } 
     if(ier == 0 && cand == 0){//If user inputs 0 and 0, end code 
     return 0; 
     } 
    } 

    while(i != 0){ 
     if((ier < 0 && cand > 0) || (ier > 0 && cand < 0) || (ier < 0 && cand < 0)){//Regulations for the code 
     printf("Values must not be negative"); 
     printf("\n\nInput only whole numbers!"); 
     printf("\nMultiplier: "); 
     scanf("%d", &ier); 
     printf("Multiplicand: "); 
     scanf("%d", &cand); 
     } 
     if((ier > 0 && cand > 0) || (ier == 0 && cand > 0) || (ier > 0 && cand == 0) || (ier == 0 && cand < 0) || (ier < 0 && cand == 0)){ 
     break; 
     } 
     if(ier == 0 && cand == 0){//If user inputs 0 and 0, end code 
     return 0; 
     } 
    } 

    if(ier > 0 && cand > 0){ 
     printf("Calculating Product:");//It should print this, but it isn't for some reason. 
     while(i != 0){ 
     if(fmod(cand, 2) != 0){ 
      if(fmod(cand, 1) != 0){ 
      product = product + floor(cand); 
      cand = floor(cand); 
      printf("%d \t%f", ier, cand); 
      } 
      else{ 
      product = product + cand; 
      printf("%d \t%f", ier, cand); 
      } 
     } 
     if(cand == 1){ 
      printf("Product: %d", product); 
      break; 
     } 
     ier *= 2; 
     cand /= 2; 
     } 
    } 
    } 
} 

Problem: Wenn ich Code und I-Eingang laufe beispiel 5 und 2, hält es ohne etwas Ausgeben auf Laufen. Ich denke, das Problem im Code ist die fmod(). Ich verwende nur fmod, weil Sie den Modulo-Operator % nicht für Float-Variablen verwenden können.
Thing, das ich getan habe: Ich änderte die fmod zu dem Modulo-Operator % und machte cand in eine ganze Zahl. Das hat funktioniert, aber jetzt habe ich ein Problem mit der Dezimalzahl, da ganze Zahlen die Dezimalzahl haben. Also ging ich zurück zu fmod.
Eingang:

gcc version 4.6.3 
Input only whole numbers! 
Multiplier: 5 
Multiplicand: 2 
/*Runtime: infinite*/ 

Ausgang Ich möchte

Multiplier: 57 
Multiplicand: 86 
Calculating product: 
114 43 
228 21  
912 5  
3648 1 
Product: 4902 
Multiplier: 48 
Multiplicand: -36 
Values must not be negative 
Multiplier: 27 
Multiplicand: 0 
Product: 0 
Multiplier: 0 
Multiplicand: 0 

P. S: Es ist mein erstes Mal fmod und floor() verwenden.

Answer 1

Nach dem Lesen Ihres Codes habe ich festgestellt, dass Sie die scanf Funktion falsch verwendet haben. In Ihrem Code nach der ersten while Schleife, haben Sie die folgenden Schritte aus:

while(i != 0){ 
    printf("\nInput only whole numbers!"); //Ask user for number to be multiplied 
    printf("\nMultiplier: "); 
    scanf("%d", &ier); 
    printf("Multiplicand: "); 
    scanf("%d", &cand); 

Wenn man sich die scanf über diesem Satz anschaut, kann man sehen, dass du getan hast (Wie @MartinR darauf hingewiesen):

scanf("%d", &cand); 

Sie müssen den Code zu folgenden Änderungen:

scanf("%f", &cand); 

Dies wird Ihre unendliche Laufzeit beheben. Außerdem kann @ABusyProgrammer die Antwort zuerst geschrieben haben (was korrekt ist), er/sie hat den Beitrag nicht richtig gelesen. Sie verwenden die fmod, die einen Float erfordert. Daher müssen Sie das oben beschriebene scanf verwenden.@ABusyProgrammer wäre nur korrekt, wenn Sie den Operator % verwenden.

Answer 2

Der Grund Code in eine Endlosschleife geht, ist nicht wegen der fmod() Funktion, sondern weil Sie cand als float Wert initialisiert haben, und nicht als int Wert. Ändern cand zu einem int Wert ermöglicht der Code weiter zu arbeiten, um das Ergebnis zu drucken:

So ändern:

float cand; 

An:

int cand; 

Das zumindest den Code verhindert hängen Genau wie @WeatherVane zeigt es auch eine Warnung an, die der Compiler für die folgende Zeile ausgeben sollte:

scanf("%d", &cand); // "cand" is a float, but your format specifier says that it is an int.