Логинов Сергей Lab#15
Conversation
|
Можно проверять |
|
К коду есть небольшое замечание. В сортировки слиянием выделять массивы на каждое объединение слишком затратно (выделять на слияние 2, 4, 8 и тд элементов). Вы не заметите этого при подсчете сравнений и перестановок, зато это сказывается на времени работы. Для эффективной реализации можно выделить второй массив, такой же длины, и использовать его части как вспомогательной памятью (его размер все равно будет соответствовать необходимому размеру на последний итерации, поэтому перерасхода памяти не будет). Наблюдаются поразительные сходства с кодом (вплоть до магических констант): #1 |
|
Понял,исправлю в кратчайшие сроки воскресенье, 19 декабря 2021г., 12:57 +03:00 от ValentinVolokitin ***@***.*** :
…К коду есть небольшое замечание. В сортировки слиянием выделять массивы на каждое объединение слишком затратно (выделять на слияние 2, 4, 8 и тд элементов). Вы не заметите этого при подсчете сравнений и перестановок, зато это сказывается на времени работы. Для эффективной реализации можно выделить второй массив, такой же длины, и использовать его части как вспомогательной памятью (его размер все равно будет соответствовать необходимому размеру на последний итерации, поэтому перерасхода памяти не будет).
Наблюдаются поразительные сходства с кодом (вплоть до магических констант): #1
—
Reply to this email directly, view it on GitHub , or unsubscribe .
Triage notifications on the go with GitHub Mobile for iOS or Android .
You are receiving this because you authored the thread. Message ID: @ github . com>
|
|
Merge вроде подправил, можно проверять . |
|
Код принят, но в следующий раз схожие коды принимать не буду (переименование переменных оставляет код схожим) |
|
Отчет можно проверять |
|
Лабораторная и отчет сданы в среду 30 марта. Можно проверять. Название LabWork |
|
Отчет по сортировкам принят |
| #include <stdlib.h> | ||
| #include <math.h> | ||
|
|
||
| #define SummType SimpleSumm(array, n); |
There was a problem hiding this comment.
При использование такого define, аргументы функции надо делать параметрами. Так как в коде может изменится имена массива и размера
|
|
||
| switch (choice) | ||
| { | ||
| case(1): |
There was a problem hiding this comment.
Чтобы избежать "магических" констант (1,2,3,4) можно завести enum
|
Добавил enum, но не понял, что нужно сделать с define. |
|
Изменил define |
|
Код по рядам принят (+) |
|
Код Метода Гаусса сделан, можно проверять. |
|
Отчет сделан, можно проверять |
ValentinV95
left a comment
ValentinV95
left a comment
There was a problem hiding this comment.
Код метода Гаусса принят (+-).
Из-занеправильной проверки, при вводе реально несовместной системы, выдает просто неверный ответ
| { | ||
| this->size = other.size; | ||
|
|
||
| delete[] this->arr; |
There was a problem hiding this comment.
При таком коде, если сделать:
MyVector a;
...
a=a;
то будет удалена единственная память
2 participants
No description provided.