テンプレートクラスを利用し、あらゆる型で処理ができるクラスcollectionを作る。

Question

前提・実現したいこと

現在C++を学習しており、テンプレートを利用してソースを書きました。
実行結果自体は正しいのですが、コンパイル時に出てしまう警告を解消したい為質問致しました。
ソースは、collection.h、main.cppです。

以下のサイトのproblex3-2について質問致します。
http://cpp-lang.sevendays-study.com/problemex3.html

以下問題文転載。↓
以下のプログラムでのクラスCollectionIntは、整数型の配列を保持し、その中から最大値と最少値を取得できるクラスである。このクラスをテンプレートを使って、double,stringなど、あらゆる型で同じ処理ができるクラス、collectionを作り、処理を書き換えなさい。

期待される実行結果
実数の場合
1.2 3.5 8.1 4.9 2.0 2.2
最大値 : 8.1
最小値 : 1.2

整数の場合
1 5 4 2 3
最大値 : 5
最小値 : 1

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

In file included from main.cpp:2:
./Collection.h:52:5: warning: control reaches end of non-void function
[-Wreturn-type]
}
^
main.cpp:10:8: note: in instantiation of member function
'Collection<double>::showArray' requested here
c->showArray();
^
1 warning generated.

該当のソースコード

#ifndef _COLLECTION_H_
#define _COLLECTION_H_
#include <iostream>

using namespace std;

template<typename T>class Collection{
private:
    //  配列データ
    T* m_pArray;
    //  配列の長さ
    int m_length;
public:
    //  コンストラクタ
    Collection(T* array,int length){
        m_pArray = new T[length];
        m_length = length;
        for(int i = 0; i < m_length ; i++){
            m_pArray[i] = array[i];
        }
    };
    //  デストラクタ
    ~Collection(){
        delete[] m_pArray;
    };
    //  最大値の取得
    T getMax(){
        T max = m_pArray[0];
        for(int i = 0; i < m_length ; i++){
            if(max < m_pArray[i]){
                max = m_pArray[i];
            }
        }
        return max;
    };
    //  最小値の取得
    T getMin(){
        T min = m_pArray[0];
        for(int i = 0; i < m_length ; i++){
            if(min > m_pArray[i]){
                min = m_pArray[i];
            }
        }
        return min;
    };
    //  成分の表示
    T showArray(){
        for(int i = 0; i < m_length ; i++){
            cout << m_pArray[i] << " ";
        }
        cout << endl;
    }
};

#endif // _COLLECTION_H_

#include <iostream>
#include "Collection.h"

using namespace std;

int main(){
    //  配列変数の定義
    double array[] = { 1.2,1.1,8.1,8.2,3.6 };
    Collection<double>* c = new Collection<double>(array,5);
    c->showArray();
    cout << "最大値 : " << c->getMax() << endl;
    cout << "最小値 : " << c->getMin() << endl;
    delete c;
    return 0;
}

試したこと

テンプレートの呼び出し方が誤っているのではと考え、main.cpp7行目の
Collection<double>* c = new Collection<double>(array,5);
を
Collection* c = new Collection<double>(array,5);
や
Collection<double>* c = new Collection(array,5);
としましたが、エラーは出たままでした。

また、インスタンス生成が誤ってるのではと考え２行に分けて
Collection<double>* c;
c = new Collection<double>(array,5);
と書いてもやはり警告は発生しました。

実行時のコマンドは
g++ -c main.cpp -o main.o ←これを行うと警告が発生する。
g++ main.o
./a.out
です。

Answer 1

(1) 変なところにセミコロンがついてます（他の場所も）

    }; <--- NG
    //  デストラクタ

(2) showArray に return文がありません。戻り値の型をvoidにしましょう

NG    T showArray(){
OK    void showArray(){

Comments

[yuki23]

質問とは関係ありませんが、ヘッダーファイルの中で using namespace std; するのは禁じ手です。絶対やめましょう

[takuan1205]

迅速な回答ありがとうございます。
回答の通りに直しました。
クラスの最後のカッコのみは }; とするのですね。また、戻り値voidに関しても理解致しました。
using namespace std; に関してもアクセス修飾子が衝突することにより、バグの原因になるとのことで書いてはいけないと出てきたので、
using namespace std; の部分を削除し、std::cout std::endl とした所、無事エラーなくコンパイルが通る様になりました。
そこで質問なのですが、using namespace std を std:: と書くのは合っているのでしょうか。私の中でいまいち理解がなってなくて、、
質問の主旨とは異なりますが、教えていただけると幸いです。

[yuki23]

> std:: と書くのは合っているのでしょうか
はい、それでOKです。
ソースファイルで using namespace std; するのも黒に近いグレーなのですが、一つのソースファイルを完全に掌握できているなら許容できる場合もあります。しかし、ヘッダーファイルとは複数のソースファイルで使うためのものなので、using namespace すると他のソースファイルに迷惑をかけます。なのでヘッダーファイルで使ってはいけません。

[takuan1205]

ありがとうございました。問題の根本だけでなく、 using namespaceに関してのアドバイスも非常に助かりました。