参照を、このときどうつかっていいのかわかりません。

[ピクルス]

#define _CRT_SECURE_NO_WARNINGS


#include <iostream>
#include <cstring>
#include<cstdlib>
using namespace std;

class samp {
char *s;
public:
samp() { cout << "a"; s = '\0'; }
~samp() { if (s) free(s); cout << "sを開放する\n"; }
void show() { cout << s << "\n"; }
void set(char* str);
};

void samp::set(char* str) {
s = (char*)malloc(strlen(str) + 1);
if (!s) {
cout << "メモリ割り当てエラー\n";
exit(1);
}

strcpy(s, str);

}

samp input() {

char s[80];
samp str;
cout << "文字列の入力： ";
cin >> *s;

str.set(s);
return str;
}

int main()
{
samp ob;

ob = input();
ob.show();

return 0;
}


というプログラムを書いたところ、　a のみしか表示されません。
参照の＆だけ使えば解決できるらしいのですが、この場合どこにも＆が入れられる場所がないように思えます。
どのようにすれば、このプログラムはきちんと動くのでしょうか。

[ピクルス]

C++の質問です。

コード:

#define _CRT_SECURE_NO_WARNINGS


#include <iostream>
#include <cstring>
#include<cstdlib>
using namespace std;

class samp {
    char *s;
public:
    samp() { cout << "a"; s = '\0'; }
    ~samp() { if (s) free(s); cout << "sを開放する\n"; }
    void show() { cout << s << "\n"; }
    void set(char* str);
};

void samp::set(char* str) {
    s = (char*)malloc(strlen(str) + 1);
    if (!s) {
        cout << "メモリ割り当てエラー\n";
        exit(1);
    }

    strcpy(s, str);

}

samp input() {

    char s[80];
    samp str;
    cout << "文字列の入力： ";
    cin >> *s;

    str.set(s); 
    return str;
}

int main()
{
    samp ob;

    ob = input();
    ob.show();

    return 0;
}

というプログラムを書いたところ、　a のみしか表示されません。
参照の＆だけ使えば解決できるらしいのですが、この場合どこにも＆が入れられる場所がないように思えます。
どのようにすれば、このプログラムはきちんと動くのでしょうか。

[あたっしゅ]

コード:

#define _CRT_SECURE_NO_WARNINGS


#include <iostream>
#include <cstring>
#include<cstdlib>
using namespace std;

class samp {
    char *s;
public:
    samp() { cout << "a"; s = '\0'; }
    ~samp() { if (s) free(s); cout << "sを開放する\n"; }
    void show() { cout << s << "\n"; }
    void set(char* str);
};

void samp::set(char* str) {
    cout << __LINE__ << ":" << str << endl;
    s = (char*)malloc(strlen(str) + 1);
    if (!s) {
        cout << "メモリ割り当てエラー\n";
        exit(1);
    }

    strcpy(s, str);

}

void input( samp& ob ) {
    char s[80];
    cout << "文字列の入力： ";
    cin >> s;
    cout << __LINE__ << ":" << s << endl;

    ob.set(s); 
}

int main()
{
    samp ob;

    input( ob );
    ob.show();
    
    return 0;
}


// end.

とりあえず、元のプログラムをできるだけ残して、修正。
https://www.onlinegdb.com/ で確認。

[みけCAT]

とりあえず、元のプログラムをできるだけ残して、修正。

コンパイルエラーになりました。

コード:

prog.cc: In constructor 'samp::samp()':
prog.cc:12:31: error: invalid conversion from 'char' to 'char*' [-fpermissive]
   12 |     samp() { cout << "a"; s = '\0'; }
      |                               ^~~~
      |                               |
      |                               char

どのようにすれば、このプログラムはきちんと動くのでしょうか。

• samp::sはポインタなので、文字'\0'ではなくポインタを代入するようにする
• sampのコピーによりポインタがコピーされ、二重開放などの問題の原因になるのを防ぐため、
  内容をコピーするためのコピーコンストラクタと代入演算子を定義する(The Rule of Three)
• input関数内でsをデリファレンスするのをやめ、文字ではなく文字列を入力するようにする

コード:

#define _CRT_SECURE_NO_WARNINGS


#include <iostream>
#include <cstring>
#include<cstdlib>
using namespace std;

class samp {
    char *s;
public:
    samp() { cout << "a"; s = NULL; }
    samp(const samp& sa) {
        s = NULL;
        if (sa.s != NULL) set(sa.s);
    }
    ~samp() { if (s) free(s); cout << "sを開放する\n"; }
    samp& operator=(const samp& sa) {
        if (&sa != this) {
            free(s);
            s = NULL;
            if (sa.s != NULL) set(sa.s);
        }
        return *this;
    }
    void show() { cout << s << "\n"; }
    void set(char* str);
};

void samp::set(char* str) {
    s = (char*)malloc(strlen(str) + 1);
    if (!s) {
        cout << "メモリ割り当てエラー\n";
        exit(1);
    }

    strcpy(s, str);

}

samp input() {

    char s[80];
    samp str;
    cout << "文字列の入力： ";
    cin >> s;

    str.set(s);
    return str;
}

int main()
{
    samp ob;

    ob = input();
    ob.show();

    return 0;
}