ループ抜け後、変数に足算し、再度ループを回したいです。

Question

入力H × W 個の '$' で描かれた長方形を出力するプログラムを作成しています。
例ーーーーーーーーーーーーーーーーー
入力が「3 4」[5 6]の場合。
$$$$
$$$$
$$$$

$$$$$$
$$$$$$
$$$$$$
$$$$$$
$$$$$$
ーーーーーーーーーーーーーーーーーー

私が作成したプログラムが下記となります。簡単にご説明させていただきます。
まず、入力された数値を配列に格納します。そして、配列の各要素に多重ループを用いて参照し、"$"を出力しようと思った次第です。
※配列vec(300)に関してですが、配列の要素数には変数が適用できないと考え、適当に設定しました。

#include <bits/stdc++.h>
using namespace std;

int main(){ 
  int l,z=0;
  vector<int>vec(300);
  
  while(cin>>l){
      vec[z] = l;
      z++;
  }
  
  //cout << vec[0] << " " << vec[1]<< " " << vec[2] << " " << vec[3] << endl;
  
  int count = 0;

    for(int i=0; i<vec[count]; i++){
      for(int p=1; p<vec[count+1]; p++){
        cout << "#";
      }
      cout << "#" << endl; 
    }
  
}

最初の入力の[3 4]であれば、望んでいる出力結果となります。
しかし、次に入力される[5 6]等を上手く出力できずに困っています。
count変数を用意し、ループ終了直後に、countを＋２して、再度ループを回す事ができれば、配列の次の要素にアクセスできるのではと思いました。（第1ループ：vec[0][1]を参照。→ループを抜けた後にcountに+2→第2ループ：vec[2][3]を参照。....以下ループのようなイメージです。）
しかし、ループを抜けた後に、count+2して再度ループを回すやり方がどうしても思いつきません。
何か良い方法はありますでしょうか。ご助言いただけると幸いです。何卒よろしくお願いいたします。

Comments

[BeatStar]

> 配列vec(300)に関してですが、配列の要素数には変数が適用できないと考え…

違いますよ。そもそも前提が違う。確かにstd::vectorも配列ではあるけど、動的配列(C++でいうnew/deleteでやる配列)をクラスのメンバ変数として包んでメンバ関数を介してアクセスするやつです。生成時に要素数を指定する場合はコンストラクタに渡しているため、関数にとっての引数のようなものです。
だから一応生身のデータでも変数でも可能。

[BeatStar]

後、入力値はどのように渡されますか？
同じ標準入力を使うにしても、
「縦 横」をペアで一行で渡されてそれの複数行なのか、一括で「縦 横 縦 横 …」と一行で渡されるのか、縦と横をそれぞれ一行で渡される(つまりニ行で一つのペア)のかでも変わってきますよ。

やろうと思えば質問のコードをベースにして修正することは可能ですが、回りくどくなるので…

[fana]

（すっごいどうでもいい話だけど，$って指定されているなら$を出力しようよ）

Answer 1

まずvecはサイズ300など与えずに0で構築して、各数値を格納します。
あとは以下のようにcountの増分値を+2でvecサイズまでループすればよいです。

	for( int count = 0; count < vec.size(); count += 2){
		cout << vec[count] << " " << vec[count+1] << endl;
	}

Comments

[Moco]

ご回答いただき、誠にありがとうございます。
vec(300)からvec(0)に変更して各数値を格納し、頂いたプログラムを追記して実行しましたが、エラーが発生してしまいました。
ご教示いただいた訂正箇所を、私が間違えてプログラム訂正している為だと考えられます。
もしよろしければ、私のプログラムのどの位置にどう追記すれば良いのか、教えていただきたく存じます。
お忙しいところ大変恐縮ですが、何卒よろしくお願いいたします。

[can110]

こちらではとくにエラーなど発生しません。
エラーメッセージ、発生するコードなどを提示すると何か分かるかもしれません。

[Moco]

お世話になっております。
返信が遅れてしまい、大変申し訳ございません。
エラーコードが「139」となった為、恐らく配列の範囲外にアクセスし、Runtime Errorが出たのかと思われます。また、vec(300)の場合、下記のコードは作動しますが、vec(0)にすると作動しません。
下記に追記したコードを提示させていただきます。
#include <bits/stdc++.h>
using namespace std;

int main(){
int l,z=0;
vector<int>vec(0);

while(cin>>l){
vec[z] = l;
z++;
}

//cout << vec[0] << " " << vec[1]<< " " << vec[2] << " " << vec[3] << endl;

int count = 0;
for (int count = 0; count < vec.size(); count += 2){
for(int i=0; i<vec[count]; i++){//i<3間ループ。//1⃣i=0,p=1//i=0,p=2//i=0,p=3
for(int p=1; p<vec[count+1]; p++){//p<4間ループ。//1⃣i=0,p=1//i=0,p=2//i=0,p=3
cout << "#";
}
cout << "#" << endl;
}
}

}

お忙しい中大変恐縮ですが、ご助言いただけると幸いです。何卒よろしくお願いいたします。

[can110]

「vec[z] = l」のように代入するのではなく追加しないとだめです。
具体的にどうしたらいいか分からない場合はvectorの基本的な使い方を学んでください。

[Moco]

「vec[z] = l」を「vec.push_back(l);」に変更したら無事に解決しました。
数多くの質問に対して、ご丁寧に解答していただき、誠にありがとうございました。大変助かりました。

Answer 2

H と W が入力されるごとに、'$' で長方形を描けば、配列も vector も要らないでしょう。

#include <iostream>  // cin, cout, endl
#include <string>    // string
using namespace std;

int main()
{
	int h, w;
	while (cin >> h >> w)
		for (string s(w, '$'); --h >= 0; cout << s << endl) ;
}

追記
なお、質問のコードを修正するとすれば、int count = 0; を
for (int count = 0; count < z; count += 2) に変更するだけ。

Comments

[Moco]

ご回答ありがとうございます。
配列もvectorも要らない方法がある事を知り、驚きました。簡単明瞭で、感動しております。
追記のコードに関しましても、正常に作動しました。
ご丁寧に説明していただき、誠にありがとうございました。非常に助かりました。

Answer 3

for( int count=0; count<2; count++ )で回したら良いかと思います。

Comments

[Moco]

類似コードを追記させていただき、無事に作動いたしました。
非常に助かりました。ご回答いただき、誠にありがとうございました。