Question

大家好我正在处理一个在文件中找到最长字和行的分配，我正在处理它，但是当我收到这个错误时就卡住了。

错误：有符号和无符号整数表达式之间的比较[-Werror = sign-compare]

现在我明白了这个错误，但我很难解决它，我已经尝试在我自己的第一个问题上修复它但是被迫求助于此。

#include <iostream>
#include<fstream>

using namespace std;

int main(int argc, char *argv[]) {

   int longest = 0;
   string line;

    if(argc == 0)
    {
       cout << " " << endl;
    }
   else
   {

   for(int i = 1;i<argc;i++){
      ifstream file (argv[i]); //declare in the for loop
      if (!file.is_open() ){
      cout << argv[i] << " FILE NOT FOUND\n"; // watch out for /n
      }
      else{
         while (getline(file,line))
         {
            if(line.size() > longest){
               longest = line.size();
               cout << "The length of the longest Line is: " << longest << endl;
         }
         }


}
   }
   }
   }

我知道错误在这里：

if(line.size() > longest){

编辑：解决方案：这里的修复是使用size_t而不是int来声明最长的，我在尝试修复它时尝试做的是在if语句中包含size_t，这对我来说是完全错误的。

错误：int longest = 0; 右：size_t最长= 0; 这仅适用于这类情况。

Answer 1

如上所述，您应该longest无符号。

此外，支票argc==0应为argc<2（因为argv[0]是程序名称。）

最后，重印最长线＆＃34;似乎很奇怪。声明任何时候一条线长于当前最大值

Bonus：没有必要将数据读入字符串，这会导致内存分配成本。你可以计算换行符之间的字节。

这是我简化的内容：

<强> Live On Coliru

#include <vector>
#include <iostream>
#include <fstream>
#include <iterator>

namespace {
    struct accumulator {
        size_t peak = 0;
        void operator()(size_t n) { peak = std::max(n, peak); }
    };
}

int main(int argc, char **argv) {
    accumulator sample;

    for(auto fname : std::vector<std::string>(argv+1, argv+argc)) {
        std::ifstream ifs(fname, std::ios::binary);
        if (!ifs) {
            std::cout << fname << " FILE NOT FOUND\n";
            continue;
        }

        std::istreambuf_iterator<char> f(ifs), l;

        auto scan_until = [&](char delimiter) {
            size_t count = 0;
            while (f!=l && *f++ != delimiter) ++count;
            return count;
        };

        while (f!=l) 
            sample(scan_until('\n'));
    }

    std::cout << "The length of the longest line is: " << sample.peak << "\n";
}

在自己的源上运行时：

The length of the longest line is: 78

加成

简单更改以打印每个文件的最长行：

<强> Live On Coliru

/Archive2/8b/001ad4e031178d/main.cpp 44
/Archive2/8b/00964b446531eb/main.cpp 37
/Archive2/8b/01880ea6d95d38/main.cpp 50
/Archive2/8b/029f129c393ce5/main.cpp 63
... 390 lines snipped...
/Archive2/8b/f94c53ac4aee45/main.cpp 93
/Archive2/8b/f9ab4b38599e1a/main.cpp 80
/Archive2/8b/fabac3a5309ea7/main.cpp 72
/Archive2/8b/fb63a91a5b2b29/main.cpp 33
/Archive2/8b/fba5d4cf806d91/main.cpp 95
/Archive2/8b/fc2fbcfddff198/main.cpp 42
/Archive2/8b/fc4bbeb11aa3e9/main.cpp 78
/Archive2/8b/fc56c06f6ed2a1/main.cpp 37
/Archive2/8b/fd5770ca49eb30/main.cpp 78
/Archive2/8b/fe0b1121edfb0a/main.cpp 59
/Archive2/8b/ff0f709404e6a1/main.cpp 84
Overall longest line: 1353

查找文件中的最长行，有符号和无符号整数表达式之间的比较

1 个答案:

加成