如何修复输出显示,以便正确显示已排序的数据

问题描述 投票:0回答:2

当我调用我的打印功能时,我的程序在通过冒泡排序运行后显示未排序的数据。我将我的代码与朋友进行了比较,并在geeksforgeeks(https://www.geeksforgeeks.org/bubble-sort/)上发现了一篇关于冒泡排序的文章,无论出于什么原因我的数据都不会排序。

我正在为我的大学数据结构类编写一个带有4个排序函数的程序。理论上应该发生的是程序读入具有8000个随机数的数据文件,并根据调用的排序(冒泡排序,插入排序,选择排序和快速排序)对数据进行排序。调用print函数并显示每1000个数据元素以显示数据确实已排序。我已经尝试在main中编码我的打印,将打印调用放在排序功能中,并使用函数进行打印,Quadruple检查了我的排序功能。我也尝试过不同的编译器和不同的计算机,但我怀疑它的代码很时髦。

原始代码:

#include <iostream>
#include <fstream>
#include <string>
#include <cmath>
#include <iomanip>
using namespace std;
const int arraysize = 8000;
int numcompares, numcopies;
void swapem(double a, double b)
{
    double temp;
    temp = a;
    a = b;
    b = temp;
}
void bubblesort(double r[], int n)
{
    int j, i;
    for (j = 0; j < n - 1; j++)
    {
        for (i = 0; i < n - 1; i++)
        {
            numcompares++;
            if (r[i] > r[i + 1])
            {
                swapem(r[i], r[i + 1]);
                numcopies += 3;
            }
        }
    }
}
void printem(double r[])
{
    cout <<  r[1000] << ", " << r[2000] << ", " << r[3000] << ", " <<
    r[4000] << ", " << r[5000] << ", " << r[6000] << ", " << r[7000] << ", " << r[7999] << endl;
}
int main()
{
    ifstream inf("data.dat");
    ofstream outf("sorted.ot");
    string sortname;
    double arraynums[arraysize];
    for (int i = 0; i < arraysize; i++)
    {
        inf >> arraynums[i];
    }
    bubblesort(arraynums, arraysize);
        system("pause");
    return 0;
}

我试过了:

void printem(double r[], int n)
{
    int i;
    for (i = 0; i < n; i++)
        cout << r[i]<< ", ";    
}
printem(arraynums,20); // in main

在主要的

for (int i = 0; i < 10; i++)
    {
        cout << arraynums[i] << endl;
    } 


这是拒绝排序的8000个随机整数中的10个

41275.4
12113.1
50676
7662.34
50688.3
-7926.28
13672.8
-3212.9
-13046.5
-16798

输出应为:-16798 -13046.5 -7926.28 -3212.9 7662.34 12113.1 13672.8 41275.4 50676 50688.3

然而它保持未分类形式:41275.4 12113.1 50676 7662.34 50688.3 -7926.28 13672.8 -3212.9 -13046.5 -16798

c++ algorithm sorting
2个回答
1
投票

欢迎来到Stack Overflow!看起来你在这里有一个经典的传递值与传递参考的情况。 C中的函数不能在其自己的作用域之外修改其参数的值,但它可以访问其参数引用的内存并对其进行修改。

swapem()仅对方法本身范围内的a和b进行更改,但不更新a和b位置引用的数据。要解决此问题,您可以将a和b in作为指针传递,并修改指针引用的值,如下所示:

void swapem(double *a, double *b) {
     double temp = *a;
     *a = *b;
     *b = temp;
}

并且方法本身将采用那些数组元素的引用而不是它们的值。对于上面的示例,它看起来像这样:

swapem(r + i, r + (i + 1))

有一组很好的相关信息here按值传递并通过引用传递。有一点技术性,你实际上是将指针的值传递给上面的例子,这些例子在技术上使它通过值传递,但它是你想要修改的实际变量的引用,所以这个概念仍然存在在这里坚持。


0
投票

了解你的bubblesort()函数的作用非常重要。它收到阵列成员'arraynums'的COPY。它对它进行排序然后终止。在此函数的末尾,您没有这个已排序的数组。

但是等等,即使你的swapem()也行不通,因为它也会收到2个值的副本并在本地进行交换。此调用后您将不会有交换的值。

要解决这个问题,您需要在swapem()函数中使用pass-by-reference。

void swapem(double & a, double & b)
{
    double temp;
    temp = a;
    a = b;
    b = temp;
}

在bubbelsort()函数中,如果要使用bubblesort进行打印,可以立即使用printem()函数。相反,你可以简单地将数组作为对bubblesort()的引用传递,并在它的末尾你将拥有修改后的arraynums数组。

© www.soinside.com 2019 - 2024. All rights reserved.