Question

下面的计数排序根据元素的ASCII值对其进行排序。下面的代码工作正常，但我想进行一些I / O修改。该代码不接受自定义输入。

我尝试进行一些更改，但行为未定义。我的第一个疑问是为什么我会得到不确定的行为。其次，请提供给我代码，使下面的代码按预期运行。评论部分是我自己尝试的。我希望它接受用户的输入。

#include<bits/stdc++.h>
#include<string.h>

using namespace std;

#define RANGE 255

void countSort(char arr[])      //void countSort(char arr[],int n)
{
    char output[strlen(arr)];   //char output[n];

    int count[RANGE + 1], i;
    memset(count, 0, sizeof(count));

    for(i = 0; arr[i]; i++) {
        count[arr[i]]++;
    }

    for (i = 1; i <= RANGE; ++i) {
        count[i] += count[i-1];
    }

    for (i = 0; arr[i]; ++i) {
        output[count[arr[i]]-1] = arr[i];
        --count[arr[i]];
    }

    for (i = 0; arr[i]; ++i) {
        arr[i] = output[i];
    }
}

// Driver code
int main()
{
    char arr[] = "geeksforgeeks";

    countSort(arr);

    cout<< "Sorted character array is "<<arr;

/*
    int n;
    cin>>n;
    char arr[n];

    for(int i=0;i<n;i++) {
        cin>>arr[i];
    }
    countSort(arr,n);

    for(int i=0;i<n;i++) {
        cout<<endl<<arr[i];
    }
*/
    return 0;
}

Answer 1

OP询问如何从用户那里获取输入信息并进行排序。而不是给定char数组中的预定义字符串。

我会给出答案。但是问题是用C ++标记的，我将其转换为C ++。

顺便说一句。问题中的代码是来自GeeksforGeeks的一对一副本，并尝试对称为here的C ++中所谓的计数排序算法进行编码。

由于该代码取自GeeksforGeeks，因此，我非常遗憾地需要将用户“ rathbhupendra”归咎于真正糟糕的C ++代码。真的很抱歉

代码正在使用：

C样式数组
可变长度数组（编译器扩展。不符合C ++）
strlen
内存集
#include<bits/stdc++.h> and #include<string.h>
使用命名空间标准

for(i = 0; arr[i]; ++i)

异常终止条件
字符数组而不是std::string
用于定义数组大小（#define RANGE 255）的宏

因此，没有C ++。

现在，答案。

您需要使用功能std::string从用户读取类型为std::getline的变量中的字符串。

std :: string可以像字符数组一样使用。没什么。

请参阅C ++解决方案：

编辑

根据MichaelDorgan的评论进行编辑

#include <iostream>
#include <string>
#include <vector>

constexpr size_t AsciiRange = 256;

// Convert signed char to unsigned size_t type.
inline size_t char2sizet(char c) { return static_cast<size_t>(static_cast<unsigned char>(c)); }

void countSort(std::string& stringToSort)      
{
    std::vector<size_t> count(AsciiRange, 0U);

    size_t i { 0U };
    for (i = 0U; i < stringToSort.size(); i++) {
        count[char2sizet(stringToSort[i])]++;
    }

    for (i = 1U; i < AsciiRange; ++i) {
        count[i] += count[i - 1U];
    }

    std::string output(stringToSort);  
    for (i = 0U; i < stringToSort.size(); ++i) {
        output[count[char2sizet(stringToSort[i])] - 1U] = stringToSort[i];
        --count[char2sizet(stringToSort[i])];
    }

    stringToSort = output;
}

int main()
{
    std::cout << "\nPlease enter a string:\n\n";

    // Get the string from the user
    std::string inputString{};
    getline(std::cin, inputString);

    // Sort it by characters
    countSort(inputString);

    // Show result
    std::cout << "\n\n\nString sorted by characters is:\n\n" << inputString << '\n';

    return 0;
}

希望这会有所帮助。。

Answer 2

我对“获取未定义的行为”感到不安，是指分段错误（有时是发生）。问题出在这一行

for(i = 0; arr[i]; i++)

您应该写

for(i = 0; i < n; i++)

您可以检查在每个循环arr[i]的第一种情况下有时是一些奇怪的字符（此字符不属于输入字符串）和{{1 }}，此char返回负数，在此处产生分段错误

count[arr[i]]

修改计数排序

2 个答案: