プログラマ専用SNS ミクプラ

non さんが書きました:21行　while((r = x%y) =! 0){//ユークリッドの互除法
間違ってる。

みけCAT さんが書きました:

non さんが書きました:21行　while((r = x%y) =! 0){//ユークリッドの互除法
間違ってる。

確かに…これはコンパイルエラー不可避ですね。

このコードではない、古いプログラムを実行している可能性があるかもしれません。

non さんが書きました:

みけCAT さんが書きました:

non さんが書きました:21行　while((r = x%y) =! 0){//ユークリッドの互除法
間違ってる。

確かに…これはコンパイルエラー不可避ですね。

このコードではない、古いプログラムを実行している可能性があるかもしれません。

コンパイルエラーはでないかも。
0を否定して、rに代入。
0がFALSEならTRUEを代入し、それを評価で永久ループ。

追記
解釈はコンパイラーによるみたいですね。失礼しました。

#include <stdio.h>

int GCD(int x, int y);//最大公約数を求める
int KEISAN(int x, int y);

int main(){
	int x,y,LCM_num;
	puts("xの値を入力");
	scanf("%d",&x);
	puts("yの値を入力");
	scanf("%d",&y);

	LCM_num = KEISAN(x,y);
	printf("%dと、%dから%dまでの最小公倍数は%dです\n",x,y,y+9,LCM_num);

	return 0;
}

int GCD(int x, int y){
	int r;
	while((r = x%y != 0){//ユークリッドの互除法
		x = y;
		y = r;
	}
	return y;
}

int KEISAN(int x, int y){
	int i;
	y = y-1;
	for(i=0;i<10;i++){
		y = y++;
		x = x / GCD(x,y)*y;
	}
	return x;
}

x = x/GCD(x,y)*y;

とした方がオーバーフローしにくくなります。

int KEISAN(int x, int y){
    int i;
    for(i=y;i<y+10;y++){
        y = i;
        x = x*y / GCD(x,y);
    }
    return x;
}

int KEISAN(int x, int y){
	int i;
	y = y-1;
	for(i=0;i<10;i++){
		y = y++;
		x = x / GCD(x,y)*y;
	}
	return x;
}

int KEISAN(int x, int y){
	int i;
	for(i=1;i<11;i++){
		y = y+i;
		x = x / GCD(x,y)*y;
	}
	return x;
}

3行目 i=y;により x=1,y=2,i=1 i<y+10は真

C言語ではエラーが出ましたが、C++だとエラーにならないようです。
http://ideone.com/tFOGrE
http://ideone.com/8YjEQV

【追記】
動作も完全に「rに右の等号の右の値を代入する」になっているようですね。
http://ideone.com/LKD1AE

Yellow さんが書きました: また、

コード:

int KEISAN(int x, int y){
	int i;
	for(i=1;i<11;i++){
		y = y+i;
		x = x / GCD(x,y)*y;
	}
	return x;
}

上のプログラムでも正しい値が返されなかったのですが、これがなぜかわかりません。

Yellow さんが書きました: それから、

3行目 i=y;により x=1,y=2,i=1 i<y+10は真

と書いてくださってましたが、=は右辺に左辺を代入するという意味ならば、i=y;はi=2ということにならないのでしょうか…。
このへんの理解が甘くわかりません。

Yellow さんが書きました: もっと安全な書き方があるということでしょうか？

non さんが書きました:

Yellow さんが書きました: それから、

3行目 i=y;により x=1,y=2,i=1 i<y+10は真

と書いてくださってましたが、=は右辺に左辺を代入するという意味ならば、i=y;はi=2ということにならないのでしょうか…。
このへんの理解が甘くわかりません。

みけCATさんの書き間違えでしょう。

Yellow さんが書きました:>GCD関数とmain関数だけでテストプログラムを実行してみました。GCD関数はC言語の時でもエラーを吐かずに動いていました。http://ideone.com/tFOGrEでエラーになっている理由がわか ... ということでしょうか？

Yellow さんが書きました:KEISAN関数の中身を変えました。

コード:

int KEISAN(int x, int y){
    int i;
    for(i=y;i<y+10;y++){
        y = i;
        x = x*y / GCD(x,y);
    }
    return x;
}

を、

コード:

int KEISAN(int x, int y){
	int i;
	y = y-1;
	for(i=0;i<10;i++){
		y = y++;
		x = x / GCD(x,y)*y;
	}
	return x;
}

にすると正しい値を返すようになりました。

non さんが書きました: while((r = x%y) =! 0)
が　=!　のままではエラーになるかならないかという話をしています。
もちろん、訂正されているように　!=　でないといけません。
ただし、添付されたプログラムは括弧が対応していません。

みけCAT さんが書きました:

Yellow さんが書きました:>GCD関数とmain関数だけでテストプログラムを実行してみました。GCD関数はC言語の時でもエラーを吐かずに動いていました。http://ideone.com/tFOGrEでエラーになっている理由がわか ... ということでしょうか？

証拠(スクリーンショットか動作しているURL)の提示をお願いできますか？

non さんが書きました: yの初期値が2なら
最初は、3からスタートし、5,8,12と増えますよ。

みけCAT さんが書きました:y = y++;という式は環境(コンパイラ)依存の気がします。
素直にy++;またはy = y+1;の方がいいのではないですか？