#include<iostream> #include<string> #include<cstring> using namespace std; int main(){ int N,len[100]; string str[100]; string max_str; cin>>N; for(int i=0;i<N;i++){ cin>>str[i]; } for(int i=0;i<N;i++){ int len[i]=strlen(str[i].c_str()); } int max=len[0]; for(int i=0;i<N;i++){ if(max<len[i]){ max=len[i]; max_str=str[i].c_str(); } } cout<<"The longest is : "<<max_str<<endl; return 0; }

仍然有问题，问题在于第三个 for 循环中的 int len[i]，不应该定义一个新的变量，而应该直接将计算出来的长度赋值给 len[i] 数组。

修改后的代码如下：

#include<iostream>
#include<string>
#include<cstring>
using namespace std;

int main(){
    int N,len[100];
    string str[100];
    string max_str;
    cin>>N;
    for(int i=0;i<N;i++){
        cin>>str[i];
    }
    for(int i=0;i<N;i++){
        len[i]=strlen(str[i].c_str());
    }
    int max=len[0];
    for(int i=0;i<N;i++){
        if(max<len[i]){
            max=len[i];
            max_str=str[i];
        }
    }
    cout<<"The longest is : "<<max_str<<endl;
    return 0;
}

希望对你有帮助！

相关问题

优化这段代码#include <iostream> #include <vector> #include <string> using namespace std; int main() { string s; s.reserve(100); char c; while (cin >> c) { s.push_back(c); } cout << "s===" << s << endl; return 0; }

这段代码的主要问题在于使用了大量的string操作，如果输入的数据量很大，会导致效率较低。可以使用字符数组来代替string，同时减少vector的使用，代码如下：

#include <iostream>
#include <cstring>
using namespace std;

int main() {
    char s[100];
    int len = 0;
    char c;
    while (cin >> c) {
        s[len++] = c;
    }
    s[len] = '\0';
    cout << "s===" << s << endl;
    return 0;
}

这里使用了字符数组s来存储输入的字符，同时使用一个变量len来记录当前字符数组中已经存储的字符数。当输入一个字符时，将其存入字符数组s中，并将len加1。最后，将字符数组s的最后一个字符设置为'\0'，以便输出时正确显示字符串。这样可以避免使用string的大量操作，从而提高了效率。

#include <iostream> #include <iomanip> #include <cstring> #define MAX_LEN 100 using namespace std; int main() { char in = (char*)malloc(sizeof(char)1024); char out = (char*)malloc(sizeof(char)*1024); cin.getline(in, 1024); int count = 0; for(int i=0;i<strlen(in);i++) { if (in[i] != ' ') { } } cout<<out<<endl; return 0; }优化这段代码

这段代码存在一些问题，例如：

1. 使用了不安全的 malloc 函数来分配内存，容易造成内存泄漏和越界访问。

2. 没有对输入的字符串长度进行判断，可能会导致越界访问和程序崩溃。

3. 没有对输出字符串进行初始化，可能会产生不可预期的结果。

4. 循环中没有对计数器 count 进行更新，导致计数结果不正确。

下面是我对这段代码的优化建议：

1. 使用 C++ 中的 string 类来代替 char 数组，避免手动分配内存和越界访问。

2. 使用 getline 函数获取输入的一行字符串，并使用 length 函数获取字符串长度，避免越界访问。

3. 使用 stringstream 类来进行字符串分割和拼接，避免手动处理字符串。

4. 使用 for-each 循环来遍历字符串中的字符，避免使用下标访问。

5. 使用 vector 容器来存储单词，避免手动分配内存和越界访问。

下面是优化后的代码示例：

#include <iostream>
#include <iomanip>
#include <string>
#include <sstream>
#include <vector>
using namespace std;

int main() {
    string input, output;
    getline(cin, input);
    stringstream ss(input);
    vector<string> words;
    string word;
    while (ss >> word) {
        words.push_back(word);
    }
    int count = 0;
    for (auto& w : words) {
        if (!w.empty()) {
            count++;
            output += w;
            if (count < words.size()) {
                output += ' ';
            }
        }
    }
    cout << output << endl;
    return 0;
}

优化后的代码更加简洁、安全和易读，能够正确地统计单词数量并输出去除多余空格的字符串。