質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.36%

  • C++

    4069questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

C++で競技プログラミング DFS 論理エラーの特定について

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 71

hon.ki

score 5

http://poj.org/problem?id=2386の問題を、蟻本のp36をみながら解いておりましたところ、次のような論理エラーが起こりました。
input:
10 12
W........WW.
.WWW.....WWW
....WW...WW.
.........WW.
.........W..
..W......W..
.W.W.....WW.
W.W.W.....W.
.W.W......W.
..W.......W.
期待するoutput
3
実際のoutput 
(出力なし)

#include <iostream>
using namespace std;
const int MAX_N = 100;
const int MAX_M = 100;
int N, M;
char field[MAX_N][MAX_M + 1]; //庭

//現在位置(x,y)
void dfs(int x, int y)
{
    //今いるところを、.に置き換える
    field[x][y] = '.';

    //移動する8方向について,ループする
    for (int dx = -1; dx <= 1; dx++)
    {
        for (int dy = -1; dy <= 1; dy++)
        {
            //x方向にdx, y方向にdy 移動した場所を(nx,ny)とする
            int nx = x + dx, ny = y + dy;

            //nxとnyが水たまりかどうか、またfield[nx][ny]が水たまりかどうかを特定する
            if (0 <= nx && nx < N && 0 <= nx && ny < M && field[nx][ny] == 'W')
                dfs(nx, ny);
        }
    }
    return;
}
void solve()
{
    int res = 0;
    for (int i = 0; i < N; i++)
    {
        for (int j = 0; j < M; j++)
        {
            if (field[i][j] == 'W')
            {
                //Wが残っているなら、そこからdfsを始める
                dfs(i, j);
                res++;
                //cout << res << endl;
            }
        }
    }
    //cout << res << endl;
}
int main()
{
    cin >> N >> M;
    for (int i = 0; i < N; i++)
    {
        for (int j = 0; j < M; j++)
        {
            cin >> field[i][j];
        }
    }
    return 0;
    solve();
}


<試してみたこと>
入力の受け取りの後(57行目)に、

for (int i = 0; i < N; i++)
    {
        for (int j = 0; j < M; j++)
        {
            cout << field[i][j]<<endl;
        }
    }


としたところ、
W........WW..WWW.....WWW....WW...WW..........WW..........W....W......W...W.W.....WW.W.W.W.....W..W.W......W...W.......W.
という出力がありました。このことから、入出力に問題はない考えました。
solve関数の中に問題があるのではと考え、コメントアウトした2箇所で変数resを出力してみましたが、いずれも出力は何もされませんでした。といって、条件式を確認しましたが、間違っているところはわかりませんでした。

なお、ローカル環境とは、mac OS High Sierra10.13.6
コンパイラは
Apple LLVM version 10.0.0 (clang-1000.10.44.4)
Target: x86_64-apple-darwin17.7.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
になります。デフォルトのC++14で、コンパイルしました。
VSCodeを使っています。

凡ミスである可能性が高いと思うのですが、どうしてもわからず、質問させていただきました。どうぞよろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • aikon_marimo

    2019/02/10 21:55

    とりあえず、main関数内でsolve()の前にreturnしているのを直して試してみてください。

    キャンセル

  • hon.ki

    2019/02/10 21:59

    aikon_marimoさん、たしかにそこは、おかしいですね。直して試してみたところ、まだoutputは何もなし、となっております。回答ありがとうございます。

    キャンセル

回答 1

0

少なくとも例題にあった入力例では、3と表示されましたよ。

#include <iostream>
using namespace std;
const int MAX_N = 100;
const int MAX_M = 100;
//int N, M;
//char field[MAX_N][MAX_M + 1];  //庭

static const int N = 10, M = 12;
char field[N][M] = {
    {'W', '.', '.', '.', '.', '.', '.', '.', '.', 'W', 'W', '.'},
    {'.', 'W', 'W', 'W', '.', '.', '.', '.', '.', 'W', 'W', 'W'},
    {'.', '.', '.', '.', 'W', 'W', '.', '.', '.', 'W', 'W', '.'},
    {'.', '.', '.', '.', '.', '.', '.', '.', '.', 'W', 'W', '.'},
    {'.', '.', '.', '.', '.', '.', '.', '.', '.', 'W', '.', '.'},
    {'.', '.', 'W', '.', '.', '.', '.', '.', '.', 'W', '.', '.'},
    {'.', 'W', '.', 'W', '.', '.', '.', '.', '.', 'W', 'W', '.'},
    {'W', '.', 'W', '.', 'W', '.', '.', '.', '.', '.', 'W', '.'},
    {'.', 'W', '.', 'W', '.', '.', '.', '.', '.', '.', 'W', '.'},
    {'.', '.', 'W', '.', '.', '.', '.', '.', '.', '.', 'W', '.'},
};

//現在位置(x,y)
void dfs(int x, int y)
{
    //今いるところを、.に置き換える
    field[x][y] = '.';

    //移動する8方向について,ループする
    for (int dx = -1; dx <= 1; dx++) {
        for (int dy = -1; dy <= 1; dy++) {
            // x方向にdx, y方向にdy 移動した場所を(nx,ny)とする
            int nx = x + dx, ny = y + dy;

            // nxとnyが水たまりかどうか、またfield[nx][ny]が水たまりかどうかを特定する
            if (0 <= nx && nx < N && 0 <= nx && ny < M && field[nx][ny] == 'W')
                dfs(nx, ny);
        }
    }
    return;
}
void solve()
{
    int res = 0;
    for (int i = 0; i < N; i++) {
        for (int j = 0; j < M; j++) {
            if (field[i][j] == 'W') {
                // Wが残っているなら、そこからdfsを始める
                dfs(i, j);
                res++;
            }
        }
    }
    std::cout << res << std::endl;
}
int main()
{
    //    cin >> N >> M;
    //    for (int i = 0; i < N; i++) {
    //        for (int j = 0; j < M; j++) {
    //            cin >> field[i][j];
    //        }
    //    }
    solve();
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • C++

    4069questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。