https://pastebin.com/nXkpm1dC
1) выкинуть srand() и использовать <random> 2) забыть о new и пользоваться умными указателями и контейнерами для начала
Не раскидывать код, который нигде не повторяется, по функциям.
Всячески Не использовать new Не использовать rand
Прекрасный совет, просто гениальный!
+ У меня тогда некоторые функции будет строк на 1000
Ну тогда нормальный класс писать для матрицы, а не вот это вот всё.
Там тоже все в одной функции все сделать?
<chrono> не проходил, а каким образом о new забыть? можно по другому память выделить?
А в чём цель этого кода?
Что значит "<chrono> не проходил"?
прошу прощения под chrono имелся в виду random
std::unique_ptr std::vector
ну, не дошел я еще до этой библиотеки
В чём смысл этого кода? Для чего ты это писал?
там еще кусок кода для работы с этой матрицей, задание в универе такое. само задание готово, мне интересно, как код можно улучшить. сейчас сижу читаю как обойтись без new и delete
Текст задания можно?
Да не надо без NEW/DELETE пока.
Создать матрицу N*N и вывести по спирали, начиная с одного из центральных элементов, все элементы
Я должен тебя немного огорчить, что это — не совсем матрица. Тебе нужно уточнить, что в задании имелось в виду под понятием "матрица".
квадратная матрица
Ну вот это не матрица, это - массив массивов.
там не написано условий для реализации, а я пока только так могу, через двумерный массив (ну и поскольку ввод размера с клавы - динамический). я прошел на данный момент процедурную часть с++, только вот добрался до классов, ознакомился с базовыми понятиями, конструкторами и тд, но тут нужно реализовать именно отдельно функциями (проверка знаний, а не просто выношу код от балды), дальше уже будет через классы и тд.
Обсуждают сегодня