0

我不久前拿起了这段代码，作为从文本文件中选择随机行并输出结果的一种方式。不幸的是，它似乎只输出它选择的行的第一个字母，我无法弄清楚它为什么这样做或如何修复它。任何帮助，将不胜感激。

#include "stdafx.h"
#include <stdio.h>
#include <iostream>
#include <fstream>
#include <string>
#include <time.h>
using namespace std;

#define MAX_STRING_SIZE 1000

string firstName()
{
    string firstName;
    char str[MAX_STRING_SIZE], pick[MAX_STRING_SIZE];
    FILE *fp;
    int readCount = 0;

    fp = fopen("firstnames.txt", "r");
    if (fp)
    {
        if (fgets(pick, MAX_STRING_SIZE, fp) != NULL)
        {
            readCount = 1;
            while (fgets (str, MAX_STRING_SIZE, fp) != NULL)
            {
                if ((rand() % ++readCount) == 0)
                {
                    strcpy(pick, str);
                }
            }
        }
    }
    fclose(fp);
    firstName = *pick;
    return firstName;
}

int main() 
{
    srand(time(NULL));

    int n = 1;
    while (n < 10)
    {
        string fn = firstName();
        cout << fn << endl;
        ++n;
    }

    system("pause");
}
4

2 回答 2

2 
 firstName = *pick;

我猜这是问题所在。

pick这里本质上是一个指向数组第一个元素的指针char*，所以当然*pick是char.. 类型或者数组的第一个字符。

另一种看待它的方式是*pick == *(pick +0) == pick[0]

有几种方法可以修复它。最简单的就是执行以下操作。

return pick;

构造函数会自动为您进行转换。

于 2013-01-27T14:47:51.103 回答
0

由于您没有指定文件的格式，我将介绍两种情况：固定记录长度和可变记录长度；假设每个文本行都是一条记录。

读取随机名称、固定长度记录

这个是直截了当的。

  1. 确定所需记录的索引（随机）。
  2. 计算文件位置=记录长度*索引。
  3. 将文件设置到该位置。
  4. 从文件中读取文本，使用std::getline.

读取随机名称、可变长度记录

这假设文本行的长度不同。由于它们各不相同，因此您不能使用数学来确定文件位置。

要从文件中随机选择一行，您要么必须将每一行放入容器中，要么将行开头的文件偏移量放入容器中。

建立容器后，确定随机名称编号并将其用作容器的索引。如果您存储了文件偏移量，请将文件定位到偏移量并读取该行。否则，从容器中拉出文本。

应该使用哪个容器？这取决于。存储文本速度更快，但会占用内存（您实际上是将文件存储到内存中）。存储文件位置占用的空间更少，但您最终会读取每行两次（一次查找位置，第二次获取数据）。

对这些算法的增强是对文件进行内存映射，这对读者来说是一个练习。

编辑 1：示例

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

using std::string;
using std::vector;
using std::fstream;

// Create a container for the file positions.
std::vector< std::streampos > file_positions;

// Create a container for the text lines
std::vector< std::string > text_lines;

// Load both containers.
// The number of lines is the size of either vector.
void
Load_Containers(std::ifstream& inp)
{
  std::string    text_line;
  std::streampos file_pos;
  file_pos = inp.tellg();
  while (!std::getline(inp, text_line)
  {
    file_positions.push_back(file_pos);
    file_pos = inp.tellg();
    text_lines.push_back(text_line);
  }
}
于 2013-01-27T17:39:20.223 回答