[C++][STL][for_each] Почему не работает?

попробуй

size->begin(), size->end()

> Использовать for не хочется. Не модно.

Ну только если действительно за «модой» гнаться...

for_each же не телепат, в отличие от нас - прибиндь calcSize к this.

> прибиндь calcSize к this

что это значит? :/

size->begin(), size->end()

по-моему, это точно работать не будет... я кстати вместо size поставил dimensions, а то путаница имела место быть

[гонюсь-за-модой-mode]

class Cell
{
public:
  Cell(std::vector<double>& sizes)
  {
    this->sizes = sizes;
    std::for_each(sizes.begin(), sizes.end(), calculate_size);
    volume = calculate_size.GetResult();
  }
  
private:
  class Multiplier
  {
    double result;
    
  public:
    Multiplier()
      : result(1.0)
    {
    }
    
    void operator() (double size)
    {
      result *= size;
    }

    double GetResult() const
    {
      return result;
    }
  } calculate_size;

private:
  std::vector<double> sizes;
  double volume;
};

не очень понимаю в целом :/

это функатор вводится?

и что есть this?

> прибиндь calcSize к this

что это значит? :/

видимо использование указателя this. как-то вот так: this->calcSize

я понял. я не знаю, что есть this и не нашёл понятной документации по нему на великомогучем, не подскажете что-нибудь? или просто вкратце объясните.

Кратко, но должно помочь. [url]http://www.cplusplus.com/doc/tutorial/classes2/[/url]

Объясните мне вот какой момент: накой foreach передавать начало и конец коллекции? Я себе представлял, что foreach работает для _всей_ коллекции сразу, т.е. должен быть синтаксис вроде

 for_each(vector, f) 

Если коротко, то this - это указатель на экземпляр класса для которого вызывается метод. Любой метод класса на самом деле принимает всегда на один аргумент больше, чем ты пишешь. Первый «невидимый» аргумент как раз и есть this.

Тоеть если ты пишешь

class Cell{
//...
   void calcSize(double r){
   //...
   }
//...
}

То, на самом деле, имеется ввиду

class Cell{
//...
   void calcSize(Cell* this, double r){
   //...
   }
//...
}

Естественно, что такую функцию с двумя аргументами в твоем for_each использовать нельзя, так как там функция должна применяться каждый раз к одному double.

Скорее всего потому, что он для перебора элементов внутри старт - енд? :)

Это я понимаю. Зачем это делать аргументом функции, если foreach — это более абстрактная сущность. Коллекция, которую перебирают, очевидно уже имеет итератор для перебора и указатели на начало и конец.

Да я согласен :) Шутка была. Я вообще не понимаю нужности данной реализации форича.

Используйте mem_fun. А еще лучше - for :)

#include <vector>
#include <functional>
#include <algorithm>
#include <iostream>

using namespace std;

class Cell
{
        vector<double> dimensions;      //массив размеров ячейки по осям (r) 
        double volume;      //объём (ν) 
        void calcSize(double r)
        {
                volume*=r;
        }
        public:
                Cell(vector<double> inputDimensions) {
                        dimensions=inputDimensions;
                        volume=1;
                        for_each(dimensions.begin(), dimensions.end(), bind1st(mem_fun(&Cell::calcSize), this));
                        cout << "Dimmension: " << volume << endl;
                }
};

int main(int argc, char *argv[])
{
        vector<double> t;
        t.push_back(2);
        t.push_back(3);
        t.push_back(4);
        t.push_back(5);

        Cell cell(t);

        return 0;
}

http://www.cs.ioc.ee/yik/schools/sum2006/generic_programming/iterators.html

Ну и SICP по новой читать.

Используйте mem_fun. А еще лучше - for :)

Ладно, не мудрствуя лукаво использую for.

public:
    Cell(vector<double> inputDimensions, vector<NearCell> inputNearCells) {
        dimensions=inputDimensions;
	nearCells=inputNearCells;
        volume=1;
        for (int i=0; i<dimensions.size(); ++i) {
            volume*=dimensions[i];
        }

foreach — это более абстрактная сущность

именно поэтому: в данном случае это не перебор элементов одной коллекции, а перебор элементов между двумя итераторами. более абстрактная сущность, ага

SICP

Map является важным конструктом, не только потому, что она фиксирует общую схему, но и потому, что она повышает уровень абстракции при работе со списками. В исходном определении scale-list рекурсивная структура программы привлекает внимание к поэлементной обработке списка. Определение scale-list через map устраняет этот уровень деталей и подчеркивает, что умножение преобразует список элементов в список результатов. Разница между этими двумя определениями состоит не в том, что компьютер выполняет другой процесс (это не так), а в том, что мы думаем об этом процессе по-другому. В сущности, map помогает установить барьер абстракции, который отделяет реализацию процедур, преобразующих списки, от деталей того, как выбираются и комбинируются элементы списков. Подобно барьерам на рисунке 2.1, эта абстракция позволяет нам свободно изменять низкоуровневые детали того, как реализованы списки, сохраняя концептуальную схему с операциями, переводящими одни последовательностив в другие. В разделе 2.2.3 такое использование последовательностей как способ организации программ рассматривается более подробно.

Обращаем внимание на выделенный фрагмент. vector.Begin() и vector.End() — это детали реализации контейнера, которые совершенно неважны в данном контексте, и так понятно, что раз надо что-то перебирать, то соответствующие методы у коллекции быть должны.

лучше:

register size_t len = dimensions.size();
for (register size_t i = 0; i < len; i++) {
    volume *= dimensions[i];
}

Ну или:

for (vector<double>::const_iterator i = dimensions.begin(); i != dimensions.end(); i++) {
    volume *= *i;
}

А твой use case в духе абстрактного программирования должен решаться комбинированием filter и map(foreach): сначала мы их коллекции достаем элементы с заданными свойствами (например, определенного положения), а потом делаем с ними, что необходимо. Такая идиома работает не только с коллекциями-обобщениями массивов, но и, например, с бесконечными потоками.

А твой use case в духе абстрактного программирования должен решаться комбинированием filter и map

это несколько, гм, разные вещи

сначала мы их коллекции достаем элементы с заданными свойствами

в случае использования итераторов мы ничего не достаём. мы просто определяем множество, с которым работаем, в терминах его границы

Такая идиома работает не только с коллекциями-обобщениями массивов, но и, например, с бесконечными потоками.

ну покажи как с помощью map/filter красиво и эффективно утроить (например) второй, третий, и четвёртый элемент списка

ты пытаешься использовать map (for_each) вместо fold (accumulate). ещё и неверно пытаешься

Гонишься за модой - юзай пайтон.
Хочешь сипэпэ - жуй простыни ошибок темплейтов.
Вот и вся правда.

нефункционально. да, благодаря изменяемому состоянию на С++ можно реализовать свёртку посредством for_each - но зачем?

Так трудно было прочитать по ссылке?