#の配置がおかしい

[ダイキ]

たてHよこWの周りが＃中が.の長方形を描くプログラムで＃の配置がおかしいです
どこがちがうのか教えて下さい

コード:

#include <iostream>
#include <algorithm>
#include <iomanip>
#include <string>
#include <cmath>
using namespace std;
int main(){
	int H;
	int W;
	int j;
	int i;
	for(H=1,W=1;;){
		cin>>H>>W;
		if(H==0&&W==0)break;
		for(i=0; i<H; i++){
			for(j=0; j<W; j++){
			if(i==0||j==0||i==W||j==H||i==H||j==W)cout<<"#";
			else cout<<".";
			}
			cout<<endl;
		}
		cout<<endl;
	}
	return 0;
}

[box]

こんな感じ？if文の条件によけいなものが入っているのと、1つ値がずれているところを直しました。

コード:

#include <iostream>

using namespace std;

int main(void)
{
    int H, W, i, j;

    while (1) {
        cin >> H >> W;
        if (H == 0 && W == 0) break;
        for (i = 0; i < H; i++) {
            for (j = 0; j < W; j++) {
                if (i == 0 || j == 0 || i == H - 1 || j == W - 1) cout << "#";
                else cout << ".";
            }
            cout << endl;
        }
        cout << endl;
    }
    return 0;
}

[ダイキ]

できました。ありがとうございます
よろしければif文のな中の意味をおしえていただけますか？

[box]

よろしければif文のな中の意味をおしえていただけますか？

その質問が出るということは、元のコードでは意味がわからずにif文を書いていたということですか？
それはまずいですね。さておき、

コード:

        if (H == 0 && W == 0) break;

高さHと幅Wが両方とも0だったら、無限ループを抜けます。

コード:

                if (i == 0 || j == 0 || i == H - 1 || j == W - 1) cout << "#";

変数iは「何行目か」つまり高さをコントロールする変数で、
変数jは「何列目か」つまり幅をコントロールする変数です。
i == 0
は、いちばん上の行のことです。
j == 0
は、いちばん左の列のことです。
i == H - 1
は、いちばん下の行のことです。
j == W - 1
は、いちばん右の列のことです。

つまり、この4つの条件のいずれかが成り立つことで、長方形の周囲を表わします。

元のコードにおける問題点は、1つズレていたことを除くと、
高さをコントロールするはずの i と幅Wとを比較していること
幅をコントロールするはずの j と高さHとを比較していること
です。

[ダイキ]

ありがとうございます