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

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

新規登録して質問してみよう
ただいま回答率
85.50%
関数

関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

C++

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

Q&A

解決済

2回答

2447閲覧

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

langhtorn

総合スコア104

関数

関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

C++

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

0グッド

0クリップ

投稿2020/06/26 03:15

###実現したいこと
重複している値があるかどうかを判定する関数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++)の中で記述しているので何が原因なのかわかりません。
###コード

C++

1//値の重複 2 3#include<iostream> 4#include<vector> 5#include<algorithm> 6 7//vectorに重複して値が含まれているか判定する関数 8bool has_duplicate_numbers(std::vector<double>& values) 9{ 10 int i; 11 int n=values.size(); //配列のデータ数 12 //データをソートする 13 std::sort(values.begin(),values.end()); 14 15 //各隣接要素が同じ値を持つかどうか 16 for(i=0;i<n;i++){ 17 if(values[i]==values[i+1]){ 18 return true; 19 }else{ 20 return false; 21 } 22 } 23} 24 25int main() 26{ 27 int i,n; 28 std::cout<<"データ数を入力してください\n"; 29 std::cin>>n; 30 std::cout<<"データを入力してください\n"; 31 //配列の定義&入力 32 std::vector<double> data; 33 for(i=0;i<n;i++){ 34 int d; 35 std::cin>>d; 36 data.push_back(d); 37 } 38 if(has_duplicate_numbers(data)){ 39 std::cout<<"重複した値が含まれている\n"; 40 }else{ 41 std::cout<<"重複した値が含まれていない\n"; 42 } 43 return 0; 44} 45 46/*実行結果 47*/

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

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

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

投稿2020/06/26 03:20

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2020/06/26 03:22

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

2020/06/26 03:34

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

0

ベストアンサー

c++

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

投稿2020/06/26 03:21

asm

総合スコア15147

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

langhtorn

2020/06/26 03:34

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問