Почему код не работает C++?

Question

[mamadaliev]

Доброго времени суток,

сам код,

#include <iostream>
#include <cmath>

using namespace std;

int main()
{
    int m, n;
    float x;
    float s1=0;
    float s2=0;
    float y=0;
    cout << "Input x:" << endl;
    cin >> x;
    cout << "Input m:" << endl;
    cin >> m;
    cout << "Input n:" << endl;
    cin >> n;

    for (int i=1; i<n; i++)
    {
        for (int k=1; k<m; i++)
            s1 += (1/k+k);

        for (int j=1; j<m; j++)
            s2 += (sqrt(x+j/i));

        y += ((1+x*s1) / (3+s2));
    }
    cout << "Result: " << y << endl;
}

Comments

[Денис Загаевский]

Никакой ты не программист

[Mercury13]

Да, не программист ты, а ученик. До программиста ещё года три такой учёбы. Не меньше.

[mamadaliev]

Я только начал учить C++, не судите строго, пожалуйста!

Answer 1

[Mercury13]

Этим переменным не присваивали начального значения и в них могло оказаться что угодно.

Переводите с английского!

Comments

[mamadaliev]

Всёравно не работает

[Mercury13]

Шерзод Мамадалиев, Задача-то какая?

[mamadaliev]

Я лишь добавил такой код,
float s1=0;
float s2=0;
float y=0;

ошибки ушли, но код ничего не выводит на экран.

[Mercury13]

Шерзод Мамадалиев, В цикле по k ошибся с шагом цикла.

[Mercury13]

Шерзод Мамадалиев, Из-за чего зациклилось.

[Mercury13]

Шерзод Мамадалиев, Это называется «программирование копипастой», и оно — огромный источник ошибок даже у профессионала (читай блог PVS-Studio). Что уж говорить про ученика.

[mamadaliev]

Mercury13, разобрался! Спасибо большое за ответ и советы.

[mamadaliev]

Mercury13, подписался на PVS-Studio, буду читать, спасибо

Answer 2

[Андрей Капров]

Запустил на этот код PVS-Studio. Он выявил 2 ошибки:
s1 += (1/k+k);
V636 The '1 / k' expression was implicitly cast from 'int' type to 'float' type. Consider utilizing an explicit type cast to avoid the loss of a fractional part. An example: double A = (double)(X) / Y;. consoleapplication2017.cpp 34

Здесь целочисленное деление. Т.к. k это минимим 1, то при делении всегда будет получаться на первом проходе 1, а потом всё время 0. Исправление:
s1 += (1.0f/k+k);

Хотя мне кажется здесь ещё и скобки неверно поставлены и должно быть:

s1 += 1.0f/(k+k);

Вторая ошибка:

for (int i=1; i<n; i++)
    {
        for (int k=1; k<m; i++)

V533 It is likely that a wrong variable is being incremented inside the 'for' operator. Consider reviewing 'i'. consoleapplication2017.cpp 33

Опечатка. Во втором цикле увеличивается i, а надо k.

Используйте PVS-Studio. :)

Comments

[Деревянко Александр]

да, PVS-Studio...или просто учить язык базово, как надо.

[Андрей Капров]

да, PVS-Studio...или просто учить язык базово, как надо.

Одно другому не мешает. :) Даже профессиональные программисты не застрахованы, например, от опечаток в духе V533 - примеры V533.

[Денис Загаевский]

Внезапно. И часто вы приходите в такие вопросы?)

[Андрей Капров]

Нет. Если честно, случайно забрёл.