我的冒泡排序是在数组中引入值“ 0”

时间:2019-03-28 09:17:34

标签: c++ arrays multidimensional-array

我在C ++程序中使用了“冒泡排序”,但是它在分数贪婪程序中在数组中引入了随机的“ 0”值

int sorteaza()
{
    int aux,schimb,i;
    do
    {
        schimb=0;
        for (i=0;i<=n;++i)
            if (G[i][3]<G[i+1][3])
            {
            swap(G[i], G[i+1]);
            }
    }
    while (schimb);
}

这是我的完整代码:

#include<iostream>
using namespace std;

int n; // Numarul de elemente
float G[100][3]; // Obiecte + detalii masa profit potenta
int masa = 0;

int read_data()
{
cout << "Greutatea Rucsac" << endl;
cin >> masa;
cout << "Obiecte: " << endl;
cin >> n;
for(int i = 1; i<=n;i++)
{
    for(int j = 1; j<=2;j++)
    {
        cin >> G[i][j];
        if(G[i][1] != 0 && G[i][2] != 0)
        {
         G[i][3] = G[i][2] / G[i][1];
        }

    }
}
}
// 2 500
// 4 500

int sorteaza()
{
    int aux,schimb,i;
    do
    {
        schimb=0;
        for (i=0;i<=n;++i)
            if (G[i][3]<G[i+1][3])
            {
            swap(G[i], G[i+1]);
            }
    }
    while (schimb);
}
int verify()
{
for(int i = 1; i<=n;i++)
{
    for(int j = 1; j<=3;j++)
    {
        cout << G[i][j];
        cout << endl;
        //G[i][3] = G[i][1] / G[i][2];
    }
}
}

int greedy()
{

    float profit = 0;
    int i = 1;
    int aux;
    while(i<=n && masa>=0)
    {
        //cout << "G[i][1]: " << G[i][1] << endl;
        if(masa>=G[i][1]) {
        //cout << "Am ajuns aici";
        profit=profit+G[i][2];
        masa=masa-G[i][1];
        }
        else {
                        //cout << "Am ajuns dincolo";
                aux= (masa*100)/G[i][1];
                profit = profit + (aux * G[i][2])/100;
                break;
                        }
    i++;
   }
cout << profit;
    }










int main()
{
    read_data();
    sorteaza();
    verify();
   // greedy();

}

2 个答案:

答案 0 :(得分:2)

您可能需要<n而不是≤n(这是未初始化的值,即0的来源)。您会错过气泡排序中的一个循环。现在,您只将最小的元素冒泡到列表的末尾。

也不知道您在使用schimb和条件时正在做什么。

此外,您将G定义为float[100][3],因此您不能仅使用G[i][3]来使用G[i][2]

int sorteaza()
{
    int i,j;
    for (i=0; i<n; i++)
    {
        for (j=i+1; j<n; j++)
        {
            if (G[i][2] < G[j][2])
            {
                swap(G[i], G[j]);
            }
        }
    }
}

答案 1 :(得分:2)

学习从零开始索引所有数组。

float G[100][3];

法律索引为0到99和0到2。因此此代码应为

for (int i = 0; i < n; i++)
{
    for (int j = 0; j < 2; j++)
    {
        cin >> G[i][j];
    }
    if (G[i][0] != 0 && G[i][1] != 0)
    {
        G[i][2] = G[i][1] / G[i][0];
    }
}

,此代码应为

        if (G[i][2] < G[i+1][2])
        {
            swap(G[i], G[i+1]);
        }

所有数组都从零开始。我相信您已经被告知了这一点,但是您必须开始将其付诸实践。

通常,这样编写for循环

for (int i = 0; i < N; ++i)

这是大小为N的数组的正确循环。