0

Почему-то НЕ выполняется malloc и как следствие

(*value_comblock)[1] = 0;

не выполняется

Весь код

#pragma hdrstop
#pragma argsused

#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <windows.h> //FOR WINDOWS ONLY

#ifdef _WIN32
#include <tchar.h>
#else
typedef char _TCHAR;
#define _tmain main
#endif



// Структура двогичного (бинарного) дерева

struct btree {
    double data; // данные
    struct btree *left; // левая ветвь
    struct btree *right; // правая ветвь
    struct btree *parent; // родитель
};

bool read_arg(char**, signed long long int**); // сигн. ф. считывания команд

int _tmain(int argc, _TCHAR* argv[]) {

    /* ~~~~~~~~~~~~~~~~~~~~~~ ПЕРЕМЕННЫЕ ~~~~~~~~~~~~~~~~~~~~~~~~~ */

    // Динамические массивы
    char *command_comblock; // Массив комманд
    signed long long int *value_comblock; // Массив значений

    /* ~~~~~~~~~~~~~~~~~~~~~ КОММАНДЫ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ */

    // Включаем русский в консоли
    SetConsoleOutputCP(1251); // FOR WINDOWS ONLY

    // Ввод и анализ на верность комманд
    if (read_arg(&command_comblock, &value_comblock) != true) {
        // Если что-то пошло не так при вводе в первый раз (попробуем еще разок)
        if (read_arg(&command_comblock, &value_comblock) != true) {
            // Что-то явно не так со вводом (тест на похмелье :D)

            // ScreenCleaner
            system("cls"); // FOR WINDOWS ONLY
            // system("clear"); //FOR LINUX ONLY

            // Выдача ошибки
            return 315;
        }
    }

    // Разбор введеных аргументов
    system("pause");
    return 0;
}

bool read_arg(char** command_comblock, signed long long int** value_comblock) {
    // ПЕРЕМЕННЫЕ
    char temp;

    // Первоначальное выделение места массивам
    if ((*command_comblock = (char *)malloc(sizeof(char))) == NULL) {
        printf("Realloc Error! [command_comblock]");
        return false;
    }

    if ((*value_comblock = (signed long long int *)malloc((sizeof(signed long long int)))) == NULL) {
        printf("Realloc Error! [value_comblock]");
        return false;
    }

    // Непосредственное считывание
    printf("Вводите комманды, как в k806.ru [Для Win. Ctrl+Z = EOF]\n");

    printf("%d", sizeof(*value_comblock));

    (*value_comblock)[0] = 0; // Количество операций
    (*value_comblock)[1] = 0; // Количество значений

    while (temp != EOF) {
        temp = getchar();
        switch (temp) {

            // Операция вывода дерева
        case 'p':
            (*value_comblock)[0] = (*value_comblock)[0] + 1; // Кол-во операций

            if ((*command_comblock = (char*)realloc(*command_comblock,
            (((*value_comblock)[0])*sizeof(char)))) == NULL) {
                printf("Realloc Error! [command_comblock]");
                return false;
            }

            (*command_comblock)[(*value_comblock)[0] - 1] = temp;
        break;

        //Операция "вставка в дерево"
    case '+':
        (*value_comblock)[0] = (*value_comblock)[0] + 1; // Кол-во операций
        (*value_comblock)[1] = (*value_comblock)[1] + 1; // Кол-во значений

        if ((*command_comblock = (char*)realloc(*command_comblock,
            (((*value_comblock)[0])*sizeof(char)))) == NULL) {
            printf("Realloc Error! [command_comblock]");
            return false;
        }

        if ((*value_comblock = (char*)realloc(*value_comblock,
            (((*value_comblock)[0]+2)*sizeof(char)))) == NULL) {
            printf("Realloc Error! [command_comblock]");
            return false;
        }

        (*command_comblock)[(*value_comblock)[0] - 1] = temp;
        scanf("%d", &(*value_comblock)[(*value_comblock)[1] + 1]);

        break;
    }

}

return true;
}
14
  • 1
    (*value_comblock)[1] = 0; - при копировании не заметил
    – SlimRG
    30 апр 2018 в 11:03
  • Смотрите комментарий //Операция "вставка в дерево"
    – SlimRG
    30 апр 2018 в 11:03
  • (2*sizeof(signed long long int) - НЕ ПОМОГАЕТ!!!
    – SlimRG
    30 апр 2018 в 11:21
  • Исправил (((*value_comblock)[0]+2)*sizeof(char)))) на (((*value_comblock)[0]+2)*sizeof(signed long long int))))
    – SlimRG
    30 апр 2018 в 11:38
  • 1
    Да, это была подгоревшая планка RAM - сорри народ
    – SlimRG
    12 мая 2018 в 13:44

2 ответа 2

1

Что попросили -

*value_comblock = (signed long long int *)malloc((sizeof(signed long long int)))

то и дал - sizeof(signed long long int) байт, на один элемент...

0

Вот здесь выделяется память под один элемент:

if ((*value_comblock = (signed long long int *)malloc((sizeof(signed long long int)))) == NULL) {
        printf("Realloc Error! [value_comblock]");
        return false;
    }

А ниже по коду попытка обращения ко второму элементу:

(*value_comblock)[1] = 0; 

поэтому и ошибка.

20
  • (2*sizeof(signed long long int)) - НЕ ПОМОГАЕТ!!! (Только что попробовал - все та же фигня)
    – SlimRG
    30 апр 2018 в 11:22
  • @SlimRG У вас ещё ошибка во втором realloc - вместо sizeof(char) должен быть sizeof(signed long long int).
    – zed
    30 апр 2018 в 11:26
  • Спасибо, но все равно не присваиваются кроме нулевой в другие ячейки массива
    – SlimRG
    30 апр 2018 в 11:36
  • @SlimRG Ещё ошибка в scanf - вместо %d надо писать %I64d - у вас же массив 64-х битных чисел.
    – zed
    30 апр 2018 в 11:58
  • 1
    В realloc для *value_comblock умножение делается на sizeof(char)... Вот это вот манера делать явное приведение типа на результат функции аллокации и манера использовать имена типов под sizeof - всегда красный флаг, предупреждающий, что код будет набит такими ошибками. 5 июн 2018 в 21:22

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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