3

私は「宝石強盗」のための貪欲なアルゴリズム (Python 3.xx) を書いています。一連の宝石と値が与えられると、プログラムは、バッグの重量制限を超えずにバッグに収まる最も価値のある宝石を取得します。ここには 3 つのテスト ケースがあり、そのうちの 2 つで完全に機能します。

各テスト ケースは同じ方法で記述されます。最初の行はバッグの重量制限であり、その後のすべての行はタプル (重量、値) です。

サンプルケース 1 (動作):

10
3 4
2 3
1 1

サンプル ケース 2 (機能しない):

575
125 3000
50 100
500 6000
25 30

コード：

def take_input(infile):
    f_open = open(infile, 'r')
    lines = []
    for line in f_open:
        lines.append(line.strip())
    f_open.close()
    return lines

def set_weight(weight):
    bag_weight = weight
    return bag_weight

def jewel_list(lines):
    jewels = []
    for item in lines:
        jewels.append(item.split())
    jewels = sorted(jewels, reverse= True)
    jewel_dict = {}
    for item in jewels:
        jewel_dict[item[1]] = item[0]
    return jewel_dict

def greedy_grab(weight_max, jewels):
    #first, we get a list of values
    values = []
    weights = []
    for keys in jewels:
        weights.append(jewels[keys])
    for item in jewels.keys():
        values.append(item)
    values = sorted(values, reverse= True)
    #then, we start working
    max = int(weight_max)
    running = 0
    i = 0
    grabbed_list = []
    string = ''
    total_haul = 0
    # pick the most valuable item first. Pick as many of them as you can.            
    # Then, the next, all the way through.
    while running < max:
        next_add = int(jewels[values[i]])
        if (running + next_add) > max:
            i += 1
        else:
            running += next_add
            grabbed_list.append(values[i])
    for item in grabbed_list:
        total_haul += int(item)
    string = "The greedy approach would steal $" + str(total_haul) + " of  
             jewels."
    return string

infile = "JT_test2.txt"
lines = take_input(infile)
#set the bag weight with the first line from the input
bag_max = set_weight(lines[0])
#once we set bag weight, we don't need it anymore
lines.pop(0)

#generate a list of jewels in a dictionary by weight, value
value_list = jewel_list(lines)
#run the greedy approach
print(greedy_grab(bag_max, value_list))

ケース2でうまくいかない理由を誰かが知っていますか? よろしくお願いいたします。 編集: ケース 2 の予想される結果は $6130 です。私は6090ドルを手に入れたようです。

4

3 に答える 3

3

辞書のキーは整数ではなく文字列であるため、並べ替えようとすると文字列のように並べ替えられます。したがって、次のようになります。

['6000', '3000', '30', '100']

代わりに欲しかった：

['6000', '3000', '100', '30']

この関数を次のように変更し、整数キーを持つようにします。

def jewel_list(lines):
    jewels = []
    for item in lines:
        jewels.append(item.split())
    jewels = sorted(jewels, reverse= True)
    jewel_dict = {}
    for item in jewels:
        jewel_dict[int(item[1])] = item[0]  # changed line
    return jewel_dict

これを変更すると、次のようになります。

The greedy approach would steal $6130 of jewels.
于 2013-10-24T07:10:33.057 に答える
1 
In [237]: %paste
def greedy(infilepath):
  with open(infilepath) as infile:
    capacity = int(infile.readline().strip())
    items = [map(int, line.strip().split()) for line in infile]

  bag = []
  items.sort(key=operator.itemgetter(0))
  while capacity and items:
    if items[-1][0] <= capacity:
      bag.append(items[-1])
      capacity -= items[-1][0]
    items.pop()
  return bag

## -- End pasted text --

In [238]: sum(map(operator.itemgetter(1), greedy("JT_test1.txt")))
Out[238]: 8

In [239]: sum(map(operator.itemgetter(1), greedy("JT_test2.txt")))
Out[239]: 6130
于 2013-10-24T07:25:00.557 に答える