おそらくやりたいのは以下のようなことだと思われます。
元の処理では`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 positive".format(idx))
            fp.append(idx)
    else:
        if value >= theta:
            print("patient id {} is false negative".format(idx))
            fn.append(idx)
        else:
            print("patient id {} is true negative".format(idx))
            tn.append(idx)

print(len(tp))  # 7   ->  4
print(len(fp))  # 65  ->  5
print(len(tn))  # 1   ->  4
print(len(fn))  # 127 -> 12
```