Исправить код с массивами

@LexaNoob · Регистрация: 09.08.2019

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

Если запускать через локальный отладчик WIndows то перестает выдавать числа после 4 строки, хотя я ждал, а если нажимать на F10 то все работает. Подскажите в чем проблема, пожалуйста. И если массив total[] увеличить до 96, d1 до 24 в цикле do,поставить в цикле for(int = 1; i<=20;i++) изменить на 96, то тоже зависает, и если через локальный отладчик тоже зависает
и через в F10 тоже. Помогите пожалуйста!!!!!!!!!!!!!!!!!!!!!!!!!!!!

C++

#include "pch.h"
#include <iostream>
#include <string>
#include <string.h>
#include <ctime>
using namespace std;
int main()
{
    int a, b, c, d, e, f, i, ac, r1;
    int r2 = 0; 
    int u = 0;
    int t = 0;
    r1 = 0;
    b = -1;
    ac = 0;
    c = 0;
    d = 0;
    a = 0;
    f = 0;
    i = 0;
    e = 0;
    int d1 = 0;
    char str[4];
    string slovo = "1234";
    int size = slovo.length();
    srand(time(NULL));
    char total[20]; 
    int arr2[4];
    for (int i = 0; i < size; i++)
    {
        str[i] = '0';
    }
    for (int i = 0; i < 20; i++)
    {
        total[i] = '0';
    }
    //////////////////////////////////////////////
    do
    {
        srand(time(NULL));
        int k = rand() % size;
        for (int i = 0; i < size; i++)
        {
            if (arr2[i] == k)
            {
                r2 = 0;
            }
        }
        if (r2 != 0)
        {
            arr2[e++] = k;
            str[a++] = slovo[k];
            f++;
        }
        r2 = 5;
    } while (f != size);
    a = 0;
    f = 0;
    e = 0;
    for (int i = 0; i < size; i++)
    {
        arr2[i] = -1;
    }
    do 
    {
            for (int i = 1; i <= 20; i++)
            {   
                c = i;
                c--;
                b++;
                if (total[c] == str[b] && total[c] != '0'&& str[b] != '0')
                {
                    t++;
                }
                if (i % 4 == 0 && i != 0)
                {
                    b = -1;
                    t = 0;
                }
                if (t >= 3)
                {
                    t = 0;
                    b = -1;
                    u = 3;
                    break;
                }
                else
                {
                    u = 1;
                }
            }
            if (u == 1)
            {
                b = -1;
                t = 0;
                for (int i = 0; i < size; i++)
                {
                    if (total[r1] == '0')
                    {
                        total[r1] = str[i];
                        cout << total[r1];
                        r1++;
                    }
                } 
                d1++;
                cout << endl;
            }
                do
                {
                    srand(time(NULL));
                    int k = rand() % size;
                    for (int i = 0; i < size; i++)
                    {
                        if (arr2[i] == k)
                        {
                            r2 = false;
                        }
                    }
                    if (r2 != false)
                    {
                        arr2[e++] = k;
                        str[a++] = slovo[k];
                        f++;
                    }
                    r2 = 5;
                } while (f != size);
                a = 0;
                f = 0;
                e = 0;
        for (int i = 0; i < size; i++)
        {
            arr2[i] = -1;
        }
    } while (d1 != 5);
    return 0;
}

@zss · 27.08.2019, 16:36

1.
строки 40 и 110:

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

srand(time(NULL));

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

srand(time(NULL));

не просто лишние, а вредные - заставляют генерировать одинаковые числа, т.к. между вызовами меньше 1 секунды.
2.
строка 28

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

int arr2[4];

объявили и неинициализировали
а в строке 44

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

if (arr2[i] == k)

пытаемся этот мусор сравнить с со значением переменной k

@LexaNoob · 27.08.2019, 23:18 **[ТС]**

C++

