Правильно ли написан код?

[Никита2313]

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

int main()
{   setlocale(LC_ALL,"rus");
    int n,x;
    printf("Cумма Y из k=1 по n значений формулы\n");
    printf("Введите n - ");
    scanf("%d", &x);
    printf("Введите x - ");
    scanf("%d", &n);
    if(n<=1)
    {
        printf("Данные введены некорректно, повторите попытку");
    }
    else
    {
        int k,x;
        float Y,Y1;
        float a1,a2,a3 = 1;
        for(k=1;k<=n;k++)
        {
            a1=pow(-3,3*k+1);//
            a2=2*(k-2);
            a3=pow(x,3*k+1);
        }
            Y1=a1/a2*a3;
            Y=Y+Y1;

    printf("Y= %3f\n", Y);
    }



    int k;
    printf("Введите k - ");
    scanf("%d", &k);
    k=k+2;
    if(k<=1)
    {
        printf("Данные введены некорректно, повторите попытку");
    }
    else
    {
        int m;
        float X, X1;
        float a1,a2;
        for(m=1;m<=k;m++)
        {
            a1=pow(m,3)-8;
            a2=m-4;
        }
               X1=a1/a2;
               X=X*X1;
     printf("X= %3f\n", X);
     }
    return 0;
}

[ForenLi]

Нет, у вас же надо вывести S, а вы выодите X, Y. Кто это такие?

printf("Введите n - ");
scanf("%d", &x);
printf("Введите x - ");
scanf("%d", &n);

Просите одно, вводите другое.

for(k=1;k<=n;k++)
{
a1=pow(-3,3*k+1);//
a2=2*(k-2);
a3=pow(x,3*k+1);
}

Смысл переприсваивать переменные в цикле?.. Результат будет как при k=n.

Y=Y+Y1;

Y - неинициализированная переменная

int k;
printf("Введите k - ");
scanf("%d", &k);

Не надо k вводить, оно от суммы должно быть. Вы формулу не умеете читать?

if(k<=1)

Вы же уже прибавили 2. Смысл проверять?

Про вычисления - аналогично Y.