おそらくやりたいのは以下のようなことだと思われます。
元の処理ではcancer_id
をfor
ループで回すという処理が間違っているのではないでしょうか?
data = [0.0049, 0.2351, 0.8173, 0.9115, 0.8093, 0.1836, 0.2198, 0.9955, 0.3846, 0.1468, 0.9478, 0.92, 0.9127, 0.3558, 0.8828, 0.9998, 0.5782, 0.5649, 0.4276, 0.1114, 0.6143, 0.7477, 0.4198, 0.2642, 0.0728]
cancer_id = [3,5,8,11,12,15,16,21]
tp = [] # 真陽性
fp = [] # 偽陽性
tn = [] # 真陰性
fn = [] # 偽陰性
theta = 0.8 # 閾値
#### enumerate()でインデックス値とデータを1回で取得。countの処理を不要とする
for idx,value in enumerate(data):
#### id_numberとcountの比較ではなくインデックス値が陽性患者リストに含まれるかを判定
if idx in cancer_id:
if value >= theta:
print("patient id {} is true positive".format(idx))
tp.append(idx)
else:
print("patient id {} is false negative".format(idx))
tn.append(idx)
else:
if value >= theta:
print("patient id {} is false positive".format(idx))
fp.append(idx)
else:
print("patient id {} is true negative".format(idx))
fn.append(idx)
print(len(tp)) # 7 -> 4
print(len(fp)) # 65 -> 5
print(len(tn)) # 1 -> 4
print(len(fn)) # 127 -> 12