Что не правильно делаю с free();

[Winston123]

Доброго времени суток. Пишем на Microsoft Visual C++ 2008

Задача: Дано натуральное число n. Сформировать целочисленный массив IM(2n), элементами которого являются числа
1 2n 2 2n-1 3 2n-2 ... n n+1



Вот код:
#include "stdafx.h"
#include <stdio.h>
#include <iostream>
#include <stdlib.h>
#include <math.h>
#include <conio.h>

int main()
{
int tr;
printf ("vvedite znachenie n: ");
scanf ("%d", &tr);
int *A=(int *)malloc(tr*sizeof(int));

for (int i=0; i<tr; i++)
{
A[2*i]=i+1;
printf("%d ", A[2*i]);
A[2*i+1]=2*tr-i;
printf("%d ", A[2*i+1]);
}

free(A);
getch();

}


Вопрос по free(A);
Дело в том что при компилировании файла вылетает сообщение следующего рода:

ОС Windows инициировала точку останова в 8888.exe.
Это может быть вызвано повреждением кучи и указывает на ошибку в 8888.exe или в одной из загруженных им DLL.
Возможной причиной так же может быть нажатие пользователем клавиши F12, когда фокус принадлежит 8888.exe
Выведенное на экран окно содержит дополнительные данные для диагностики ошибки.

Неправильно задаю массив? Или синтаксис free неправильно юзаю???

[vvviperrr]

int *A=(int *)malloc(tr*sizeof(int));
тут надо писать так:
int *A=(int *)malloc(2*tr*sizeof(int));
исходя из вашего же условия задачи)

[Sazary]

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

Неправильно задаю массив? Или синтаксис free неправильно юзаю???

Нет. Все правильно. Программа работает (компилю в MinGW).
Это что-то студия творит...

Кстати, Math и iostream вам здесь ни к чему.

[Winston123]

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

int *A=(int *)malloc(tr*sizeof(int));
тут надо писать так:
int *A=(int *)malloc(2*tr*sizeof(int));
исходя из вашего же условия задачи)

Да, уже сам усмотрел что память неправильно выделяю...
Всем спасибо, вопрос решён..

[vvviperrr]

2Sazary она у тя не всегда будет работать) возвращаемся к теме обращения на недействительный указатель

[Sazary]

vvviperrr, ТС изменил листинг Там была строчка "razm=en*2;"
Ну а так, да, вы правы )

[Winston123]

Всё та же программа, код уже претерпел изменения в соответствии с требованиями преподавателя.



/* Контрольная работа №2
Дано натуральное число n. Сформировать целочисленный массив IM(2n), элементами которого являются числа
1 2n 2 2n-1 3 2n-2 ... n n+1


*/

#include "stdafx.h"
#include <stdio.h>
#include <iostream>
#include <stdlib.h>
#include <math.h>
#include <conio.h>

void title()
{
printf("Dano naturalnoe chislo n. Sformirovat celochislennyj massiv IM(2n)\n");
printf("1  2n  2  2n-1  3  2n-2  ...  n  n+1\n\n\n");
}

int input()
{
printf ("Vvedite znachenie n (ot 1 do 9999): ");

char str[4];
  for (int i=0;i<4;i++)
  {
        do
        {
            str[i]=getch();
            if (str[i]>=0x30 && str[i]<=0x39) printf("%c", str[i]);
        }
        while (str[i]!=0x0D && !(str[i]>=0x30 && str[i]<=0x39));
        if (str[i]==0x0D) break;
  }
return atoi(str);
}



void mas (int *Z, int y)
{
for (int i=0; i<y; i++)  
  {                          
  Z[2*i]=i+1;
  Z[2*i+1]=2*y-i;
  }
}

void output (int *R, int f)
{
if (f==0) printf("\nNevozmozhno postroit massiv: kolichestvo elementov massiva = 0");
else
{
int razm=2*f;
printf("\nKolichestvo elementov massiva = %d \n\n", razm);
printf("Massiv: \n");
for (int i=0; i<razm; i++)  
printf("%d ", R[i]);
}
printf("\n\n");
}


int main()

{
title();
char r;
do
  {
  int en = input();
  int *IM=(int *)malloc(2*en*sizeof(int));
  mas(IM, en);
  output(IM, en);
  free(IM);
  printf("PRESS <ENTER> TO CONTINUE \nPRESS <ANY KEY> TO EXIT \n");
  r=getch();
  printf("\n\n");
  }
while (r==0x0D);

}

Проблема в том, что при выводе в консольном окне, числа массива рубаются:

1111 1111 1111 1111 1111 1111 1111 111
1 1111 1111 1111 1111 1111 ...

Подскажете ли, как этого можно избежать????

[Sazary]

Измените вот так:

void output (int *R, int f)
{
if (f==0) printf("\nNevozmozhno postroit massiv: kolichestvo elementov massiva = 0");
else
{
int razm=2*f;
printf("\nKolichestvo elementov massiva = %d \n\n", razm);
printf("Massiv: \n");
for (int i=0; i<razm; i++)
 {
  printf("%d ", R[i]);
  if(i%20==0) printf("\n");
 }
}
printf("\n\n");
}

Через каждые 20 чисел будет переводиться строка.

[Winston123]

В принципе поначалу думал то же самое, но.. если мы вводим значение скажем 9999, то элементы массива будут пятизначными - соответственно 20 чисел будут занимать больше чем одну строку... Конечно можно поставить не 20, а меньше..
Опять же если мы введём кол-во элементов скажем 15, то перенос будет слишком ранним..

В общем привязываться хочется не к количеству чисел в строке.. Есть такая идея: сталкивать числа в строку и если длина превышает 80 символов, не добавлять последнего числа, а выводить на экран строку и собирать новую

C реализацией этого на языке - траблы(((

[Sazary]

Вот так сделайте:

void output (int *R, int f)
{
 char str[200],buf[100];
if (f==0) printf("\nNevozmozhno postroit massiv: kolichestvo elementov massiva = 0");
else
{
int razm=2*f;
printf("\nKolichestvo elementov massiva = %d \n\n", razm);
printf("Massiv: \n");
strcpy(str,"");
for (int i=0; i<razm; i++)
 {
  sprintf(buf,"%d",R[i]);
  if(strlen(str)+strlen(buf)>=79)
   {
    printf("%s\n",str);
    strcpy(str,buf);
   }
  else
   {
    strcat(str," ");
    strcat(str,buf);
   }
 }
 printf("%s\n",str);
}
printf("\n\n");
}