Ошибка с выделением памяти

@Акелла · Регистрация: 10.05.2009

Author24 — интернет-сервис помощи студентам

выдает ошибку в выделении памяти. никто не подскажет почему?
если что -задача звучит так примерно
вводим веественные числа пока не попадетсья ноль занося их в динамический массив, как тока ноль - посчитываем сумму путем - перемножаем сначала первый и последний элементы, к ним прибавляем произведение второго и предпоследнего и так пока не обойдем массив (т.е. последними прибавяться вновь произведения первого и последнгео)

C++

#include <conio.h>
#include <STDLIB.H>
#include <ALLOC.H>
#include <stdio.h>
 
void proc_sum();
int main()
{
    clrscr();
    proc_sum();
    getch();
    return 0;
}
void proc_sum()
{
    float *mas;
    float temp, rez=0;
    int i=0, len,j;
    mas = (float *)calloc(i, sizeof(float));
    if(!mas)
    {
        printf("\n\t Memmory Error!");
        exit(1);
    }
    while(temp!=0)
    {
        printf("\n\t Enter nomder: ");
        scanf("%f",&temp);
 
        if (temp==0) break;
        i++;
        mas = (float *)realloc(mas,i);
        mas[i-1]=temp;
 
    }
    len=j=i;
    for(i=0;i<=len;i++)
    {
        rez=rez+mas[i]*mas[j];
        j--;
    }
 
    free(mas);
    printf("\n\t Summ= %f",rez);
    getch();
}

@FunDuck · 09.06.2009, 13:15

C++

1	mas = (float )realloc(mas,isizeof(float));

@Акелла · 09.06.2009, 13:17 **[ТС]**

тут ошибку спалил... теперь код выглядит так:

C++

#include <conio.h>
#include <STDLIB.H>
#include <ALLOC.H>
#include <stdio.h>
 
void proc_sum();
int main()
{
    clrscr();
    proc_sum();
    getch();
    return 0;
}
void proc_sum()
{
    float *mas;
    float temp, rez=0;
    int i=0, len,j;
    mas = (float *)calloc(1, sizeof(float));
    if(!mas)
    {
        printf("\n\t Memmory Error!");
        exit(1);
    }
    while(temp!=0)
    {
        printf("Enter nomder: ");
        scanf("%f",&temp);
 
        if (temp==0) break;
        else mas[i]=temp;
        i++;
        mas = (float *)realloc(mas,(i+1));
 
 
    }
    len=j=i;
    for(i=0;i<=len;i++)
    {
        rez=rez+mas[i]*mas[j];
        j--;
    }
 
    free(mas);
    printf("\n\t Summ= %f",rez);
    getch();
}

