テンプレートクラスの二つのメンバ変数にメモリの動的割り当てをするのがうまくいきません。

インスタンス作成時に(電話番号,"名前の文字リテラル")を引数にしてメンバ変数を初期化するクラスを作成したのですが、以下のようなエラーが出てしまい、ググってみてもいまいち理解ができません。
どなたかわかる方がいましたら教えてくださいませ。
c++study.cpp:35:20: error: no matching constructor for initialization of
'phones<long>'
phones < long >a(10192020,"miffy");
^ ~~~~~~~~~~~~~~~~
c++study.cpp:9:5: note: candidate constructor not viable: no known conversion
from 'const char [6]' to 'const char' for 2nd argument
phones( T f,const char c)
^
c++study.cpp:5:7: note: candidate constructor (the implicit copy constructor)
not viable: requires 1 argument, but 2 were provided
class phones{
^
c++study.cpp:36:20: error: no matching constructor for initialization of
'phones<long>'
phones < long >b(19585,"john");
^ ~~~~~~~~~~~~
c++study.cpp:9:5: note: candidate constructor not viable: no known conversion
from 'const char [5]' to 'const char' for 2nd argument
phones( T f,const char c)

c++
1#include <iostream>
2using namespace std;
3
4template <class T>
5class phones{
6    T *p;
7    char *ch;
8public:
9    phones( T f,const char c)
10    {
11        cout << "コンストラクタ起動\n";
12        p = new T;
13        ch = new char;
14        
15        *p = f;
16        *ch = c;
17    }
18    ~phones()
19    {
20        cout<<"デストラクタ起動\n";
21        delete p;
22        delete ch;
23    }
24    
25    void show(){cout <<"電話番号は:"<< *p << "\n";
26        cout << "名前は:" << *ch << "\n";
27        cout << "show起動\n";
28    }
29    
30};
31
32
33int main()
34{
35    phones < long >a(10192020,"miffy");
36    phones < long >b(19585,"john");
37    a.show();
38    b.show();
39    return 0;
40    
41}
42
43
44

行動規範の内容に同意します

回答3件

ベストアンサー

名前の表示は

cpp
1//cout << "名前は:" << *ch << "\n"; ではなく
2cout << "名前は:" << ch << "\n";

とすると出力されるかなと思います。
あと配列はdeleteの方法がdeleteではなくdelete[]になります。

投稿2020/05/17 05:30

退会済みユーザー

総合スコア0

そもそもnew/deleteを書くべきではありません。代わりにスマートポインタを利用してください。また、そのうち分かるようになると思いますがさしあたり現在はおまじないとしてデストラクタにはnoexcept指定をする習慣をつけておきましょう。(当然例外を投げたらダメ)

cpp
1#include <iostream>
2#include <memory>
3#include <cstring>
4using std::cout;
5
6template <class T>
7class phones{
8    std::unique_ptr<T> p;
9    std::unique_ptr<char[]> ch;
10public:
11    phones( T f,const char* c) : p(std::make_unique<T>(f)), ch()
12    {
13        cout << "コンストラクタ起動\n";
14        const auto len = std::strlen(c);
15        ch = std::make_unique<char[]>(len + 1);
16        std::memcpy(ch.get(), c, len + 1);
17    }
18    ~phones() noexcept
19    {
20        cout<<"デストラクタ起動\n";
21    }
22
23    void show(){cout <<"電話番号は:"<< *p << "\n";
24        cout << "名前は:" << ch.get() << "\n";
25        cout << "show起動\n";
26    }
27};
28
29
30int main()
31{
32    phones < long >a(10192020,"miffy");
33    phones < long >b(19585,"john");
34    a.show();
35    b.show();
36    cout << std::flush;
37    return 0;
38}