なぜ "Payable 0"であるべきか "Payable 2"を印刷する理由

Question

私はjuiceという名前の関数を作成しました。顧客は のジュースのカップサイズを選択して価格や支払い金額を返しますが、私はそれが0を返す必要がありますswitchステートメントのデフォルトケースを選択するが、出力は2

#include <stdio.h> 

int juice(char size , int qty){ 
int price =0; 
switch(size){ 
    //'s' for small size of cup 
    case 's': 
     printf("size small"); 
     price =20*qty; 
     return price; 
     break; 
    //'m' for medium size of cup 
    case 'm': 
     printf("size medium"); 
     price =30*qty; 
     return price; 
     break; 
    //'l' for large size of cup 
    case 'l': 
     printf("size large"); 
     price =40*qty; 
     return price; 
     break; 
    //if costumer choose wrong size 
    default: 
     printf("choose proper size"); 
    } 

printf("\n%d", price); 
} 

int main() 
{ 
    int price =juice('d' ,5); 
    printf("\npayable is %i\n", price); 
    return 0; 
} 

出力です：

choose proper size 
0 
payble is 2 

Answer 1

問題は、機能がdefault場合には戻っていないということです。それは未定義の動作です。

注return price;ことは、次のbreak;が不要になり、それはまた、代わりに、読むためのコード難しくなり、以来、あなたはあなただけでreturn price;を置くことができます（defaultを除く）すべてのケースでpriceの値を設定しています終わり。最後に、defaultケースに初期化文を追加します。また、この

int juice(char size, int qty) 
{ 
    int price = 0; 
    switch (size) { 
     case 's': // 's' for small size of cup 
      printf("size small\n"); 
      price = 20 * qty; 
      break; 
     case 'm': // 'm' for medium size of cup 
      printf("size medium\n"); 
      price = 30 * qty; 
      break; 
     case 'l': // 'l' for large size of cup 
      printf("size large\n"); 
      price = 40 * qty; 
      break; 
     default: // if costumer choose wrong size 
      printf("choose proper size\n"); 
      price = -1; // Invalid value? 
      break; 
    } 
    return price; 
} 

、改行文字'\n'よう

何かがそれはストリームをフラッシュし、新しい行を作成しますので、IOストリームがラインバッファリングされている行の最後に置かれることを意図されており、それを最後に置く方が理にかなっています。