Можете ли вы захватить ссылку в рекурсивной лямбде?

Question

Я полностью решил определенную проблему на HackerRank (https://www.hackerrank.com/challenges/ctci-recursive-staircase/problem), используя рекурсивное решение с памяткой:

std::map<int, int> memoize;

int davis_staircase(int n) {
    if (n == 0) {
        return 1;
    } else if (n < 0) {
        return 0;
    }

    auto find = memoize.find(n);
    if (find != memoize.end()) {
        return find->second;
    }

    int num_steps = davis_staircase(n - 1) + davis_staircase(n - 2) + davis_staircase(n - 3);
    memoize[n] = num_steps;

    return num_steps;
}

Я хотел бы скрыть глобальный std::map (без использованиякласс), который я использую в качестве поиска и подумал, что я попытаюсь создать лямбду, которую я могу вызывать рекурсивно, а также захватить кэш / карту по ссылке. Я пробовал следующее:

int davis_staircase_2(int n) {

    std::map<int, int> memo;

    //auto recurse = [&memo](int n) -> int {                    // attempt (1)
    //std::function<int(int)> recurse = [&memo](int n) -> int { // attempt (2)
    std::function<int(std::map<int, int>&, int)> recurse = [](std::map<int, int>& memo, int n) -> int { // attempt (3)
        if (n == 0) {
            return 1;
        } else if (n < 0) {
            return 0;
        }

        auto find = memo.find(n);
        if (find != memo.end()) {
            return find->second;
        }

        //int num_steps = recurse(n - 1) + recurse(n - 2) + recurse(n - 3); // attempt (1) or (2)
        int num_steps = recurse(memo, n - 1) + recurse(memo, n - 2) + recurse(memo, n - 3); // attempt (3)

        memo[n] = num_steps;

        return num_steps;
    };

    //return recurse(n); // attempt (1) or (2)
    return recurse(memo, n); // attempt (3)
}

У меня есть 3 несколько разных попытки чередования выше, но я не могу заставить их скомпилировать. Возможно ли то, что я пытаюсь сделать, возможно?

Я использую clang на MacOS:

Apple LLVM version 10.0.0 (clang-1000.10.44.4)
Target: x86_64-apple-darwin18.2.0
Thread model: posix

Answer 1

Вы забыли захватить recurse, поэтому ваш код может быть

std::function<int(int)> recurse = [&recurse, &memo](int n) -> int { // attempt (2)

или

std::function<int(int)> recurse = [&](int n) -> int { // attempt (2)

Таким же образом, для // attempt (3):

std::function<int(std::map<int, int>&, int)> recurse = [&recurse](std::map<int, int>& memo, int n) -> int { // attempt (3)

// attempt (1) нельзя исправить как есть, так как тип recurse используется до его определения.

Чтобы сделать это без std::function, вы можете использовать Y-комбинатор (требуется C ++ 14, для общей лямбды):

int davis_staircase_2(int n) {
    std::map<int, int> memo;
    auto recurse = [&memo](auto self, int n) -> int { // attempt (4)
        if (n == 0) {
            return 1;
        } else if (n < 0) {
            return 0;
        }

        auto find = memo.find(n);
        if (find != memo.end()) {
            return find->second;
        }

        int num_steps = self(self, n - 1) + self(self, n - 2) + self(self, n - 3); // attempt (4)

        memo[n] = num_steps;

        return num_steps;
    };
    return recurse(recurse, n); // attempt (4)
}

Answer 2

Вы можете сделать рекурсивную лямбду без стирания типа (std :: function).Вот как это делается с общими лямбдами:

auto recurse = [](auto lambda) {
    return [lambda](auto&&... args) {
        return lambda(lambda, std::forward<decltype(args)>(args)...);
    };
};

auto my_recursive_lambda = recurse([](auto self, std::map<int, int>& memo, int n) {
    if (n == 0) {
        return 1;
    } else if (n < 0) {
        return 0;
    }

    auto find = memo.find(n);
    if (find != memo.end()) {
        return find->second;
    }

    int num_steps = self(self, memo, n - 1) + self(self, memo, n - 2) + self(self, memo, n - 3);

    memo[n] = num_steps;

    return num_steps;
});

my_recursive_lambda(memo, n); // magic!

Если вам действительно нужно это для c ++ 11, вам понадобится std::function:

auto recurse = std::function<int(std::map<int, int>&, int)>{};

recurse = [&recurse](std::map<int, int>& memo, int n) {
    // same as you tried.
}

Или если вы откажетесьДля удобства вы можете раскрутить свой лямбда-тип вручную:

struct {
    auto operator()(std::map<int, int>& memo, int n) -> int {
        auto&& recurse = *this;
        if (n == 0) {
            return 1;
        } else if (n < 0) {
            return 0;
        }

        auto find = memo.find(n);
        if (find != memo.end()) {
            return find->second;
        }

        //int num_steps = recurse(n - 1) + recurse(n - 2) + recurse(n - 3); // attempt (1) or (2)
        int num_steps = recurse(memo, n - 1) + recurse(memo, n - 2) + recurse(memo, n - 3); // attempt (3)

        memo[n] = num_steps;

        return num_steps;
    }
} recurse{};

Answer 3

Вам не нужна рекурсивная функция ...

int stepPerms(int n) {

  std::map<int, int> memoize;

  memoize[-2] = 0;
  memoize[-1] = 0;
  memoize[0] = 1;

 for(int i=1;i<=n;++i)
 {
   memoize[i] = memoize[i - 1] + memoize[i - 2] + memoize[i-3];
 }

 return memoize[n];
}