Не заходит в тело цикла

@Iangyl · Регистрация: 31.03.2018

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

Здравствуйте! Всех с праздниками!
Помогите пожалуйста, тут такая проблема... У меня в цикле:

C++

for (int j = 0; j != numerator; j++)
    {
        if (mas[j] > max) max = mas[j];
    }

переменная "j" сразу же получает значение "numerator" соответственно цыкл закрывается не начавшись. В чем может быть проблема?
P. S. Переменная "numerator" не равна 0.

~~Pavel_Srgv~~ · 09.04.2018, 16:53

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

переменная "j" сразу же получает значение "numerator"

Как узнал?

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

Переменная "numerator" не равна 0.

Тот же вопрос.

@zss · 09.04.2018, 17:38

Вангую, на самом деле у Вас написано:

C++

1	for (int j = 0; j != numerator; j++); // тут ;

@Iangyl · 10.04.2018, 10:19 **[ТС]**

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

Как узнал?Тот же вопрос.

Отладка

Добавлено через 1 минуту

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

Вангую, на самом деле у Вас написано:

C++

1	for (int j = 0; j != numerator; j++); // тут ;

Нда... У вас хорошее чувство юмора...

nd2 · 10.04.2018, 11:30

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

Помогите пожалуйста,

Кода мало.

@Iangyl · 10.04.2018, 15:05 **[ТС]**

nd2, Вот:

C++

#include <iostream>
#include <fstream>
#include <ctime>
using namespace std;
 
void funct_f();
void funct_file(int *array, int size);
 
void main()
{
    int size = 2;
    int mass[2];
    funct_f();
    funct_file(mass, size);
    int Dan9_JIox;
    ifstream exit_1("exesize.txt");
    cout << "35 means that had file: " << endl;
    for (int l = 0; l < 35; l++)
    {
        exit_1 >> Dan9_JIox;
        cout << Dan9_JIox << " ";
    }
    cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: " << mass[1] << endl;
    exit_1.close();
}
 
void funct_f()
{
    srand(time(0));
    ofstream enter("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        enter << 1 + rand() % 100 << " ";
    }
    enter.close();
}
 
void funct_file(int *array, int size)
{
    int numerator = 0;
    int k;
    int mas[35];
    ifstream exit("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        exit >> k;
        if (k % 10 == 5)
        {
            mas[i] = k;
            numerator++;
        }
    }
    int max = -9999;
    for (int j = 0; j != numerator; j++)
    {
        if (mas[j] > max) max = mas[j];
    }
    array[0] = numerator;
    array[1] = max;
    exit.close();
    ofstream enter_1("exesize.txt", ios::app);
    enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: " << array[1];
    enter_1.close();
}

@tdo22 · 10.04.2018, 15:11

Может я не понял суть проблемы, но у меня вроде как все работает:

C++

#include <iostream>
#include <fstream>
#include <ctime>
using namespace std;
 
void funct_f();
void funct_file(int *array, int size);
 
int main()
{
    int size = 2;
    int mass[2];
    funct_f();
    funct_file(mass, size);
    int Dan9_JIox;
    ifstream exit_1("exesize.txt");
    cout << "35 means that had file: " << endl;
    for (int l = 0; l < 35; l++)
    {
        exit_1 >> Dan9_JIox;
        cout << Dan9_JIox << " ";
    }
    cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: " << mass[1] << endl;
    exit_1.close();
 
    return 0;
}
 
void funct_f()
{
    srand(time(0));
    ofstream enter("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        enter << 1 + rand() % 100 << " ";
    }
    enter.close();
}
 
void funct_file(int *array, int size)
{
    int numerator = 0;
    int k;
    int mas[35];
    ifstream exit("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        exit >> k;
        if (k % 10 == 5)
        {
            mas[i] = k;
            numerator++;
        }
    }
    int max = -9999;
    for (int j = 0; j != numerator; j++)
    {
        if (mas[j] > max) max = mas[j];
    }
    array[0] = numerator;
    array[1] = max;
    exit.close();
    ofstream enter_1("exesize.txt", ios::app);
    enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: " << array[1];
    enter_1.close();
}

