Здравствуйте, если ли желающие мидлы\сеньоры заревьювить мою курсовую работу C++

на 500 строк? Кину в лс ссылку на гист.

@webreh , не желаешь?

в личку - нет, пусть на общественное осуждение обсуждение выносит)

мне свой код лень ревьюить, не то, что чужой. Работает - и срать, коммичусь

оно меня не тегнуло

https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093

ООП использовать нам нельзя

вот тут точно переделывать https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L91 вот это - конструктор https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L354 вот это - деструктор https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L231 вот это - что-то очень странное https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L349 вот тут хочется RAII https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L162

А тема работы? Что сделать надо было ?

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

описание какое-то надо , формальное. Функция какая... требования к вычислениям...

а что именно с первым переделывать?

@AnderX003 у вас, кажется, некоторое (java/C# перенесенное?) непонимание, что нет смысла писать settings::calculation_settings* settings и делать указатели на неполиморфные объекты, когда можно оперировать самими объектами; примерно набросал, после небольшой реорганизации приходите снова

conio.h - упразднить system()- упразднить

Может и ничего, раз ООП нельзя. Вообще, на курсовик это не тянет, лабораторка лишь...

это не так важно

+

могу только файлом кинуть, но тут блокировка

1) input_with_restricts имеет единственный выходной параметр Стоит поменять просто на возвращаемое значение 2) В разных частях то сишный I/O используется, то - плюсовый. Надо определиться

А так конечно неплохо, хоть старался...

https://asciinema.org/a/7eEJGp3CQSmAHH4R4Ghtz6r6U @webreh

это ужасно, опять линуха не поддерживают

ну я вынужден был сделать псевдо деструктор так как нельзя ооп

эм что значит "нельзя ООП"

А что есть ООП? В классе с конструктором и деструктором ООП как такового то и нет

це андроид

это ужасно, опять ведро не поддерживают

ну чисто структуры с полями мы учили, поэтому отталкиваюсь от этого

меня в любом случае смущает этот фрагмент https://gist.github.com/AnderX003/9df45420d05dd791bee040f240b8c093#file-kursova_robota-cpp-L354 тем, что это 5 согласованных массивов одной длины, там нужна какая-то серьезная мотивация, что это не один массив структур

типа что это мог бы быть один двумерный?

там вообще происходит что-то идеологически странное если мы следуем С++, то почему аллокация внешняя для calculate_for_array если мы следуем С, то почему в методах типа read_calculation_parameters не стоят goto

Не, скорее массив из структур, в каждой из которых лежат x, y, y_series, error и iterations_amount Не 5 разных массивов, а один

а с какого перепугу вдруг goto стали приветствовать в Си? вас кто так испугал?

потому что goto cleanup в С невозможно обойти и нормально

все разные.

Соответсвующие значения в них логически связаны Почему тогда это не структура?

ну оно лежит в структуре

всё-таки позиция писать на С в С++ коде принципиальная или нет? мы по-разному дальше будем весь код рассматривать

знакомтьтесь __attribute__((cleanup(free_func))) https://habr.com/ru/post/503536/

Да там не то чтобы от плюсов то много осталось. С с неймспейсами)

это точно ISO compatible?

ну у нас как бы плюсы, но пока еще без ооп

так не бывает, понимаете, если у вас нет RAII, то и организации кода как в С++ не может быть в принципе

__attribute__(cleanup) is not gcc-specific, it is also supported by clang and icc, making msvc the only major compiler that doesn't support it (and that one is pretty useless for modern C development anyway).

ну вроде понял

Кажется raii и ООП вещи ортогональные

По мнению некоторых преподавателей ООП == наличие чего-то, кроме данных, в классах/структурах Ну и RAII под бой попадает(

Господа, исправил то, о чем говорили вчера. Вот результат: https://gist.github.com/AnderX003/47876b7ad75b145d6bc5c7755aa0689f

https://gist.github.com/AnderX003/47876b7ad75b145d6bc5c7755aa0689f#file-kursova_robota_v2-cpp-L348 Попробуй без цикла.

поделить? щас попробую

сделал, спасибо. как оказалось там даже баг был)