3

Я работаю с большим двумерным массивом.

В случае, когда он размером 30 x 200, все считается. При больших объемах программа вылетает... Мне посоветовали использовать memset(), но что-то не особо помогает... Может быть у вас есть какие идеи?

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string.h>

double **B;

int main (int argc, char *argv[])
{ 
    int N, n_A;

    N = 32;
    n_A = 350; /* если сделать n_A = 200, то все работает */
    B = (double **)malloc(N * sizeof(double *));

    for(i = 0; i < N; i++)
    {
        B[i] = (double *)malloc(n_A * sizeof(double));
        memset(B[i], 0, n_A * sizeof(double));
    }

    free(B);

    return 0;
}
3
  • Ну не знаю, скопировал, объявил i, скомпилял, работает... <br />Чем собираешь, когда сыпется что говорит?<br />Изменил n_A на 3500 тоже работает... 22 фев 2011 в 21:19
  • У меня тоже работает. Нужно поменять n_A на 1000000000, и оно упадет :) 22 фев 2011 в 21:20
  • Я могу только предположить. Попробуйте написать под x64) 3 мар 2011 в 22:13

3 ответа 3

4

В целом код правильный, но нужно добавить проверку при выделении памяти: это необходимо, потому что памяти может просто не хватать, и в этом случае программа будет падать, потому что будет происходить запись в несуществующую память. То есть всякий раз, когда происходит вызов malloc, необходимо проверить, что возвращаемое значение не равно NULL.

B = (double **)malloc(N * sizeof(double *));

/* Проверить, что память выделена */
if (B != NULL)
{
    for(i = 0; i < N; i++)
    {
        B[i] = (double *)malloc(n_A * sizeof(double));

        /* Проверить, что память выделена */
        if (B[i] != NULL)
        {
            memset(B[i], 0, n_A * sizeof(double));
            free(B[i]);
        }
    }

    free(B);
}

Кроме того, нужно не забывать освобождать память, выделяемую malloc внутри цикла, иначе будут утечки.

4
  • Да, все верно, в коде из первого поста не освобождается вся память, при чем ее основная часть утекает =) 22 фев 2011 в 21:27
  • Спасибо большое, все работает =))
    – Alcheringa
    22 фев 2011 в 21:58
  • Используйте electric fence?
    – alexlz
    23 фев 2011 в 5:52
  • Ммм,попробую. Хотя его стоит применять для небольших программ.
    – Alcheringa
    23 фев 2011 в 16:26
1

Вот это все можно сделать одной строкой

double * B;
B = (double *) calloc (N * n_A, sizeof(double));
free (B);

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

Доступ к данным осуществляется по формуле

double a;
a = *(B + i * n_A + j);

где i - номер строки, j - номер столбца

0

При больших объемах программа вылетает...

Локальный буфер(точнее размер стекового фрейма) не должен иметь размер больший одной страницы. Иначе произойдёт обращение за пределы сторожевой страницы стека, расширен он не будет и возникнет исключение.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.