Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
6 / 7 / 2
Регистрация: 18.05.2015
Сообщений: 124
1

Оптимизация кода

16.12.2017, 00:35. Просмотров 414. Ответов 3
Метки нет (Все метки)

Нужно очень сильно оптимизировать код, даже пусть с использованием потоков, если это возможно!
Суть: формируется последовательность (комбинация , размещение с повторением), а потом следующая комбинация в цикле формируется на основе предыдущей. В принципе мне необходимо сгенерировать 27,5 миллиардов комбинаций. В чистом виде генерация занимает 11 минут! При добавлении некоторых вычислений программа начинает уже тормозить
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
while (!arr[0]) {
/* Здесь я всеми силами пытаюсь оптимизировать данный цикл вычислений, он очень тормозит программу */
        #pragma omp parallel for
        for (int i = 0; i < 898; i++)
        {
        chast[i] = (arr[6] * sym[7 * i] + arr[5] * sym[7 * i + 1] + arr[4] * sym[7 * i + 2] + arr[3] * sym[7 * i + 3] + arr[2] * sym[7 * i + 4] + arr[1] * sym[7 * i + 5] + arr[0] * sym[7 * i + 6]) % 31;
        }
 
        double mass[31];
        memset(mass, 0, sizeof(mass));
        double q[31];
        for (int i = 0; i < 898; i++)
        {
                mass[chast[i]]++;
        }
        double khi_2 = 1;
        for (int i = 0; i < 31; i++)
        {
            q[i] = mass[i] / 898;
        }
        for (int i = 0; i < 31; i++)
        {
            khi_2 += q[i] * q[i];
        }
        if (khi_2 < 1.0)
        {
            cout << "khi square: " << khi_2 << '\n';
            for (int i = M - 1; i >= 0; i--)
                printf("%d ", arr[i]);
            printf("\n");
        }
        arr[M-1] ++;
        int j = M-1;
        while (arr[j] == N) {
            arr[j] = 0;
            arr[--j]++;
        }
 
    }
Пожалуйста, предлагайте всё, что можно! это очень важно!
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
16.12.2017, 00:35
Ответы с готовыми решениями:

Оптимизация кода
Покритикуйте, если не сложно. А еще очень интересно, как теперь это сделать в виде приложения на...

Оптимизация кода
Пожалуйста форумчане как можно сократить этот код Особенно от 27 до 90 строки #include &lt;iostream&gt;...

Оптимизация кода
Как можно оптимизировать данный программный код? Ответ объяснить void func() { for (int i...

оптимизация кода
Задача: определить, является ли последовательность скобок действительной. Длинна строки не...

3
93 / 77 / 73
Регистрация: 11.05.2015
Сообщений: 187
16.12.2017, 02:32 2
Из самых простых рекомендаций:
1. mass[chast[i]]++ можно занести в один цикл с chast[i] = (arr[6] * sym[7 * i] ...
2. аналогично khi_2 += q[i] * q[i] можно занести в один цикл с q[i] = mass[i] / 898
C++
1
2
3
4
5
        for (int i = 0; i < 31; i++)
        {
            q[i] = ((double)mass[i]) / 898.0;
            khi_2 += q[i] * q[i];
        }
3. для mass скорей всего можно использовать целый тип, вместо вещественного
4. Как в 25 строчке khi_2 может быть меньше 1, если оно изначально было равно 1, а к нем в цикле еще прибавили q[i]*q[i]?
0
1346 / 986 / 312
Регистрация: 28.07.2012
Сообщений: 2,743
16.12.2017, 02:46 3
Kaligulaa, у тебя циклы должны неплохо векторизоваться при помощи SSE и AVX инструкций. Однако код станет зависимым от платформы.
0
Evg
Эксперт CАвтор FAQ
21121 / 8137 / 628
Регистрация: 30.03.2009
Сообщений: 22,455
Записей в блоге: 30
16.12.2017, 14:18 4
Массив mass логично сделать int'ом. Чтобы в цикле из 898 итераций работа шла с более быстрыми int'овыми операциями, а в последующий цикл из 31 итерации можно добавить лишнюю операцию преобразования из int в double. В итоге по идее будет работать немного быстрее

Добавлено через 42 секунды
Ой.. не заметил сразу

Добавлено через 1 минуту
Цитата Сообщение от ivan37 Посмотреть сообщение
1. mass[chast[i]]++ можно занести в один цикл с chast[i] = (arr[6] * sym[7 * i] ...
Просто так нельзя из-за распараллеливания. chast[i] для разных значений i могут оказаться одинаковыми, поэтому mass[chast[i]] в разных итерациях цикла могут работать с одним и тем же элементом. В omp это можно настроить, но появятся лишние синхронизации, надо экспериментировать, затормозят они работу, или нет

Добавлено через 1 минуту
В посте #2 пункт 2 - после такой правки массив q[31] не нужен вообще

Добавлено через 2 минуты
C
arr[j] = 0;
arr[--j]++;
Код выглядит бестолково. Пусть j == 10. Тогда мы сначала выполняем a[10]=0; затем j=9, затем a[9]++. На следующией итерации выполняется a[9]=0. Не понятно, зачем мы в прошлой итерации выполняли a[9]++;

Добавлено через 2 минуты
А... там условие цикла.... В общем, логику этого места не понимаю, ну да ладно

А так, просто глядя на исходники (точнее, на огрызок исходников) больше особо оптимизировать нечего через переписывание кода. Дальше идут только машинно-зависимые оптимизации компилятора. Если он сможет задействовать SIMD'ы, то это значительно ускорит код
1
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
16.12.2017, 14:18

Заказываю контрольные, курсовые, дипломные и любые другие студенческие работы здесь.

Оптимизация кода
В общем дело такое, мне нужно 2 одинаковые программы(небольшие), только одна программа должна быть...

Оптимизация кода
Здравствуйте! у меня есть такая функция, которая очень часто вызывается: int fun(int x_,int...

Оптимизация кода
В С++ я совсем недавно, вот задали задачку: Перемножить 2 матрицы MxМ, элементами которых...

Оптимизация кода
Нужно както упростить програму /* Дан массив а если в нем встречается такой элемент что а &lt;a ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2020, vBulletin Solutions, Inc.