Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
#1

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

22.06.2017, 20:32. Просмотров 266. Ответов 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
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
22.06.2017, 20:32
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Непонятное поведение функтора (C++):

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

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

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

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

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

Непонятное поведение простейшей формулы - C++
Добрый вечер. Дано: double a; double d; unsigned int c; unsigned int b; for loop{ //манипуляции

15
MrGluck
Модератор
Эксперт CЭксперт С++
7491 / 4606 / 692
Регистрация: 29.11.2010
Сообщений: 12,590
22.06.2017, 20:38 #2
У вас результат возвращается только под if веткой, надо бы в любом случае что-то вернуть

Добавлено через 1 минуту
Лучше бы знать, конечно, цель данного кода
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:05  [ТС] #3
MrGluck, сумма произведений четных эл-тов 1 вектора и нечетных 2-го вектора
http://ideone.com/J1zpst
0
MrGluck
Модератор
Эксперт CЭксперт С++
7491 / 4606 / 692
Регистрация: 29.11.2010
Сообщений: 12,590
22.06.2017, 21:35 #4
zarko97, а почему бы это просто циклом не сделать?
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:38  [ТС] #5
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
TheCalligrapher
С чаем беда...
Эксперт CЭксперт С++
3945 / 2169 / 553
Регистрация: 18.10.2014
Сообщений: 3,783
22.06.2017, 21:43 #6
Цитата Сообщение от zarko97 Посмотреть сообщение
сумма произведений четных эл-тов 1 вектора и нечетных 2-го вектора
И что? Еще раз: ваш функтор вызывает неопределенное поведение, ибо ничего не возвращает, если не выполняется условие. Если вы суммируете произведения, то возвращайте 0 - он не повлияет на сумму.

Отдельно стоит заметить ошибку в логике: оператор && вычисляется по сокращенной схеме, т.е. this->j++ будет выполняться не всегда. В результате индексы "поедут" и функтор не будет работать правильно. Оставьте дурную привычку совать несколько действий в одно выражение.
1
MrGluck
Модератор
Эксперт CЭксперт С++
7491 / 4606 / 692
Регистрация: 29.11.2010
Сообщений: 12,590
22.06.2017, 21:46 #7
Цитата Сообщение от zarko97 Посмотреть сообщение
фором как раз и сделал
Фором как раз и лучше. Иначе получался STL ради STL. А так код даже новички смогут понять. А чем проще код - тем меньше ошибок, что вы собственно и успешно доказали.
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:46  [ТС] #8
TheCalligrapher,
http://ideone.com/oha2Pf
хм...теперь 3 и 8 не учитывается
0
TheCalligrapher
С чаем беда...
Эксперт CЭксперт С++
3945 / 2169 / 553
Регистрация: 18.10.2014
Сообщений: 3,783
22.06.2017, 21:47 #9
Цитата Сообщение от zarko97 Посмотреть сообщение
хм...теперь 3 и 8 не учитывается
Отдельно стоит заметить ошибку в логике: оператор && вычисляется по сокращенной схеме, т.е. this->j++ будет выполняться не всегда. В результате индексы "поедут" и функтор не будет работать правильно. Оставьте дурную привычку совать несколько действий в одно выражение.
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
22.06.2017, 21:51  [ТС] #10
TheCalligrapher, уже увидел
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
24.06.2017, 00:53  [ТС] #11
окончательный вариант: 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
TheCalligrapher
С чаем беда...
Эксперт CЭксперт С++
3945 / 2169 / 553
Регистрация: 18.10.2014
Сообщений: 3,783
24.06.2017, 02:36 #12
Цитата Сообщение от 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
Ferrari F1
619 / 513 / 101
Регистрация: 27.01.2015
Сообщений: 2,979
Записей в блоге: 1
Завершенные тесты: 1
24.06.2017, 08:09 #13
мораль темы такова, что лучше не выеживаться, мол какой я крутой, сейчас буду юзать из библиотеки все подряд (std::binary_function кстати в с++17 удален),
типо профи, а иногда лучше простыми средствами языка, но верно решить задачу
1
Antikl
178 / 174 / 50
Регистрация: 15.07.2015
Сообщений: 934
Завершенные тесты: 6
24.06.2017, 08:19 #14
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
а иногда лучше простыми средствами языка
так и делаю цыклы рулят
0
zarko97
278 / 38 / 0
Регистрация: 11.10.2015
Сообщений: 405
24.06.2017, 12:43  [ТС] #15
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
что лучше не выеживаться, мол какой я крутой
"...фором как раз и сделал...хотел вот так ещё попробовать"
0
24.06.2017, 12:43
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.06.2017, 12:43
Привет! Вот еще темы с ответами:

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

Непонятное поведение list, vector в другом классе - C++
Салют. class _auxObserver { --- list&lt;gmObjBase*&gt; mObservedObj; vector&lt;int&gt; b; public: ...

Шаблонный класс с шаблонным оператором внутри - непонятное поведение - C++
Написал &quot;кукольный&quot; пример, дабы не смущать читателей ненужными деталями. #include &lt;iostream&gt; template &lt;typename T&gt; struct foo...

Непонятное поведение потока ввода: в одной функции букву ввёл = во всех функциях наперёд ввёл. Распродажа! - C++
Парни, у меня возникла щекотливая ситуация. Делаю маленькую программу работающую с двоичным деревом поиска. Для удобства сделал что-то...


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

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

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