boolのreturnについてエラーがでてしまう。

Question

###実現したいこと
重複している値があるかどうかを判定する関数bool had_duplicate_numbersを作りたい。

###問題点
bool has_duplicate_numbers関数でエラーがでてしまう。
######エラー

kadai25.cpp: In function 'bool has_duplicate_numbers(std::vector<double>&)':
kadai25.cpp:26:1: warning: control reaches end of non-void function [-Wreturn-type]
   26 | }
      | ^

このエラーについて調べてみたところreturnがされていないと出てくるというような記述がありました。
しかし、for(i=0;i<n;i++)の中で記述しているので何が原因なのかわかりません。
###コード

//値の重複

#include<iostream>
#include<vector>
#include<algorithm>

//vectorに重複して値が含まれているか判定する関数
bool has_duplicate_numbers(std::vector<double>& values)
{
    int i;
    int n=values.size(); //配列のデータ数
    //データをソートする
    std::sort(values.begin(),values.end());

    //各隣接要素が同じ値を持つかどうか
    for(i=0;i<n;i++){
        if(values[i]==values[i+1]){
            return true;
        }else{
            return false;
        }
    }
}

int main()
{
    int i,n;
    std::cout<<"データ数を入力してください\n";
    std::cin>>n;
    std::cout<<"データを入力してください\n";
    //配列の定義&入力
    std::vector<double> data;
    for(i=0;i<n;i++){
        int d;
        std::cin>>d;
        data.push_back(d);       
    }
    if(has_duplicate_numbers(data)){
        std::cout<<"重複した値が含まれている\n";
    }else{
        std::cout<<"重複した値が含まれていない\n";
    }
    return 0;
}

/*実行結果
*/

Answer 1

当該の場所直前のforループがループになっていません。values[0]とvalues[1]の比較だけ行って終了してしまいます。

そこを書き直せば、自然と「返り値を返さずに終端する」コードも解決するかと思います。

Comments

[maisumakun]

なお、要素数が1以下のvectorを渡された場合、values[1]が範囲外になりますので、正しく動作しません。

[langhtorn]

アドバイスありがとうございました！

Answer 2

    //各隣接要素が同じ値を持つかどうか
    for(i=0;i<n;i++){
        if(values[i]==values[i+1]){
            return true;
        }
    }
    return false;

Comments

[langhtorn]

できました。ありがとうございました。