Решение рюкзака с возвратом в C ++

Question

Решение рюкзака с возвратом в C ++

0

У меня возникают проблемы, которые пытаются решить проблему Рюкпак с помощью обратного хода.

Например, для следующих значений функция Knapsack вернет 14 в качестве решения, но правильный результат должен быть равен 7.

int n = 3, weights[] = {2, 3, 1}, values[] = {6, 15, 7};

int solution = 0, max = 2;


void Knapsack (int i, int max, int value, int *solution)
{
  int k;

  for (k = i; k < n; k++) {
    if (weights[k] <= max) {
      Knapsack (k, max - weights[k], value + values[k], solution);

      if (value+ values[k] > *solution) 
         *solution= value + values[k];
    }
  }
}

В чем проблема?

Wyvern666 17 июнь 2014, в 05:05

Источник

2

Пытались ли вы использовать действительно минимальную проблему (т.е. меньшие values[] ) и пошагово проходить по коду, пока не увидели, что он не работает, а потом задумывались, почему? В качестве альтернативы, поместите несколько строк std::cout << ... в свой цикл?
Tony Delroy 17 июнь 2014, в 03:15

Теги:

c++

backtracking

knapsack-problem

1 ответ

Ещё вопросы

Пытались ли вы использовать действительно минимальную проблему (т.е. меньшие values[] ) и пошагово проходить по коду, пока не увидели, что он не работает, а потом задумывались, почему? В качестве альтернативы, поместите несколько строк std::cout << ... в свой цикл?

pipja · Accepted Answer · 2014-06-17T01-52-00.000Z

#include<iostream>
#include<vector>

using namespace std;

int weights[] = {2, 3, 1}, values[] = {6, 15, 7};

int solution = 0, n = 3;

std::vector<int> vsol;
std::vector<int> temp;

bool issol;


void Knapsack (int i, int max, int value)
{
  for (int k = i; k < n; k++) {
    if ( max > 0)
    {
        if (weights[k] <= max)
        {
          temp.push_back(k);
          if (value+ values[k] >= solution)
          {
            solution = value + values[k];
            issol = true;
          }
        }
        if ( (k+1) < n)
        {
          Knapsack (k+1, max - weights[k], value + values[k]);
        }
        else
        {
          if (issol == true)
          {
            if (! vsol.empty()) vsol.clear();
            std::move(temp.begin(), temp.end(), std::back_inserter(vsol));
            temp.clear();
            issol = false;
          } else temp.clear();
          return;
        }
    }
    else
    {
        if (issol == true)
        {
            if (! vsol.empty()) vsol.clear();
            std::move(temp.begin(), temp.end(), std::back_inserter(vsol));
            temp.clear();
            issol = false;
        } else temp.clear();
        return;
    }
  }
}

int main()
{
    Knapsack(0, 2, 0);
    cout << "solution: " << solution << endl;
    for(vector<int>::iterator it = vsol.begin(); it != vsol.end(); it++)
        cout << *it << " ";
    return 0;
}

Вам нужно увеличить k на 1, когда вы снова вызываете функцию Knapsack, чтобы переместить индекс вперед.

Добавлен код для распечатки индексов местоположения решения. Если существует более одного решения (то есть то же самое), будет распечатываться только место для последнего решения.

Было бы легко в этом случае сохранить элементы решения?
Конечно, вы можете использовать контейнер (массив или любой другой, предпочтительно контейнер STL) для хранения позиций индекса массива весов (и значений соответственно)
Правильное место для этого - перед рекурсивным вызовом?
Если вам не нужно делать рекурсию, то я предлагаю заменить рюкзак на комбинацию циклов, что значительно облегчит определение позиций индекса конечного результата.