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

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

ただいまの
回答率

90.49%

  • C

    3707questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

C言語 画像処理 メディアンフィルタ アクセス違反の原因について

受付中

回答 1

投稿

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

APE

score 2

前提・実現したいこと

メディアンフィルタを使用してノイズが入った画像のノイズを除去するプログラムを作っています。

val[i++] = bufimage->pdata[y + a][x + b][k];
の部分で読み取りアクセス違反が発生し、実行ができませんでした。
原因がわからないため教えてください。

説明不足かもしれませんが、宜しくお願い致します。

発生している問題・エラーメッセージ

 例外がスローされました:読み取りアクセス違反。
**bufimage** 0x80 でした。 が発生しました

該当のソースコード

void median(Image image)
{
    int x, y, k, m, n;
    int i=0;
    int a = 0;
    int b = 0;
    int buf=0;
    Image bufimage;
    int val[9];
    int N = 0;
    printf("フィルタサイズ選択してください。\n");
    printf("フィルタサイズ:");
    scanf_s("%d", &N);
    N = 2 * N + 1;

    bufimage = createImage(image->height, image->width);        

    for (y = 0; y < image->height; y++) {
        for (x = 0; x < image->width; x++) {
            for (k = 0; k < N; k++)
            {
                bufimage->pdata[y][x][k] = image->pdata[y][x][k];    
            }
        }
    }

    //フィルタ処理
    for (y = (N / 2); y < image->height - (N / 2); y++) {
        for (x = (N / 2); x < image->width - (N / 2); x++) {
            for (k = 0; k < 3; k++) {

                //値をソート用バッファにコピー
                if(i<N*N)
                {
                    for (a = -(N / 2); a <= (N / 2); a++)
                    {
                        for (b = -(N / 2); b <= (N / 2); b++)
                        {
                            val[i++] = bufimage->pdata[y + a][x + b][k];
                        }
                    }
                }

                //ソート
                for (m = 0; m < (N*N) - 1; m++)
                    for (n = m + 1; n<N*N; n++)
                        if (val[m] < val[n])
                        {
                            buf = val[m];
                            val[m] = val[n];
                            val[n] = (byte)buf;
                        }

                image->pdata[y][x][k] = (byte)(val[(N*N) / 2]);
            }
        }
    }
}

試したこと

補足情報(言語/FW/ツール等のバージョンなど)

C言語

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

まずは
val[i++] = bufimage->pdata[y + a][x + b][k];の部分を

if( (y + a) < 0 || (y + a) >= image->height || 
    (x + b) < 0 || (x + b) >= image->width  || 
    i < 0 || i >= 9 ){
        printf( "x=%d,y=%d,a=%d,y=%d,i=%d\n", x, y, a, b, i);
    }
}


などに置き換えて、データ範囲外をアクセスしていないか確認しましょう。
そして範囲外アクセスしている場合はロジックを見直しましょう。

ちなみに
for (k = 0; k < N; k++)
の部分はどのような処理を意図していますか?(意図した動作をしていないのでは?)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/11 18:52

    val[i++] = bufimage->pdata[y + a][x + b][k];の部分を載せてくれたif文に変えると、最後の行のimage->pdata[y][x][k] = (byte)(val[(N*N) / 2]); にアクセル違反が発生しました。

    すみません間違えていました。
    for (k = 0; k < 3; k++)
    これで大丈夫ですか?
    宜しくお願い致します。

    キャンセル

  • 2018/01/11 19:01

    pdata[y][x][k]の[k]が何を表しているかは(説明されない限り)コードを書いた本人にしか分かりません。
    厳しい言い方になりますが、大丈夫かどうかは、あなた(コードを書いた本人)にしか判断できません。
    アクセス違反についても同様です。

    キャンセル

  • 2018/01/11 23:49

    ImageやcreateImageの中身がわからないので、何とも言えませんね。
    ○○さんが作った、○○というライブラリを使ってます!ドキュメントはhttp://○○/で読めます! のような感じの情報があれば別ですが。

    キャンセル

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

  • ただいまの回答率 90.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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

  • C

    3707questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。