Моя сортировка пузырьков представляет значения «0» в массиве

Question

Я использовал «сортировку по пузырям» для моей программы на C ++, но она вводит случайные значения «0» в массиве в программе Fractional Greedy

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();

}

Answer 1

Научитесь индексировать все ваши массивы с нуля.

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.

Answer 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]);
            }
        }
    }
}