#include "pch.h"
#include <iostream>
#include <string.h>
using namespace std;
int main()
{
    int a, b, c, d, e, f, i, ac, r1;
    int r2 = 0;
    int u = 0;
    int t = 0;
    r1 = 0;
    b = -1;
    ac = 0;
    c = 0;
    d = 0;
    a = 0;
    f = 0;
    i = 0;
    e = 0;
    int d1 = 0;
    char str[4];
    char slovo [999] ="1234";
    char total[96]; //массив для храние вариантов равен факториалу slovo * str
    int arr2[4];///// рандом числа 
    ///////////////////////////////////////////////   Заполнение массива
    for (int i = 0; i < strlen(slovo); i++)
    {
        str[i] = '0';
    }
    for (int i = 0; i < 96; i++)
    {
        total[i] = '0';
    }
    //////////////////////////////////////////////
    do
    {
        int k = rand() % strlen(slovo);
        for (int i = 0; i < strlen(slovo); i++)
        {
            if (arr2[i] == k)
            {
                r2 = 0;
            }
        }
        if (r2 != 0)
        {
            arr2[e++] = k;
            str[a++] = slovo[k];
            f++;
        }
        r2 = 5;
    } while (f != strlen(slovo));
    a = 0;
    f = 0;
    e = 0;
    for (int i = 0; i < strlen(slovo); i++)
    {
        arr2[i] = -1;
    }
    do //// основной цикл на кол-во
    {
        for (int i = 1; i <= 100; i++)
        {
            c = i;
            c--;
            b++;
            if (total[c] == str[b] && total[c] != '0'&& str[b] != '0')
            {
                t++;
            }
            if (i % 4 == 0 && i != 0)
            {
                b = -1;
                t = 0;
            }
            if (t >= 3)
            {
                t = 0;
                b = -1;
                u = 3;
                break;
            }
            else
            {
                u = 1;
            }
        }
        if (u == 1)
        {
            b = -1;
            t = 0;
            for (int i = 0; i < strlen(slovo); i++)
            {
                if (total[r1] == '0')
                {
                    total[r1] = str[i];
                    cout << total[r1];
                    r1++;
                }
            }
            d1++;
            cout << " " << d1;
            cout << endl;
        }
        do
        {
 
            int k = rand() % strlen(slovo);
            for (int i = 0; i < strlen(slovo); i++)
            {
                if (arr2[i] == k)
                {
                    r2 = 0;
                }
            }
            if (r2 != 0)
            {
                arr2[e++] = k;
                str[a++] = slovo[k];
                f++;
            }
            r2 = 5;
        } while (f != strlen(slovo));
        a = 0;
        f = 0;
        e = 0;
        for (int i = 0; i < strlen(slovo); i++)
        {
            arr2[i] = -1;
        }
    } while (d1 != 24);///////////// конец цыкла
    return 0;
}

@zss · 28.08.2019, 06:58

Не вижу особых изменений.
Массив arr2 как был неинициализированным, так и остался.
И куда делась 26 строка начальной инициализации датчика случайных чисел?

И, наконец, напишите постановку задачи.

@LexaNoob · 28.08.2019, 10:45 **[ТС]**

arr2 инициализируется потом туда записываются рандом числа, а по ним генерируется строка, а потом я их очищаю, записываю -1 потому что в массиве нет такого индекса и задача должна выводить все перестановки строки,кроме '0' и можно мне объяснить почему в том цикле который я отметил в комментариях можно написать 1, хотя по моей логике такого быть не должно. Там должна стоять цифра 4320, а с ней не работает.

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

@zss · 28.08.2019, 10:52

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

arr2 инициализируется

Еще раз:

строка 21

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

int arr2[4];

объявили и неинициализировали
далее, а в строке 40

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

if (arr2[i] == k)

пытаемся этот мусор сравнить с со значением переменной k
А уже потом в строках 56-59:

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

C++

for (int i = 0; i < strlen(slovo); i++)
 {
 arr2[i] = -1;
 }

И, ЕЩЕ РАЗ: напишите постановку задачи.

@zss Модератор 13706 / 10909 / 6473 Регистрация: 18.12.2011 Сообщений: 29,127
	27.08.2019, 16:36	2
	Сообщение было отмечено LexaNoob как решение Решение 1. строки 40 и 110: Сообщение от LexaNoob srand(time(NULL)); Сообщение от LexaNoob srand(time(NULL)); не просто лишние, а вредные - заставляют генерировать одинаковые числа, т.к. между вызовами меньше 1 секунды. 2. строка 28 Сообщение от LexaNoob int arr2[4]; объявили и неинициализировали а в строке 44 Сообщение от LexaNoob if (arr2[i] == k) пытаемся этот мусор сравнить с со значением переменной k 1

@zss Модератор 13706 / 10909 / 6473 Регистрация: 18.12.2011 Сообщений: 29,127
	28.08.2019, 06:58	4
	Не вижу особых изменений. Массив arr2 как был неинициализированным, так и остался. И куда делась 26 строка начальной инициализации датчика случайных чисел? И, наконец, напишите постановку задачи. 0

@LexaNoob 0 / 0 / 0 Регистрация: 09.08.2019 Сообщений: 88
	28.08.2019, 10:45 [ТС]	5
	arr2 инициализируется потом туда записываются рандом числа, а по ним генерируется строка, а потом я их очищаю, записываю -1 потому что в массиве нет такого индекса и задача должна выводить все перестановки строки,кроме '0' и можно мне объяснить почему в том цикле который я отметил в комментариях можно написать 1, хотя по моей логике такого быть не должно. Там должна стоять цифра 4320, а с ней не работает. Добавлено через 1 час 1 минуту Цикл в строке 62 0

	28.08.2019, 10:52

Исправить код с массивами

Решение