-1

繰り返し処理の各処理におけるパフォーマンスを配列に入れて、その中央値を求めるプログラムを書いています。繰り返し処理は、行列の足し算で各要素を足す処理を指しています。

前回までの質問に続いて、こちらの質問をします。

解決したいこと
中央値を求める部分で関数呼び出し後の型に関するエラーを解決したい

コンパイルエラー(errorのみ抜粋、warningは除く)
下二つのエラーは1つ目のエラーに起因すると考えています。
1つ目は型に関してのエラーであることは推測できるのですが、中央値を求めるmedian()関数を呼び出す前の型doubleと合わせて、どのようにコードを修正すればいいかわかりません。

$ g++ -o sample_med sample_median.cpp
sample_median.cpp:25:14: error: invalid operands to binary expression
      ('double' and 'int')
    if (size % 2 == 1) {
        ~~~~ ^ ~

sample_median.cpp:83:5: error: use of undeclared identifier 'ans'
    ans = median(timedata);
    ^
sample_median.cpp:84:18: error: use of undeclared identifier 'ans'; did you mean
      'abs'?
    std::cout << ans << std::endl;
                 ^~~
                 abs
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdlib.h:113:44: note: 
      'abs' declared here
inline _LIBCPP_INLINE_VISIBILITY long long abs(long long __x) _NOEXCEPT ...

現行のコード

#include <chrono>
#include <iostream>
#include <thread>
#include <vector>
#include <algorithm>
#include <iostream>

//繰り返し処理全体にかかった時間
using namespace std::chrono;

double median(std::vector<double>v) {
    double size = v.size();
    std::vector<double> _v(v.size());
    copy(v.begin(), v.end(), back_inserter(_v));
    double tmp;
    for (int i = 0; i < size - 1; i++){
        for (int j = i + 1; j < size; j++) {
            if (_v[i] > _v[j]){
                tmp = _v[i];
                _v[i] = _v[j];
                _v[j] = tmp;
            }
        }
    }
    if (size % 2 == 1) {
        return _v[(size - 1) / 2];
    } else {
        return (_v[(size / 2) - 1] + _v[size / 2]) / 2;
    }
}

int main()
{
    #define N 2

    double A[N][N] = {
        {3.0, 5.0},
        {9.0, 5.0}
    };

    double B[N][N] = {
        {3.0, 6.0},
        {8.0, 9.0}
    };

    double C[N][N] = {
        {0.0, 0.0},
        {0.0, 0.0}
    };
    int i, j, k, n;



    //各足し算にかかった時間を入れる配列
    std::vector<double> timedata;

    for(i=0; i<N; i++){
        for(j=0; j<N; j++){
                //測定開始
                std::chrono::high_resolution_clock::time_point start = std::chrono::high_resolution_clock::now();
                C[i][j] += A[i][j]+B[i][j];
                //測定終了
                high_resolution_clock::time_point end = high_resolution_clock::now();
                double time =  std::chrono::duration_cast<std::chrono::nanoseconds>(end - start).count();

                //配列にappend
                timedata.push_back(time);
            }
    }

    //ソート前
    for (auto value : timedata) {std::cout << value << ", ";
        std::cout << std::endl;
    }

    //配列をソート
    std::cout << "After Sorted"<< std::endl;
    std::sort(timedata.begin(), timedata.end());
    for (auto value : timedata) {std::cout << value << ", ";
        std::cout << std::endl;
    }

    ans = median(timedata);
    std::cout << ans << std::endl;

}

1 件の回答 1

0

まず最初のエラーについて

sample_median.cpp:25:14: error: invalid operands to binary expression
      ('double' and 'int')
    if (size % 2 == 1) {
        ~~~~ ^ ~

これは関数median(std::vector<double>v)内の変数、sizeの型の問題です。

double median(std::vector<double>v) {
    double size = v.size();

と、sizedoubleで定義していますが、double型の値に%演算子は使えないというのがエラーの意味です。sizeの使われ方を見ると、個数やインデックスとしてのみ用いられており、doubleにしなければならない理由が全くありません。ここは

double median(std::vector<double>v) {
    size_t size = v.size();

とすべきです。


次に

sample_median.cpp:84:18: error: use of undeclared identifier 'ans'; did you mean
      'abs'?
    std::cout << ans << std::endl;

について。

これは、単純にans変数が定義されていないから出ているエラーです。main()の最後の方で

    double ans = median(timedata); // ここでansを定義
    std::cout << ans << std::endl;

}

と定義しましょう。

2
  • ご回答いただきありがとうございます。ご説明いただいた通りに修正したところ、コンパイルエラーはなくなったのですが、中央値の出力が0となりました。この件は別の質問として投稿すべきでしょうか。
    – user32568
    Commented 2019年7月31日 14:05
  • @NPP 質問の内容が変わってしまう(コンパイルエラーの問題から、アルゴリズムの問題)ので、別件にした方がよいと思います。
    – Hideki
    Commented 2019年7月31日 14:11

この質問に回答するには、ログインする必要があります。