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

OverKuro さんが書きました: ↑

5年前

この掲示板で初めて質問させていただきます。
もう少しきれいな書き方があれば教えていただけないでしょうか

　やりたいこと：Startに設定した値から、Endに設定した値までStepの値ずつ繰り返し値を表示したい。

　条件：
　・StartとEndはかならず表示したい。
　　->Start = 5, End = 0, Step = 2の場合 : 5, 3, 1 ではなく 5, 3, 1, 0と表示したい
　・実際にはdouble型の値を使うので、for文を使いたくない。
　
現在やりたいことは実質達成できているのですが、個人的に無理やり感があるため、もう少しきれいな書き方があるかどうかを模索している最中です。

コード:

#include <stdio.h>
#include <iostream>
using namespace std;

void main(){
	//設定値
	double start = 5;
	double end = 0;
	double step = 2;
	
	//Startの値を代入
	double count = start;

	do
	{
		//countの値を表示
		cout << count << endl;
		//値がendと一致した場合ループを抜ける
		if(count == end){
			break;
		}
		//countをStepの値だけ減らす
		count -= step;
		
		//countの値がend未満の場合、countの値をendに変更してコンティニュー
		if(count < end){
			count = end;
			continue;
		}
	} while (count >= end);	//結果：5, 3, 1, 0 と表示されます

//他に試した文
	//do
	//{
	//	cout<< count << endl;
	//	count -= step;
	//	if(count < end){
	//		count = end;			//結果：5, 3, 1, 0, 0, 0 ･･･
	//	}
	//	
	//} while (count >= end);

	//do
	//{
	//	cout<< count << endl;
	//	
	//	if(count < end){
	//		count = end;			//結果：5, 3, 1
	//	}
	//	count -= step;
	//} while (count >= end);
}

他に良い書き方がある、もしくはこうすればもっときれいに書けるなど
アドバイスがいただければ幸いです。

環境 ：Windows10上でVisualStudio2012を使用しています。

かずま さんが書きました: ↑

5年前

End は Start より必ず小さいのですか？

コード:

#include <iostream>
using namespace std;

int main()
{
	double start = 5, end = 0, step = 2;

	double count = start;
	do {
		cout << count << endl;
		count -= step;
	} while (count > end);
	cout << end << endl;
}

#include <stdio.h>
#include <iostream>
using namespace std;

int main(){
	//設定値
	double Begin = 5;
	double End = 0;
	double Step = 2;
	//表示用カウント
	double Count = Begin;

	do
	{
		//値を表示
		cout << Count << endl;

		//カウントがEndより大きい場合
		if(Count > End){
			//Stepの値だけ減らす
			Count -= Step;
			//CountがEnd未満なら値をEndに合わせる
			if(Count < End) Count = End;
		//Down未満ならループを抜ける
		}else break;

	} while (Count >= End);	//End以上ならループ

	return 0;
}

tanu_kichi さんが書きました: ↑

5年前

プログラムの条件を拝見する限り、私も似たようなコーディングをすると思います。

重箱の隅を突っつくようで申し訳ありませんが、気になった点を述べますと、
１． void main ( ) はC++の仕様上許されていないかと思います。
　　こちらの環境　OS:Ubuntu 18.04.3 LTS コンパイラ:g++ 7.4.0　でエラーがでます。

コード:

　　int main() {
         ...
         return 0;
      }

　　と書く方がいいかと思います。

２．英語の場合、慣用的に start , stop あるいは begin , end が対になります。
　　start , end は、英語を母国語とされる方は通常は使いません。
　　begin , end は start , stop より少しフォーマルな（堅い）ニュアンスがあります。

３．これは無視できることですが、当ケースでは

コード:

  do {
     ...
  } while ( ... );

と

コード:

  while ( ... ) {
     ...
  }

は同等です。
do { } while ( ) ; は while 条件式が偽であっても、１回は処理される。
while ( ) { }　はwhile 条件式が偽である場合、１回も処理されない。