Программа на СИ с массивами

[abstractionhawk]

Здравствуйте! Следующую задачку задали.
Пусть x[i],y[i] (i=1,2,...) определены как
x[i]=((y[i-1]-y[i-2])/i)
y[i]=((x[i-1]*x[i-1]+x[i-2]+y[i-1])/i)
Получить x[1],...,x[n]; y[1],...,y[n];

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


void main()
{ int p;
  printf("Vvedite dlinu massiva X i dlinu massiva Y \n"); scanf("%i",&p);

    int i;
    int x[p],y[p];
    x[0]=1;y[0]=1;
    x[1]=2;y[1]=2;


    for(i=2;i<p;i++)
     x[i]=((y[i-1]-y[i-2])/i);
     y[i]=((x[i-1]*x[i-1]+x[i-2]+y[i-1])/i);

    printf("Massiv X:\n");
for(i=0;i<p;i++)
     printf("%i ",x[i]);

    printf("\n Massiv Y:\n");
for(i=0;i<p;i++)
     printf("%i ",y[i]);

getch();
}

Вот что я пока что своими силами сделал. Но результат не очень похож на правду. Посмотрите пожалуйста код кто может, правельно ли все сделано? Заранее благодарю (Сильно не кидайте камнями в меня, СИ никогда не учил, за что сейчас расплачиваюсь)

[Abstraction]

Компилятор должен был капитально выругаться.
Всё почти правильно, но
1) Тело цикла состоит из одной инструкции либо блока, ограниченного {}:

    for(i=2;i<p;i++) {
     x[i]=((y[i-1]-y[i-2])/i);
     y[i]=((x[i-1]*x[i-1]+x[i-2]+y[i-1])/i);
    }

2) Неясно, должны члены последовательности быть целыми числами или вещественными. Если второе, то надо поменять типы x и y, а также спецификатор вывода printf(). Пока будем считать, что всё правильно.

3) (главное) Недопустимо объявлять переменные, которые занимают "неизвестно сколько места":

   int x[p],y[p]; //Эй, сколько места занимает x? А y?

Есть два способа. Первый - задать размер "с избытком": если известно, что n<1000, то

   int x[1000],y[1000];

решает проблему.
Второй, более правильный - использовать динамический массив:

int *x; //Указатель на массив int-ов
x = malloc(p*sizeof(int)); //Выделяем место, когда p уже известно
int *y;
y = malloc(p*sizeof(int)); //То же для y

[Bugrimov]

(i = 1, 2.....) - это вероятно указание, что цикл начинается с 1 и т.д.

[Bugrimov]

int *x; //Указатель на массив int-ов
x = (int*)malloc(p*sizeof(int)); //Выделяем место, когда p уже известно
int *y;
y = (int*)malloc(p*sizeof(int)); //То же для y

Лично мне такой вариант нравится!
Кстати компилятор на стену полез!!!!!!!!!!

[Базиля]

Какие ошибки?

[abstractionhawk]

У меня CodeBlocs нормально запускает программу

[abstractionhawk]

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