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

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

ただいまの
回答率

88.92%

C++で変数を関数に渡すときに変数型が一致しない(OpenCV)

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 4,264

sug1t0m0

score 40

プログラムのエラーが出る箇所を以下に抜き出しました。
なお、このプログラムは HOG特徴量の計算に掲載されているものです。

// ある矩形領域の勾配ヒストグラムを求める
// ここでいう矩形はHOG特徴量のセルに該当
void calculateHOGInCell(Mat& hogCell, Rect roi, const vector<Mat>& integrals) {
    int x0 = roi.x, y0 = roi.y;
    int x1 = x0 + roi.width, y1 = y0 + roi.height;
    for (int i = 0; i < N_BIN; i++) {
        Mat integral = integrals[i];
        float a = integral.at<double>(y0, x0);
        float b = integral.at<double>(y1, x1);
        float c = integral.at<double>(y0, x1);
        float d = integral.at<double>(y1, x0);
        hogCell.at<float>(0, i) = (a + b) - (c + d);
    }
}


// セルの大きさ(ピクセル数)
#define CELL_SIZE 20
// ブロックの大きさ(セル数)奇数
#define BLOCK_SIZE 3
// ブロックの大きさの半分(ピクセル数)
#define R (CELL_SIZE*(BLOCK_SIZE)*0.5)

// HOG特徴量を計算する
// pt: ブロックの中心点
Mat getHOG(Point pt, const vector<Mat>& integrals) {
    // ブロックが画像からはみ出していないか確認
    if (pt.x - R < 0 ||
        pt.y - R < 0 ||
        pt.x + R >= integrals[0].cols ||
        pt.y + R >= integrals[0].rows
    ) {
            return Mat();
    }

    // 与点を中心としたブロックで、
    // セルごとに勾配ヒストグラムを求めて連結
    Mat hist(Size(N_BIN*BLOCK_SIZE*BLOCK_SIZE, 1), CV_32F);
    Point tl(0, pt.y - R);
    int c = 0;
    for (int i = 0; i < BLOCK_SIZE; i++) {
        tl.x = pt.x - R;
        for (int j = 0; j < BLOCK_SIZE; j++) {
            calculateHOGInCell(hist.colRange(c, c+N_BIN),
                               Rect(tl, tl+Point(CELL_SIZE, CELL_SIZE)),
                               integrals);
            tl.x += CELL_SIZE;
            c += N_BIN;
        }
        tl.y += CELL_SIZE;
    }
    // L2ノルムで正規化
    normalize(hist, hist, 1, 0, NORM_L2);
    return hist;
}

で以下のようなエラーが出ます。

HOGTest.cpp:114:13: error: no matching function for call to 'calculateHOGInCell'
            calculateHOGInCell(hist.colRange(c, c+N_BIN),
            ^~~~~~~~~~~~~~~~~~
HOGTest.cpp:71:6: note: candidate function not viable: expects an l-value for
      1st argument
void calculateHOGInCell(Mat& hogCell, Rect roi, const vector<Mat>& integrals) {
     ^
1 error generated.
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

こんにちは。

cv::Mat::colRange()はcv::Mat型のインスタンスを返却します。
変数に代入せずに直接他の関数のパラメータへ与える場合、一時変数として渡されます。
Visual C++は一時変数を参照で受け取ってもエラーにならなかったと思いますが、gccはエラーになります。(Visual Studio 2013時代の記憶ですので、2015ではまた異なるかも知れません。)

つまり、リンク先のサイトはVisual C++で動作確認しているのではないかと思います。

適切に動作させるには下記でいけると思います。

Mat temp=hist.colRange(c, c+N_BIN);
calculateHOGInCell(temp,
                   Rect(tl, tl+Point(CELL_SIZE, CELL_SIZE)),
                   integrals);


ちょっと虚しく感じますが、標準規格的にはgccの動作の方が正しいです。


【なぜ一時変数を参照で受け取れないのか?】
通常、参照は値を返却する場合に使います。なので参照で受け取った変数はその後も「生きている」ことが期待されます。
しかし、一時変数はその文が終われば破棄されますから、関数側で何か値を設定してもそれが無駄になるのが「普通」です。
そのミスをプログラマに通知するためにエラーにしているのだろうと思います。

OpenCVのcv::Mat型はstd::shared_ptr<>のような一種のスマート・ポインタですので値の設定はそれが管理するデータへ設定します。これはcv::Matが破棄されても直ちに破棄されるわけではなく、そのデータを参照している全てのcv::Matが破棄されるまで維持されます。
つまり、上記の「普通」とは異なる使い方なので、今回の使い方では問題は発生しない筈です。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/07 13:39 編集

    書いている間に進んでました。もう少し考えてみます。

    ああ、NEWBIEEBIEEさんはMatの参照(Mat& hogCell)で受けてますが、私の回答のように参照ではなく普通の変数(Mat temp)で受ければ良いはずです。

    キャンセル

  • 2017/02/07 13:48

    ありがとうございます。コンパイラの説明までいただけて、嬉しい反面自分の勉強不足を痛感しているところです。

    キャンセル

  • 2017/02/07 13:55

    いえいえ。実は同様な問題に以前ハマったことがあるので、たまたま知っているだけなのですよ。

    キャンセル

0

expected l-valueとあるので今入っているのはr-value(右辺値)のはず、
Mat& hogCell = hist.colRange(c, c+N_BIN);
として左辺値に変えるのはどうですか。
それから該当する関数に代入しましょう

後、OpenCV2か3かわかりませんが
暗黙の型変換などにも注意した方がいいかもしれません。

for (int j = 0; j < BLOCK_SIZE; j++) {
    Mat* hogCell = hist.colRange(c, c+N_BIN); <--ここ
    calculateHOGInCell(hogCell , <--ここも変えて
    Rect(tl, tl+Point(CELL_SIZE, CELL_SIZE)),
    integrals);
    tl.x += CELL_SIZE;
    c += N_BIN;
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/07 13:34

    const Mat& hogCell = hist.colRange(c, c+N_BIN); でやってみましょう(無保証)

    キャンセル

  • 2017/02/07 13:36

    ごめんなさい、 Mat* hogCell = hist.colRange(c, c+N_BIN);だと思います。

    キャンセル

  • 2017/02/07 13:46

    Mat foo = hist.colRange(c, c+N_BIN);
    で解決しました。ありがとうございます。

    キャンセル

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

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

関連した質問

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