在int temp = a[i].pw_ratio;中，您将比率存储在int变量中，有效地截断了它。当您稍后执行a[j].pw_ratio = temp;时，原始值不会恢复。
整个交换过程可以通过一次交换整个struct item来简化，这样就可以在任何地方使用正确的类型。

if (a[i].pw_ratio > a[j].pw_ratio) {
    struct item temp = a[i];
    a[i] = a[j];
    a[j] = temp;

    // swap back index
    int idx = a[i].index;
    a[i].index = a[j].index;
    a[j].index = idx;
}

字符串

您的排序函数使用temp（一个int变量）在交换时临时存储值。这会导致某些值被截断为int。你也不交换index，尽管不清楚这是错误还是故意的。
注意，可以使用标准库函数qsort对数组进行排序。除了更正确和更少的代码之外，当你的数组变长时，它会更有效。

sort函数使用一个int变量来执行值交换，导致将double值转换为int，这有效地截断了这些值。在排序之后，您只需交换结构并重新构造index成员。
还请注意，您的排序方法不起作用：你应该只比较索引值为i和j的元素，使得i < j。
以下是修改后的版本：

void sort(struct item a[], int n) {
    for (int i = 0; i < n; i++) {
        for (int j = i + 1; j < n; j++) {
            if (a[i].pw_ratio > a[j].pw_ratio) {
                struct item temp = a[i];
                a[i] = a[j];
                a[j] = temp;
            }
        }
    }

    for (int i = 0; i < n; i++) {
        a[i].index = i + 1;
    }
    
    printf("Sorted items: \n");
    for (int i = 0; i < n; i++) {
        printf("%d\t%f\t%f\t%f\n", a[i].index, a[i].pw_ratio,
               a[i].profit, a[i].weight);
    }
}

字符串
您也可以使用<stdlib.h>中定义的qsort：

#include <stdlib.h>

int compare_ratios(const void *a, const void *b) {
    const struct item *aa = a;
    const struct item *bb = b;
    int cmp = (aa->pw_ratio > bb->pw_ratio) - (aa->pw_ratio < bb->pw_ratio);
    // implement stable sorting:
    //     keep elements that compare equal in the original order.
    if (cmp == 0) {
        cmp = (aa->index > bb->index) - (aa->index < bb->index);
    }
    return cmp;
}

void sort(struct item a[], int n) {
    qsort(a, n, sizeof(*a), compare_ratios);

    for (int i = 0; i < n; i++) {
        a[i].index = i + 1;
    }
    
    printf("Sorted items: \n");
    for (int i = 0; i < n; i++) {
        printf("%d\t%f\t%f\t%f\n", a[i].index, a[i].pw_ratio,
               a[i].profit, a[i].weight);
    }
}

型