1

Изучаю C++. Буду очень признателен за любую помощь. Не нашел ответа на сайте. Нужно переопределить оператор присваивания. Сейчас сделано так:

    Stroka& Stroka::operator=(const Stroka& s)
{
    strcpy(str, s.str);
    return *this;
}

Но нужно, чтобы он работал с использованием конструкции вида - без использования strcpy:

void Stroka::kopir(char *first, char *second)
{
    while (*second) {
        *first++ = *second++;
    }
    *first = '\0'; // символ перевода строки, = \n
}

Полный код программы взят из учебного примера, выглядит следующим образом:

#include "pch.h"
#include <iostream>
#include <iomanip>
#include <stdlib.h>
#include <conio.h>
#include <stdio.h>
#include <windows.h>
#include <string.h>

using namespace std;

class Stroka {
    char str[80];
public:
    // Конструктор класса 
    Stroka(char*);
    Stroka(const char*);
    // Конструктор класса 
    Stroka() {}
    Stroka(const Stroka&); // Конструктор копирования
    Stroka& operator=(const Stroka&); // переопределение/перегрузка операции присваивания
    Stroka& operator+(const Stroka&); // переопределение операции сложения
    int operator==(const Stroka&); // переопределение операции сравнения
    void vvod();
    void vyvod();
    void kopir(char *first, char *second);
}
;

Stroka::Stroka(char *string)
{
    strcpy(str, string);
}

Stroka::Stroka(const Stroka& s)
{
    strcpy(str, s.str);
}

Stroka& Stroka::operator+(const Stroka& s)
{
    strcat(str, s.str);
    return *this;
}

// метод класса Stroka
int Stroka::operator==(const Stroka& s)
{
    if ((strcmp(str, s.str)) == 0)
        return 1;
    else
        return 0;
}

void Stroka::vvod()
{
    cout << "Введите текст:" << endl;
    cin >> str;
}

void Stroka::vyvod()
{
    cout << "Вывод текста на экран:" << endl;
    cout << str << endl;
}

Stroka& Stroka::operator=(const Stroka& s)
{
    strcpy(str, s.str);
    return *this;
}

void Stroka::kopir(char *first, char *second)
{
    while (*second) {
        *first++ = *second++;
    }
    *first = '\0'; // символ перевода строки, = \n
}

int main(int)
{
    setlocale(0, ""); // установить русскую локацию
    Stroka s1;
    s1.vvod();
    Stroka s3;
    s3.vvod();
    Stroka s4;
    s4 = s1;
    Stroka s5;
    s3.vyvod();
    s5 = s1 + s3 + s4;
    s5.vyvod();
    if (s1 == s5)
        cout << "строки s1 и s5 равны";
    else
        if (s1 == s4)
            cout << "строки s1 и s4 равны";

}
;
6
  • 2
    А проблема в чем? Ну используйте вместо strcpy - вашу kopir. Только имеет смысл сделать ее статической. Еще - непонятен комментарий // символ перевода строки, = \n, ну а саму функцию я бы сократил до одной строки - while (*first++ = *second++); - если не нужна защита от дурака типа не нулевые ли указатели переданы... Ну, и фиксированный размер массива для строки тоже несколько смущает. Что будете делать, если пользователь введет слово в 90 символов?
    – Harry
    28 янв 2019 в 10:17
  • 1
    А вам здесь вообще не нужно самому писать ни копирующий конструктор, ни копирующее присваивание. Компилятор сгенерирует их за вас. 28 янв 2019 в 11:29
  • @HolyBlackCat, и как компилятор выполнит присвоение массивов? К тому же ему нужно просто менять массив на указатель 28 янв 2019 в 11:40
  • @ARHovsepyan "и как компилятор выполнит присвоение массивов?" Немного не понимаю вопрос. Просто присвоит каждому элементу одного массива соответствующий элемент другого. "нужно просто менять массив на указатель" Да, так было бы намного удобнее... 28 янв 2019 в 12:04
  • @HolyBlackCat, вот так и нужно обьяснять. Будет понятнее для незнающих. Я специально задал вопрос... 28 янв 2019 в 12:15

1 ответ 1

1

Можно вот так:

Stroka& Stroka::operator=(const Stroka& s)
{
    this->kopir(str, s.str);
    return *this;
}
3
  • а можно и без this, просто kopir(str, s.str); 28 янв 2019 в 11:30
  • Если так переопределить присваивание, то выдается ошибка "аргумент типа const char *" не совместимы с параметром типа "char *". 28 янв 2019 в 12:37
  • @kornilcdima тип второго параметр метода? kopir() стоит поменять на const char * - во первых ошибка исчезнет, во вторых будет понятно что этот метод не собирается менять второй параметр.
    – Slava
    28 янв 2019 в 16:02

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.