Что можете сказать про данный код?

Question

[Acaunt]

Сделал обёртку, для личного пользования, над функциями:

cout;
cin;

Вот код, с примерами использования:

Код

#include <iostream>
#include <vector>

void output(std::vector <std::string> list) {
    int size = list.size();
    for (int i = 0; i < size; i++) {
        std::cout << list[i] << std::endl;
    }
}

std::string input() {
    std::string str;
    getline(std::cin, str);
    return str;
}

std::string num(auto number) {
    return std::to_string(number);
}

static std::vector <std::string> list;

int main() {
    output({"Hello World!"});
    
    list = {"Good time of day"};
    output(list);
    
    list = {
        "", 
        "I", 
        "Wish", 
        "You", 
        "Happiness", 
        ""
    };
    output(list);
    
    int sum = 5 + 7;
    output({"five plus seven = " + num(sum) + " - that's a good number"});
    
    list = { input() };
    output(list);
    
    output({ input() });
    return 0;
}

Answer 1

[Dmitrii]

1. Если вам удобно такое, то здорово, применяйте, но как минимум в input я большого смысла не вижу, ибо это заменятся одной строкой и будет выглядеть более явно. Я бы предложил добавить в реализацию данных функций шаблоны, иначе для каждого типа данных, хранимых контейнером придется писать новую функцию. Можно даже тип контейнера в шаблон завернуть. Для обучения практика вам будет полезная.

2. Вы принимаете аргументы в функции по значению, что плохо в данном случае. Классика - принимать ко константной ссылке.

3. Имя лист для вектора неудачное, лучше array или strArray в вашем конкретном случае.

4. Функция NUM бесполезна, так как является обёрткой над понятной шаблонной функцией std::to_string. В вашем случае не ясно что делает функция при прочтении ее имени

5. Вы уверены, что код типа output({ input() }); вам необходим? Вы здесь выполняете слишком много лишних операций. Хотя по факту это должна быть просто одна строка std::cout

Comments

[Acaunt]

Спасибо за критику
1) input я понимаю, что бесполезная функция. Сделал её только потому что, чтобы всё в одной стилистике выглядело.
А зачем мне другие типы данных это же просто вывод строк в консоль?

2) не совсем понял

3) list, потому что я думал об этой переменной как страница текста

4)num не знал как обозвать, сделал её, чтобы сократить запись std::to_string()

5) написал просто как можно использовать для примера, да и к тому же зачем выводить в консоль текст который пользователь только что ввёл

[Dmitrii]

Acaunt,
1) Если вы внезапно захотите добавить вывод в консоль std::list/std::stack или что-нибудь ещё, то вам потребуется вторая функция, третья. Шаблоны же позволяют написать лишь 1 функцию (а контейнеры сделаны таким образом, что подобную функцию будет легко написать), почитайте про это на досуге.

2) void output(std::vector list);
void output(const std::vector & list);
Первый в вашем случае использовать нет необходимости, и даже вредно. Почитайте про ссылки и разницу при передачи аргументов по ссылке и по значению. Можно ещё и про указатели гляуть.

3) использовать название из двух-трех слов не грех (в коммерческой разработке иногда имена могут состоять из 5 и более слов, но там дело в сложности терминов/деталях). textList, page, bookPage - все эти названия ближе к странице текста, чем простой list и лично у меня вызвало ассоциацию с std::list, то есть это как назвать переменную int BOOL = 3; можно, но лучше подумать над названием ещё раз.

4) std::to_string() отличное название, оно кратко описывает что ожидать от этой функции. num же не говорит ни о чем. Он вернет число? Какое число? Опять же, я бы заменил num_to_string. Если так хочется написать свою функцию (но не вижу смысла), ибо если вы начнете писать что-то маломальски большое, то запутаетесь в названия num, str, a, b, i.