все что я сделал это изменил void main() на int main() ну и в конце return 0;
numerator в начале цикла равна 3

nd2 · 10.04.2018, 18:27

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

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

Карты так легли.
Iangyl, это лучше в main перенести:

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

C++

1	srand(time(0));

И учитывай, что в файл записываются случайные числа, среди которых может и не оказаться подходящих под условие:

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

C++

if (k % 10 == 5) 
{ 
    mas[i] = k; 
    numerator++; 
}

Тогда numerator останется нулевым, и не будет захода в цикл.

@Iangyl · 10.04.2018, 21:27 **[ТС]**

tdo22, nd2, Вот я все поменял по вашим рекомендациям. На счет: предусмотреть если numerator = 0 - хорошая идея, спасибо. По поводу сменить void на int, я конечно не догнал, как это может повлиять, но решил попробывать, теперь знаю, что никак, тоже спасибо(ибо не проверил бы не узнал бы).
Но прога не пашет numerator не равен 0, но max по прежнему не находится.

Вот актуальный код:

C++

#include <iostream>
#include <fstream>
#include <ctime>
using namespace std;
 
void funct_f();
void funct_file(int *array, int size);
 
int main()
{
    srand(time(0));
    int size = 2;
    int mass[2];
    funct_f();
    funct_file(mass, size);
    int Dan9_JIox;
    ifstream exit_1("exesize.txt");
    cout << "35 means that had file: " << endl;
    for (int l = 0; l < 35; l++)
    {
        exit_1 >> Dan9_JIox;
        cout << Dan9_JIox << " ";
    }
    if (mass[1] = -9999) cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: none" << endl;
    else cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: " << mass[1] << endl;
    exit_1.close();
    return 0;
}
 
void funct_f()
{
    ofstream enter("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        enter << 1 + rand() % 100 << " ";
    }
    enter.close();
}
 
void funct_file(int *array, int size)
{
    int numerator = 0;
    int k;
    int mas[35];
    ifstream exit("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        exit >> k;
        if (k % 10 == 5)
        {
            mas[i] = k;
            numerator++;
        }
    }
    int max = -9999;
    for (int j = 0; j != numerator; j++)
    {
        if (mas[j] > max) max = mas[j];
    }
    array[0] = numerator;
    array[1] = max;
    exit.close();
    ofstream enter_1("exesize.txt", ios::app);
    if (array[1] == -9999) enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: none";
    else
    enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: " << array[1];
    enter_1.close();
}

А вот результаты компиляции:

Кликните здесь для просмотра всего текста

nd2 · 11.04.2018, 00:26

Вот так измени кусок, и выложи скрин работы:

C++

    int max = -9999;
    cout << "numerator = " << numerator << endl;
    for (int j = 0; j != numerator; j++)
    {
        cout << "mas[j] = " << mas[j] << endl;
        if (mas[j] > max) 
            max = mas[j];
    }

@zss · 11.04.2018, 01:08

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

int max = -9999;

Возможно, что это и не важно (все mas[j]>-9999), но таких ВОЛШЕБНЫХ чисел в алгоритмах должно быть поменьше

C++

1	nt max = mas[0];

@sourcerer · 11.04.2018, 09:12

Комментарий модератора

Iangyl, пожалуйста, прочитайте правила форума.
Особое внимание обратите на пункт 4.7 (как правильно сформулировать ТЗ).

.

@GbaLog- · 11.04.2018, 09:57

if (mass[1] = -9999)

@Iangyl · 11.04.2018, 15:01 **[ТС]**

nd2, Вот:

@Iangyl · 11.04.2018, 15:02 **[ТС]**

Сообщение от GbaLog-

Где вы видели такой э-нт в моем коде?

@GbaLog- · 11.04.2018, 15:58

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

Где вы видели такой э-нт в моем коде?

строка 24 в #9.

nd2 · 11.04.2018, 16:58

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

Вот:

На этом скрине не видно куска изменённого кода.

