1

Не очень понимаю, что я делаю не так. Программа просто завершается (с кодом 0) и ничего не выводит.

struct Date {
    int day;
    int month;
    int year;

    char *toString() {
        char *out = nullptr;
        sprintf(out, "%d-%d-%d", day, month, year);
        return out;
    }

    Date(int day, int month, int year) {
        this->day = day;
        this->month = month;
        this->year = year;
    }
};

int main() {
        Date *date = new Date(25, 11, 17);
        printf("%s", (*date).toString());
}
3
  • Так char *out = nullptr; писать нельзя. Дата выводится "вникуда" и как итог - пусто в return out.
    – nick_n_a
    7 ноя 2017 в 13:25
  • 1
    Замени char *out = nullptr; на char *out = new char[50]; 7 ноя 2017 в 13:25
  • 1
    Вот эта задача показывает, что в "перспективе" char *out = new char[50] приведёт к утечке памяти, что бы её не было, нужно ссылки коллекционировать - потом освобождать, позже это стало GC называться. Ещё как вариант "для отладки" или для single-thread можно подставлять "статический буфер". Но в даном случае new можно использовать.
    – nick_n_a
    7 ноя 2017 в 13:28

1 ответ 1

6

Ну кто же пишет по нулевому адресу?

    char *out = nullptr;
    sprintf(out, "%d-%d-%d", day, month, year);

Сделайте, например, так:

struct Date {
    int day;
    int month;
    int year;
    char buf[20];

    char *toString()
    {
        sprintf(buf, "%d-%d-%d", day, month, year);
        return buf;
    }

и в конструкторе еще сразу же вызов toString() или buf[0]=0;.

5
  • nullptr это же не обязательно нулевой адрес? 7 ноя 2017 в 15:42
  • 1
    Это в любом случае несуществующий адрес - в смысле, который не может принадлежать ни одному объекту...
    – Harry
    7 ноя 2017 в 15:49
  • T.е. термин "нулевой адрес" здесь подразумевает не адрес ячейки под номером ноль, а нулевой указатель? 7 ноя 2017 в 22:01
  • А ещё Герб Саттер советует запомнить одно простое правило: "никогда не используйте sprintf".
    – retorta
    8 ноя 2017 в 10:27
  • @retorta В данном случае проблема не в этом :) Кстати, любая паранойя хороша в меру: если выделить достаточный буфер, то можно спокойно работать - так как здесь ситуация точно известная, зная, длиннее чего не окажется вывод int, проблемы с выходом за границы можно гарантированно избежать.
    – Harry
    8 ноя 2017 в 12:33

Ваш ответ

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

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