Loading...
DimovIvan avatar DimovIvan 16 Точки

Compare Matrices - греша ли във функцията за освобождаване на памет на тримерен масив?

Write a program that reads two integer matrices (2D arrays) from the console and compares them element by element. 
You are not allowed to use STL.

For better code reusability, you could do the comparison in a function, which returns true if they are equal and false if not.
Each matrix definition on the console will contain a line with a positive integer number R – the number of rows in the matrix – followed by R lines containing the numbers in the matrix, separated by spaces (each line will have an equal amount of numbers.
The matrices will have at most 10 rows and at most 10 columns.
Print equal if the matrices match, and not equal if they do not match.

Решавам задачата чрез динамично заделяне на тримерен масив, но явно не освобождавам паметта правилно. Ако не използвам функцията за освобождаване на динамично заделен тримерен масив взимам 100 точки, но ако я използвам ми дава time limit на всички тестове. Кажете ми къде греша във функцията.

https://pastebin.com/2XKT7GAk

Тагове:
0
C++ Advanced
MartinBG avatar MartinBG 4803 Точки

Има memory leak в кода, но същинският проблем е, че решението е доста усложнено и най-вероятно затова му се "губи краят". :)

По условие е казано, че матрицата ще е с максимален размер 10 * 10. Използвайте тази информация, вместо да се опитвате да направите решението "универсално", което не е практически възможно при формата на входните данни (липсва броя на колоните, имаме само редовете).

Препоръчвам Ви да пренапишете кода изцяло - най-добре само в main() и едва след като се уверите, че всичко работи както го искате, да го разбиете на методи/функции.

 

1
20/04/2022 20:43:16
DimovIvan avatar DimovIvan 16 Точки

Имам memory leak защото съм закоментирал функцията за освобождаването на паметта на 3D масива. Не съм търсил универсално решение, а просто исках да се упражня в динамично заделяне и освобождаване на 3D масив. Затова входа го чета като 3D масив съставен от два 2D масива. Заделянето ми е вярно защото програмата работи и вземам 100 точки, но когато разкоментирам функцията за освобождаване на паметта и ми гръмва с time limit. Явно не я освобиждавам правилно и исках да разбера къде греша. Иначе си направих универсално решение като се съобразих с максималния размер 10×10 и даже решението ми работи при различни дължини на отделните редове на 2D масивите:   https://pastebin.com/afYV0icG

0
emmausa avatar emmausa -5 Точки

good

-2
12/04/2023 11:00:57
tt104 avatar tt104 15 Точки

You're releasing rowsCols, then dereferencing via it - surprise.

Swap:

    releaseMemoryIntArr2D(rowsCols, numbDimensions);
    //releaseMemoryIntArr3D(arr3D, size3D, rowsCols);

to:

    releaseMemoryIntArr3D(arr3D, size3D, rowsCols);
    releaseMemoryIntArr2D(rowsCols, numbDimensions);

Uncomment and comment:

        releaseMemoryIntArr2D(arr3D[i], rowsCols[0][i]);
        /*for (int j = 0; j < rowsCols[0][i]; j++)
        {
            delete[] arr3D[i][j];
            arr3D[i][j] = nullptr;
        }
        delete[] arr3D[i];
        arr3D[i] = nullptr;*/

Profit :)

Tested with:

2
2 2
2 2
2
2 2
2 2

only.

Testers:

clang++ -fsanitize=undefined,leak,address -std=c++11 -O0 -g foo.cpp -o foo
valgrind --leak-check=full ./foo

 

2
tomusa avatar tomusa -3 Точки

The information you share is excellent and interesting; thanks to that, I know more valuable things. Keep posting interesting things krunker and I will keep an eye on your posts. 

-2
n00be88 avatar n00be88 -1 Точки

It is amazing to be here with everyone, and I appreciate you sharing your expertise with me. The knowledge and information you have provided here are really lolbeans helpful to me.

 

-2
jerrym avatar jerrym 0 Точки
Thank you for sharing this useful fnf information with everyone. It's very useful for everyone.
-2
Mathew23 avatar Mathew23 -1 Точки

Any coder can find it difficult to compare two matrices. You must properly interpret the numerical relationships between letter boxed two different sets of data in order to complete the process. 

-2
emmausa avatar emmausa -5 Точки

I am very glad to read your post. The knowledge you shared helped me a lot to supplement the professional knowledge I lacked. Pokemon Showdown

-2
Можем ли да използваме бисквитки?
Ние използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Можете да се съгласите с всички или част от тях.
Назад
Функционални
Използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Използваме „сесийни“ бисквитки, за да Ви идентифицираме временно. Те се пазят само по време на активната употреба на услугите ни. След излизане от приложението, затваряне на браузъра или мобилното устройство, данните се трият. Използваме бисквитки, за да предоставим опцията „Запомни Ме“, която Ви позволява да използвате нашите услуги без да предоставяте потребителско име и парола. Допълнително е възможно да използваме бисквитки за да съхраняваме различни малки настройки, като избор на езика, позиции на менюта и персонализирано съдържание. Използваме бисквитки и за измерване на маркетинговите ни усилия.
Рекламни
Използваме бисквитки, за да измерваме маркетинг ефективността ни, броене на посещения, както и за проследяването дали дадено електронно писмо е било отворено.