но теперь сумму он не считает=(

@FunDuck · 09.06.2009, 13:25

C++

1	len=j=i - 1;

@Акелла · 09.06.2009, 13:30 **[ТС]**

не катит=( выдает ахинею в виде суммы.

@FunDuck · 09.06.2009, 13:32

C++

1	mas = (float *)realloc(mas,(i+1));

эта строка так и осталась?

@Акелла · 09.06.2009, 13:38 **[ТС]**

даа

@FunDuck · 09.06.2009, 13:43

Меняй на эту!!:

C++

1	mas = (float )realloc(mas,isizeof(float));

Иначе в сумму числа за пределами массива попадать будут.

Evg · 09.06.2009, 13:45

Сообщение от Акелла

даа

Та же ошибка, что и в посте #2

Добавлено через 40 секунд

Сообщение от FunDuck

Меняй на эту!!:

C++

1	mas = (float )realloc(mas,isizeof(float));

Иначе в сумму числа за пределами массива попадать будут.

По его логике вместо "i" должно быть "i+1"

@Акелла · 09.06.2009, 13:51 **[ТС]**

FunDuck, изменил как ты показал, теперь в качестве суммы выдаеться последний элемент.
может у меня в другом ошибка?

@FunDuck · 09.06.2009, 14:03

Странно... вроде правильно работает...

C++

#include <conio.h>
#include <STDLIB.H>
#include <ALLOC.H>
#include <stdio.h>
 
void proc_sum();
int main()
{
        //clrscr();
        proc_sum();
        getch();
        return 0;
}
void proc_sum()
{
        float *mas;
        float temp, rez=0;
        int i=0, len,j;
        mas = (float *)calloc(1, sizeof(float));
        if(!mas)
        {
                printf("\n\t Memmory Error!");
                exit(1);
        }
        while(temp!=0)
        {
                printf("Enter nomder: ");
                scanf("%f",&temp);
 
                if (temp==0) break;
                else mas[i]=temp;
                i++;
                mas = (float *)realloc(mas,i*sizeof(float));
 
 
        }
        len=j=i;
        for(i=0;i<len;i++)
        {
                rez=rez+mas[i]*mas[j];
                j--;
        }
 
        free(mas);
        printf("\n\t Summ= %f",rez);
        getch();
}

Насколько я понял прога должна выводить сумму элементов умноженную на последний...?

@Акелла · 09.06.2009, 14:10 **[ТС]**

нет, она должна выводить сумму:
первый*последний + второй* предпоследний +третий* предпредпоследний...последний*первый

Evg · 09.06.2009, 14:13

Перед 41-й строкой воткни печать i и j - у тебя там явная лажа. Да и вообще. Хоть немного напряги мозги

@Акелла · 09.06.2009, 14:17 **[ТС]**

Evg,
кхм... используя борланд, я пробегаю свою прогу функцие watch встроенной в компилятор - когда ввожу 4-ый по счету элемент, то значения предыдущих 3 (которые вполне нормально записались) куда то улетают, получаю в этих ячейках памяти что то левое. либо сдвигаются.

а на счет мозгов - раз написал, значит етсь -я же не прошу мне ее написать с нуля, а показать более опытных где у меня ошибка. если не можешь -нечего МНЕ про мозги говорить

Evg · 09.06.2009, 14:29

Сообщение от Акелла

а на счет мозгов - раз написал, значит етсь -я же не прошу мне ее написать с нуля, а показать более опытных где у меня ошибка. если не можешь -нечего МНЕ про мозги говорить

Не хотел тебя обидеть. Просто по этой теме и соседней ты очень похож на кучу товарищей, которые откуда-то передерут программу, она не работает, а дальше их надо водить за руку и диктовать изменение каждого байта. Если не так - приношу извинения

Тогда по существу - внутри цикла ввода воткни печать всех уже введённых элементов. Т.е. при вводе вторго числа печтатешь первое. При вводе третьего печатаешь первое и второе и т.п.

@FunDuck · 09.06.2009, 14:29

Так делай...

C++

        while(temp!=0)
        {
                printf("Enter nomder: ");
                scanf("%f",&temp);
 
                if (temp == 0) break;
                else {
                    mas = (float *)realloc(mas,(i+1)*sizeof(float));
                    mas[i]=temp;
                }
                i++;
        }
        len=j=i - 1;

@Акелла · 09.06.2009, 14:41 **[ТС]**

FunDuck,
сделал.
сумму выдает ноль. в чем причина... ну вот тут действительно омзгов не хватает

Evg · 09.06.2009, 14:51

Перед тем, как считать свой результат, напиши ещё один цикл, который распечатает значение всех элементов массива. Опять-таки чтобы понять, косячит ввод или подсчёт суммы

@Акелла · 09.06.2009, 14:54 **[ТС]**

косячит ввод это омжно простым пошаговым просмотром увидеть. в любом компиляторе - я написал же:

Сообщение от Акелла

используя борланд, я пробегаю свою прогу функцие watch встроенной в компилятор - когда ввожу 4-ый по счету элемент, то значения предыдущих 3 (которые вполне нормально записались) куда то улетают, получаю в этих ячейках памяти что то левое. либо сдвигаются.

воот.. теперь стали вылезать только левые значения каждый раз после

Сообщение от FunDuck

if (temp == 0) break;
* * * * * * * * * * * * * * * * else {
* * * * * * * * * * * * * * * * * * * * mas = (float *)realloc(mas,(i+1)*sizeof(float));
* * * * * * * * * * * * * * * * * * * * mas[i]=temp;
* * * * * * * * * * * * * * * * }
* * * * * * * * * * * * * * * * i++;

Evg · 09.06.2009, 15:14

А покажи ещё раз весь исходник. А то уже сложнопонять, что ты там наисправлял по отношению к исходному

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 14:10 [ТС]	12
	нет, она должна выводить сумму: первыйпоследний + второй предпоследний +третий* предпредпоследний...последний*первый 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	09.06.2009, 14:51	18
	Перед тем, как считать свой результат, напиши ещё один цикл, который распечатает значение всех элементов массива. Опять-таки чтобы понять, косячит ввод или подсчёт суммы 1

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 14:54 [ТС]	19
	косячит ввод это омжно простым пошаговым просмотром увидеть. в любом компиляторе - я написал же: Сообщение от Акелла используя борланд, я пробегаю свою прогу функцие watch встроенной в компилятор - когда ввожу 4-ый по счету элемент, то значения предыдущих 3 (которые вполне нормально записались) куда то улетают, получаю в этих ячейках памяти что то левое. либо сдвигаются. воот.. теперь стали вылезать только левые значения каждый раз после Сообщение от FunDuck if (temp == 0) break; * * * * * * * * * * * * * * * * else { * * * * * * * * * * * * * * * * * * * * mas = (float )realloc(mas,(i+1)sizeof(float)); * * * * * * * * * * * * * * * * * * * * mas[i]=temp; * * * * * * * * * * * * * * * * } * * * * * * * * * * * * * * * * i++; 0

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 13:30 [ТС]	5
	не катит=( выдает ахинею в виде суммы. 0

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 13:38 [ТС]	7
	даа 0

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 13:51 [ТС]	10
	FunDuck, изменил как ты показал, теперь в качестве суммы выдаеться последний элемент. может у меня в другом ошибка? 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	09.06.2009, 14:13	13
	Перед 41-й строкой воткни печать i и j - у тебя там явная лажа. Да и вообще. Хоть немного напряги мозги 1

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 14:17 [ТС]	14
	Evg, кхм... используя борланд, я пробегаю свою прогу функцие watch встроенной в компилятор - когда ввожу 4-ый по счету элемент, то значения предыдущих 3 (которые вполне нормально записались) куда то улетают, получаю в этих ячейках памяти что то левое. либо сдвигаются. а на счет мозгов - раз написал, значит етсь -я же не прошу мне ее написать с нуля, а показать более опытных где у меня ошибка. если не можешь -нечего МНЕ про мозги говорить 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	09.06.2009, 14:29	15
	Сообщение от Акелла а на счет мозгов - раз написал, значит етсь -я же не прошу мне ее написать с нуля, а показать более опытных где у меня ошибка. если не можешь -нечего МНЕ про мозги говорить Не хотел тебя обидеть. Просто по этой теме и соседней ты очень похож на кучу товарищей, которые откуда-то передерут программу, она не работает, а дальше их надо водить за руку и диктовать изменение каждого байта. Если не так - приношу извинения Тогда по существу - внутри цикла ввода воткни печать всех уже введённых элементов. Т.е. при вводе вторго числа печтатешь первое. При вводе третьего печатаешь первое и второе и т.п. 1

@Акелла Сонный металюга 46 / 46 / 13 Регистрация: 10.05.2009 Сообщений: 295
	09.06.2009, 14:41 [ТС]	17
	FunDuck, сделал. сумму выдает ноль. в чем причина... ну вот тут действительно омзгов не хватает 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	09.06.2009, 15:14	20
	А покажи ещё раз весь исходник. А то уже сложнопонять, что ты там наисправлял по отношению к исходному 1