"？"の後の複数の命令オペレータ

Question

私はすでにnominatorとdenominatorとis_correct()という2つの必要条件をチェックする関数を持っています。しかし今は私の分数を還元不可能にするshorten()関数を作らなければなりません。

私はgcd()関数（最大公約数）を作ったので、shorten()関数を自分の構造体に入れる必要があります。 gcd()戻っていない1値が、その後、私は、この値によってnを分割し、この値でdを分割したいのであれば

struct fraction { 
       int n,d; 
       bool is_correct(){return d!=0?(abs(n)<abs(d)?true:false):false;}; 
       void shorten(){gcd(n,d)!=1?(n=n/gcd(n,d);d=d/gcd(n,d)):;}; 
       }; 

int gcd(int x, int y) 
{ 
    int z; 
    while (y!=0){z=x%y;x=y;y=z;} 
    return x; 
} 

：それはこのようになります。

私がそれを私に期待しているようにしようとすると "：" "before"; "多くの場所でトークン。私は間違って何をしていますか？

Answer 1

?:オペレータの:部分に式を指定していません。

void shorten(){gcd(n,d)!=1?(n=n/gcd(n,d);d=d/gcd(n,d)):;};

が

void shorten() { gcd(n, d) != 1 ? (n = n/gcd(n, d) ; d = d/gcd (n , d)):; }; 
                ^^^^^^      ^^ 
              missing      colon should be 
              paranthesis/    in between the 
              semicolon doesn't   statements 
              belong here 

だからです：あなたは、エラーを見つけるだろう、正しくコードをフォーマットすることにより

int g = gcd(n, d); 
if (g != 1) 
{ 
    n = n/g; 
    d = d/g; 
} 

Answer 2

：あなたはこのようifの文の代わりに、三項演算子を使用する必要があります

void shorten() { gcd(n, d) != 1 ? (n /= gcd(n, d)) : (d /= gcd(n, d)); } 

またはより良い

void shorten() 
{ 
    if (gcd(n, d) != 1) 
     n /= gcd(n, d); 
    else 
     d /= gcd(n, d); 
}