Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405

Непонятное поведение функтора

22.06.2017, 20:32. Показов 1052. Ответов 15
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Почему-то не считается произведение 1 и 6 ...
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
#include <iostream>
#include <algorithm>
#include <functional>
#include <vector>
#include <numeric>
 
template<class T>
struct multiply_range : public std::binary_function<T, T, T> {
public:
    multiply_range() : i(0), j(1) { }
    T operator()(T a, T b) {
        if (!(this->i++ % 2) && (this->j++ % 2)) {
        return a * b;
            } 
    }
private:
    std::size_t i, j;
};
 
 
int main() {
    std::vector<int> vec_numsx = {1,2,3,4,5};
    std::vector<int> vec_numsy = {5,6,7,8,9,10};
    std::cout << std::inner_product(vec_numsx.begin(), 
                                vec_numsx.end(), 
                                std::next(vec_numsy.begin()), 
                                0, 
                                std::plus<int>(),
                    multiply_range<int>());
    return 0;
}
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
22.06.2017, 20:32
Ответы с готовыми решениями:

Непонятное поведение
Всем привет! В книге &quot;Прата - Язык программирования C++. Лекции и управжения 2011&quot; нашёл вот такой пример: const free_throws...

Непонятное поведение функции
Есть некоторый метод класса: int mustAttack(Chess &amp;chess, Player&amp; p1, Player&amp; p2, int atackerName, int chessid) { if (chess.alive)...

Непонятное поведение программы
Привет народ, вот кароч: #include&quot;stdafx.h&quot; #include&quot;iostream&quot; using namespace std; class calc{ int *i, *j; public: int...

15
Форумчанин
Эксперт CЭксперт С++
 Аватар для MrGluck
8216 / 5047 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
22.06.2017, 20:38
У вас результат возвращается только под if веткой, надо бы в любом случае что-то вернуть

Добавлено через 1 минуту
Лучше бы знать, конечно, цель данного кода
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:05  [ТС]
MrGluck, сумма произведений четных эл-тов 1 вектора и нечетных 2-го вектора
http://ideone.com/J1zpst
0
Форумчанин
Эксперт CЭксперт С++
 Аватар для MrGluck
8216 / 5047 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
22.06.2017, 21:35
zarko97, а почему бы это просто циклом не сделать?
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:38  [ТС]
MrGluck, фором как раз и сделал...хотел вот так ещё попробовать
C++
1
2
double sum = double();
for (std::size_t i = 0, j = 1; i < vec_numsx.size() && j < vec_numsy.size(); i += 2, j += 2) sum += vec_numsx[i] * vec_numsy[j];
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12938 / 6805 / 1821
Регистрация: 18.10.2014
Сообщений: 17,224
22.06.2017, 21:43
Цитата Сообщение от zarko97 Посмотреть сообщение
сумма произведений четных эл-тов 1 вектора и нечетных 2-го вектора
И что? Еще раз: ваш функтор вызывает неопределенное поведение, ибо ничего не возвращает, если не выполняется условие. Если вы суммируете произведения, то возвращайте 0 - он не повлияет на сумму.

Отдельно стоит заметить ошибку в логике: оператор && вычисляется по сокращенной схеме, т.е. this->j++ будет выполняться не всегда. В результате индексы "поедут" и функтор не будет работать правильно. Оставьте дурную привычку совать несколько действий в одно выражение.
1
Форумчанин
Эксперт CЭксперт С++
 Аватар для MrGluck
8216 / 5047 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
22.06.2017, 21:46
Цитата Сообщение от zarko97 Посмотреть сообщение
фором как раз и сделал
Фором как раз и лучше. Иначе получался STL ради STL. А так код даже новички смогут понять. А чем проще код - тем меньше ошибок, что вы собственно и успешно доказали.
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:46  [ТС]
TheCalligrapher,
http://ideone.com/oha2Pf
хм...теперь 3 и 8 не учитывается
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12938 / 6805 / 1821
Регистрация: 18.10.2014
Сообщений: 17,224
22.06.2017, 21:47
Цитата Сообщение от zarko97 Посмотреть сообщение
хм...теперь 3 и 8 не учитывается
Отдельно стоит заметить ошибку в логике: оператор && вычисляется по сокращенной схеме, т.е. this->j++ будет выполняться не всегда. В результате индексы "поедут" и функтор не будет работать правильно. Оставьте дурную привычку совать несколько действий в одно выражение.
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:51  [ТС]
TheCalligrapher, уже увидел
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
24.06.2017, 00:53  [ТС]
окончательный вариант: http://ideone.com/E6FxAM

Добавлено через 43 секунды
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
#include <iostream>
#include <algorithm>
#include <functional>
#include <vector>
#include <numeric>
using namespace std::placeholders;
 
template<class T>
struct multiply_range : public std::binary_function<T, T, T> {
public:
    multiply_range() : i(0), j(1) { }
    T operator()(T a, T b) {
       /*if (!(this->i++ % 2)) {
           if (this->j++ % 2)
              return a * b;
       } else {
            this->j++;
            return 0;
        }*/
       return (!(this->i++ % 2)) ? ((this->j++ % 2) ? a * b : 0) : (this->j++, 0);
    }
private:
    std::size_t i, j;
};
 
 
int main() {
    auto Multiply = std::bind<int>(multiply_range<int>(), _1, _2);
    std::vector<int> vec_numsx = {1,2,3,4,5};
    std::vector<int> vec_numsy = {5,6,7,8,9,10};
    std::cout << std::inner_product(vec_numsx.begin(), 
                                    vec_numsx.end(), 
                                    std::next(vec_numsy.begin()), 
                                    0, 
                                    std::plus<int>(),
                                    Multiply);
    return 0;
}
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12938 / 6805 / 1821
Регистрация: 18.10.2014
Сообщений: 17,224
24.06.2017, 02:36
Цитата Сообщение от zarko97 Посмотреть сообщение
окончательный вариант
Однако сама идея этого варинта странна до ужаса. Индекс i начинает свою жизнь со значения 0, индекс j начинает свою жизнь со значения 1. При каждом вызове функтора оба индекса обязательно увеличиваются на 1.

Это означает, что на любом вызове функтора четность i гарантированно противоположна четности j. А это означает, что нет никакого смысла проверять четности i и j раздельно и делать какие-то ветвления, основанные на раздельной проверке четности i и j. И это также означает, что содержать два индекса тоже никакого смысла нет. Зачем все это?

Все, что вы пытаетесь реализовать - это чтобы ваш функтор возвращал a * b каждый четный вызов и возвращал 0 каждый нечетный вызов. Но делаете вы это каким-то до ужаса запутанным и нагроможденным способом.

Достаточно было бы сделать просто

C++
1
2
3
4
5
6
7
template<class T>
struct multiply_range {
  T operator ()(T a, T b) 
    { return (even = !even) ? a * b : 0; }
private:
  bool even = false;
};
Отдельно можно добавить, чей в общем случае функтор, чей результат зависит от правильности и стабильности порядка его вызова - не самая лучшая практика. Также std::binary_function - устаревшая фича, пользоваться котрой в наше время смысла нет.
1
807 / 534 / 158
Регистрация: 27.01.2015
Сообщений: 3,017
Записей в блоге: 1
24.06.2017, 08:09
мораль темы такова, что лучше не выеживаться, мол какой я крутой, сейчас буду юзать из библиотеки все подряд (std::binary_function кстати в с++17 удален),
типо профи, а иногда лучше простыми средствами языка, но верно решить задачу
1
с++
1282 / 523 / 225
Регистрация: 15.07.2015
Сообщений: 2,562
24.06.2017, 08:19
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
а иногда лучше простыми средствами языка
так и делаю цыклы рулят
0
 Аватар для zarko97
279 / 39 / 13
Регистрация: 11.10.2015
Сообщений: 405
24.06.2017, 12:43  [ТС]
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
что лучше не выеживаться, мол какой я крутой
"...фором как раз и сделал...хотел вот так ещё попробовать"
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12938 / 6805 / 1821
Регистрация: 18.10.2014
Сообщений: 17,224
24.06.2017, 21:15
Цитата Сообщение от zarko97 Посмотреть сообщение
C++
1
auto Multiply = std::bind<int>(multiply_range<int>(), _1, _2);
Также неясно, зачем было сделано вот это.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
24.06.2017, 21:15
Помогаю со студенческими работами здесь

Непонятное поведение программы
Доброго времени суток. Уважаемые форумчане, очень нуждаюсь в вашей консультации насчёт своей программы. Я не стану освещать задачу, которую...

Непонятное поведение программы
Пишу статическую либу. В ней есть кусок кода, который уходит в рекурсию.. Только причины мне непонятны.. Вот кусок кода std::string...

Непонятное мне поведение функции
Эта функция себя странно ведёт. Она работает и если с первого раза ввести цифры с клавиатуры, то работает корректно, но если сначала ввести...

Непонятное поведение простой функции
Решал одну задачу на рекурсию, долго не мог добится результата пока не понял что проблема не в рекурсии а в аргументах функции, сделал...

Непонятное поведение std::valarray
Есть такой код: #include &lt;iostream&gt; #include &lt;valarray&gt; using namespace std; int main() { valarray&lt;int&gt; a(2); a = 2;...


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

Или воспользуйтесь поиском по форуму:
16
Ответ Создать тему
Новые блоги и статьи
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru