Неожиданный результат рекурсивной функции задачи Knapsack

Мне нужно написать простую программу для решения задачи о рюкзаке. Я написал этот код на данный момент, и я думаю, что это все, что мне нужно, но каждый раз, когда я его выполняю, я получаю разные результаты. Все, что я могу думать, это то, что есть проблема с освобождением памяти, но я не знаю, как ее решить. Есть идеи?

P.S. Может быть, это глупый вопрос, но я никогда не работал в C.

#include <stdio.h>

int max(int a, int b){
  if (a > b) {
    return a;
  } else {
    return b;
  }
}

int knapsack(int prices[], int weight[], int n, int max_weight){
  if (n < 0)
    return 0;
  if (weight[n] > max_weight)
    return knapsack(prices, weight, n-1, max_weight);
  else
    return max(knapsack(prices, weight, n-1, max_weight), (knapsack(prices, weight, n-1, max_weight - weight[n]) + prices[n]));
}

int main(int argc, char const *argv[]) {
  int i, weight[] = {2,3,3,4}, prices[] = {1,5,2,9}, max_weight = 7, n, result;
  for (i=0; i<argc; i++) {
    printf("%d: \"%s\"\n", i, argv[i]);
  }
  n = (sizeof(weight))/(sizeof(weight[0]));
  result = knapsack(prices, weight, n, max_weight);
  printf("%d\n", result);
  return 0;
}

РЕЗУЛЬТАТЫНеожиданный результат рекурсивной функции задачи Knapsack

weight[n] не определено
Ôrel 10.04.2019 16:43

@Ôrel О да, конечно... Мне нужно еще кофе. Спасибо!

Federico Galfione 10.04.2019 16:51
Стоит ли изучать PHP в 2023-2024 годах?
Стоит ли изучать PHP в 2023-2024 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
0
2
40
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

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

n = (sizeof(weight))/(sizeof(weight[0]));

Вы не можете индексировать вес в n, потому что у него есть только индексы от 0 до n-1

Попробуйте вызвать

result = knapsack(prices, weight, n-1, max_weight);

Да, это была проблема, это было прямо передо мной, я такой глупый! @Orel показал мне ошибку в разделе комментариев

Federico Galfione 10.04.2019 16:58

Другие вопросы по теме