Правильно решено задание?

Question

[evg_96]

В общем то задача тривиальная и по сути очень легкая, но у меня не было опыта работы c C++, все таки подводных камней в нем много... Только начал изучать его. Подскажите пожалуйста, правильно ли описано решение.
Нужно было написать программу которая запрашивает имя, фамилию, а затем конструирует, сохраняет и отображает строку в виде "Фамилия, Имя". Использовать нужно было char и функции из cstring.

Вроде как работает, но мало ли. Может стоит сделать как то по другому? И как можно избавиться от статического размера массивов?

#include <iostream>

using namespace std;

const int Buffer_size = 100;

int main()
{
	char first_name[Buffer_size];
	char last_name[Buffer_size];
	char full_name[Buffer_size * 2];

	cout << "Enter your first name: ";

	cin >> first_name;

	cout << "Enter your last_name: ";

	cin >> last_name;

	strcpy_s(full_name, Buffer_size + 1, last_name);
	strcat_s(full_name, ", ");
	strcat_s(full_name, first_name);

	cout << "Here's the information in a single string: " << full_name << endl;

	system("pause");

	return 0;
}

Comments

[Толстый Лорри]

Сюда есть пара змечаний:
strcpy_s(full_name, Buffer_size + 1, last_name);
Здесь второй параметр - это размер буфера назначения. Но у нас:
sizeof(full_name) == Buffer_size * 2

То, что вы написали Buffer_size + 1 (даже если буфер был бы размера Buffer_size) не имеет смысла, ибо нуль-символ внутри буфера, а еще +1 "залезло" бы за границу массива, с этого момента при некотором стечении обстоятельств (копируемая строка была бы на 1 символ длиннее, чем буфер назначения) поведение программы было бы не определено (undefined behaviour).

[evg_96]

Толстый Лорри, Что то не совсем понял про второй параметр. Он просто указывает размер буфера куда будет скопировано значение? В данном случае стоило бы указать sizeof full_name или что то другое? Не понятно конкретно для чего это параметр и на что он влияет, потому что даже без указания его все работает.

[Толстый Лорри]

evg_96, это специфика безопасной функции с *_s: вы явно указываете размер принимающего буфера (первый параметр), все, что туда не влезет (строка-источник из третьего параметра оказалась больше) - будет обрезано и память лежащая за целевым буфером ним не повредится.

Такая себе проверка границ массива для бедных (=

[evg_96]

Толстый Лорри,
Получается если написать так, то при вводе например строки большей чем буфер назначения, все лишнее обрежется и в буфер полностью заполнится?
strcpy_s(full_name, sizeof full_name, last_name);

Возможно я что то не так понял, но в таком примере, при вводе строки больше 9 символов вылетает вот такое вот окно. Это является ошибкой? Что это значит?

#include <iostream>

using namespace std;

const int Buffer_size = 10;

int main()
{
	char last_name[Buffer_size];
	char full_name[Buffer_size];

	cout << "Enter your last_name: ";

	cin >> last_name;

	strcpy_s(full_name, sizeof full_name, last_name);

	cout << "Here's the information in a single string: " << full_name << endl;

	system("pause");

	return 0;
}

[evg_96]

Толстый Лорри,
Я нашел еще вот такое вот решение на github. Насколько оно лучше моего решения? И что лучше использовать strncpy или strcpy_s?

#include <iostream>
#include <cstring>

const int BUFFER_SIZE = 50l;

int main()
{
    char first_name[BUFFER_SIZE], last_name[BUFFER_SIZE], concat[BUFFER_SIZE*2];

    std::cout << "Enter your first name: ";
    std::cin.getline(first_name, BUFFER_SIZE);
    first_name[BUFFER_SIZE - 1] = '\0';
    std::cout << "Enter your last name: ";
    std::cin.getline(last_name, BUFFER_SIZE);
    last_name[BUFFER_SIZE - 1] = '\0';

    strncpy(concat, last_name, BUFFER_SIZE);
    strcat(concat, ", ");
    strncat(concat, first_name, BUFFER_SIZE - strlen(", "));

    std::cout << "Here’s the information in a single string: " << concat << std::endl;

    return 0;
}

[Толстый Лорри]

evg_96,

вылетает вот такое вот окно. Это является ошибкой? Что это значит?

Вот это как раз и будет результат проверки. Видимо, вы слишком длинную строку передаете (больше Buffer_size)

[Толстый Лорри]

evg_96,

Я нашел еще вот такое вот решение на github. Насколько оно лучше моего решения,

В принципе, особой разницы нет. Только что в гитхабовом можно вводить пробелы, а в вашем - нет (будет считаться будто вы совершили два ввода, а не один).

что лучше использовать strncpy или strcpy_s?

При прочих равных *_s лучше, ибо они поднимают ошибку (которую вы видели), вместо того, чтобы молча поломать память, но есть пара нюансов:

• они появились в недавних стандартах (C++17 и C11), поэтому не везде могут оказаться применимы (в старых проектах, ибо здесь вам не веб, инструменты по 10-15 лет могут не меняться)
  
• они часто бывают нестандартным расширением конкретного компилятора. До выхода новых стандартов это была такая не переносимая фишка MS Visual Studio. Каким-либо другим компилятором вы такой код собрать не смогли бы.
  

То, что компилятор показывает warning - это просто его соображения на тему, что так будет лучше. Слушать его или нет - ваш выбор. Но озадачиться, почему он просит делать так, а не иначе - стоит обязательно.

Answer 1

[Армянское Радио]

Это какой-то C -- с -- классами. Используйте std::string для строк.

Comments

[Армянское Радио]

Ivan Sokolov, Ну, автору стоит вышвырнуть учебник или учителя за окошко.

[Толстый Лорри]

Армянское Радио, очевидно, что это учебное задание (где велосипеды - это отличный способ разобраться в устройстве сложного, а не пользоваться готовой "магией"), а не продакшн код. Если вы откроете исходники std::string, то там будет пресловутые char'ы. С поправкой на шаблон, конечно.

Подобные замечания стоило бы писать в комментарии к вопросу.

[Therapyx]

Толстый Лорри, это не отличный способ разобраться, он уже использует функции... Отличный способ был на операционных системах и микропроцессорах с ассемблером в универе хД Но это не начальный уровень, пусть пока делает так
string firstName, string lastName... stringToPrint = firstName+lastName
а когда разберется с основами, идет уже задаваться вопросами, мол почему это так? ибо имхо по самоучке это дикая потеря времени и возможно даже мотивации с желанием :D

[Толстый Лорри]

Therapyx, функции, А не головокружительные шаблонные классы. Дело в "имхо". Есть две методики обучения: "снизу вверх" и "сверху вниз". Второй подход подарит ложное ощущение владение предметом.
В универе или не в универе - это уже какая-то метафизика. Какая разница где это делать, если человек осилятор?

[Therapyx]

Толстый Лорри, дак а то, что ему говорят сейчас делать в книге - это не ложное получается?)

[Толстый Лорри]

Therapyx, человек набивает шишки в работе с сырой памятью, учится пользоваться нуль-терминированными строками. Это базовые вещи, которые останутся тайной при работе с std::string и подобными классами.

[evg_96]

Therapyx, это всего лишь примеры заданий. Следующее за этим заданием было тоже самое но с использованием string. К тому же тема была про массивы и указатели, string все равно внутри является массивом символов.

[evg_96]

Армянское Радио, книга Стивена Прата. Язык программирования C++. Лекции и упражнения. чем она вам не угодила?
Подобные задания есть в любой книге.