@Iangyl · 11.04.2018, 17:18 **[ТС]**

Сообщение от GbaLog-

строка 24 в #9.

Вижу, спасибо.

@Iangyl · 11.04.2018, 18:04 **[ТС]**

nd2,
Вот скрин:

Вот актуальный код:

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <fstream>
#include <ctime>
using namespace std;
 
void funct_f();
void funct_file(int *array, int size);
 
int main()
{
    srand(time(0));
    int size = 2;
    int mass[2];
    funct_f();
    funct_file(mass, size);
    int Dan9_JIox;
    ifstream exit_1("exesize.txt");
    cout << "35 means that had file: " << endl;
    for (int l = 0; l < 35; l++)
    {
        exit_1 >> Dan9_JIox;
        cout << Dan9_JIox << " ";
    }
    if (mass[1] == -9999) cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: none" << endl;
    else cout << endl << "Amount of means that ended on 5: " << mass[0] << "\nMax mean that ended on 5: " << mass[1] << endl;
    exit_1.close();
    return 0;
}
 
void funct_f()
{
    ofstream enter("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        enter << 1 + rand() % 100 << " ";
    }
    enter.close();
}
 
void funct_file(int *array, int size)
{
    int numerator = 0;
    int k;
    int mas[35];
    ifstream exit("exesize.txt");
    for (int i = 0; i < 35; i++)
    {
        exit >> k;
        if (k % 10 == 5)
        {
            mas[i] = k;
            numerator++;
        }
    }
    int max = -9999;
    cout << "numerator = " << numerator << endl;
    for (int j = 0; j != numerator; j++)
    {
        if (mas[j] > max) max = mas[j];
    }
    array[0] = numerator;
    array[1] = max;
    exit.close();
    ofstream enter_1("exesize.txt", ios::app);
    if (array[1] == -9999) enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: none";
    else
    enter_1 << "Amount of means that ended on 5: " << array[0] << "\nMax mean that ended on 5: " << array[1];
    enter_1.close();
}

nd2 · 11.04.2018, 18:11

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

Вот скрин:

Теперь сравни твой код с кодом в 10-м посте.

@GbaLog- Любитель чаепитий 3742 / 1798 / 566 Регистрация: 24.08.2014 Сообщений: 6,016 Записей в блоге: 1
	11.04.2018, 09:57	13
	if (mass[1] = -9999) 0

@Iangyl 32 / 34 / 21 Регистрация: 31.03.2018 Сообщений: 495
	11.04.2018, 15:01 [ТС]	14
	nd2, Вот: 0

~~Pavel_Srgv~~ Заблокирован
	09.04.2018, 16:53	2
	Сообщение от Iangyl переменная "j" сразу же получает значение "numerator" Как узнал? Сообщение от Iangyl Переменная "numerator" не равна 0. Тот же вопрос. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	10.04.2018, 11:30	5
	Сообщение от Iangyl Помогите пожалуйста, Кода мало. 0

@Iangyl 32 / 34 / 21 Регистрация: 31.03.2018 Сообщений: 495
	11.04.2018, 15:02 [ТС]	15
	Сообщение от GbaLog- Где вы видели такой э-нт в моем коде? 0

@GbaLog- Любитель чаепитий 3742 / 1798 / 566 Регистрация: 24.08.2014 Сообщений: 6,016 Записей в блоге: 1
	11.04.2018, 15:58	16
	Сообщение от Iangyl Где вы видели такой э-нт в моем коде? строка 24 в #9. 1

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	11.04.2018, 16:58	17
	Сообщение от Iangyl Вот: На этом скрине не видно куска изменённого кода. 0

@Iangyl 32 / 34 / 21 Регистрация: 31.03.2018 Сообщений: 495
	11.04.2018, 17:18 [ТС]	18
	Сообщение от GbaLog- строка 24 в #9. Вижу, спасибо. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	11.04.2018, 18:11	20
	Сообщение от Iangyl Вот скрин: Теперь сравни твой код с кодом в 10-м посте. 0