and-se / 1kurs-vesna-2018

0 stars 2 forks source link

Task 2 - Zolotuhin #4

Closed and-se closed 6 years ago

and-se commented 6 years ago

Отлично!

Единственное - раз Вы уже умеете делать модули, так почему бы описанные в начале main.cpp функции:

void help(); void write (IntArray array); void move (IntArray array , int direct); void sort (IntArray array, int direct); void put (IntArray array, int index, int value); void reverse (IntArray array);

and-se commented 6 years ago

Ещё замечание - не нужно в структуре IntArray хранить длину массива в динамической переменной - это лишняя опасность забыть освободить память. Раз Вы точно знаете размер данных (а это int), так заведите "обычное" поле int size.

MrFang commented 6 years ago

@and-se Все функции объявленные внутри main -- наследие прошлого семестра. По факту это просто программа, которую я писал в том полугодии только приспособленная под другой массив

MrFang commented 6 years ago

@and-se put можно исправить. Индекс от 1 для удобства пользователя, но можно i-1 делать в тексте main

and-se commented 6 years ago

Можете не исправлять уже, это на будущее.

P.S. Словарь Даля вечером посмотрю, с ходу не завёлся.

MrFang commented 6 years ago

@and-se Исправил тип данных в структуре