指针和std :: string-奇怪的行为-C ++

坡瓦国王

由于在上一篇文章中曾问过同样的问题,所以我事先表示歉意,但是正如有人正确指出的那样,我没有发布真实的代码。因此,我再次提出相同的问题,试图比以前更清楚。

作为练习,我正在创建一个操作字符串的程序。特别是,我想删除2'*'之间的部分字符串。我必须强调,我已经成功地使用库字符串的功能创建了相同的程序;实际上,问题在于使用char指针操纵给定的字符串。我将发布完整代码并进行深入讨论。

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

int main() {

    string frase;
    getline (cin, frase); // Takes as input the phrase
    int size = frase.size();

    cout << frase[0]; // <- this line is not even processed (I've used it to test the problem) However, if I put it before the first if, it will be sent in output.

    char* pa1 = NULL; // The pointer which will "point" to the first *
    char* pa2 = NULL; // The pointer which will "point" to the second *
    bool stop = false; // When the pointers find 2 asterisk, stop = true
    for(int i = 0; i < size - 1 || stop == true; i++){ // FOR LOOP n.1
        if(frase[i] == '*'){
            if(*pa1 == '*'){
                pa2 = &frase[i];
                stop = true;
            }
            pa1 = &frase[i];
        }
    }

 // I've debugged the program and find that the problem is before this line, probably
 // linked to the address of pointers. I will explain later what I mean.
 // I've came up with this conclusion after trying to ignore part of the program and processing it in another file.
 // However, I'm not fully sure with this result, since the problem regards the visualization of the content of the pointers.

    if(pa2 == NULL){ // if it's a null pointer, this means  that second asterisk has not been found.
        if(pa1 == NULL){// if also this is a null pointer, there is no asterisk at all
            cout << "Non ci sono asterischi. Non verrà eliminata nessuna parola.\n\n";
        }
        cout << "C'è un solo asterisco. Verrà eliminato unicamente l'asterisco.\n\n";
        for(; pa1 < &frase[size - 1]; pa1++){ // FOR LOOP n.2
            *pa1 = *(pa1 + 1);
        }
    }

    else{
        for(; pa1 < pa2 + 1; pa1++){ // this removes asterisk and 
        //the phrase between them, by overwriting the existing characters. FOR LOOP n.3
            *pa1 = *(pa1 + 1);

        }
    }

cout << "La frase dopo l'eliminazione è:\n" << frase;
return 0;
}

发布之前,我做了一些努力来了解问题的本质。我看到了意外的行为:如果我初始化指向内存地址的指针,例如:

pa1 = &frase[i];

其中包含任何星号,然后(由于for循环n.1中的'if'条件)将其地址更改为第一个星号后,我尝试通过编写使其可视化(忽略代码的其余部分) :

cout << *pa1;

该程序不会崩溃并输出星号。但是,对pa2进行相同操作并创建带有2星号的短语会导致程序崩溃。将指针pa1初始化为'NULL'并执行相同的过程将导致程序崩溃。

然后,我提出了2个假设

1 -也许我不能管理与字符指针一个String对象,虽然我只是管理角色的指定字符串。但是,如果我将指针初始化为字符串的现有地址,则可以轻松显示该短语的字符。

2 -问题是与我也在处理空字符的事实,如“空间”和一致好评。因此,问题出在for循环n.2或n.3(请参阅代码)中。

我知道我可以用char []数组处理问题,并且我知道用字符串函数处理问题会更好,但是我想解决这个问题以便充分理解字符串对象和char指针之间的相关性。因此,我只是在寻求帮助来查找此代码中的错误;我不需要新的代码(因为这对您来说是浪费时间,而且,即使我们正在谈论单独的练习,这也意味着以某种方式利用您)。我希望我能清楚地解释这个问题。先感谢您。

编辑:我忘了指出,我也认为问题可能与大小(以字节为单位的int值)有关,而我将其视为包含字符的字符串中的插槽数。我认为这些信息会很有用,但是我不确定它的可访问性。

编辑2:@lilscent解决了与引用空指针有关的问题。我更改了代码,并将pa1指针和pa2指针初始化为

pa1 = &frase[0];
pa2 = nullptr;

编辑3:按照建议,我删除了布尔变量,并在第一个循环中使用break代替。我还更改了最后一个for循环,因为该代码现在可以运行,但是无法执行应做的事情。我还编辑了第二个循环,添加了else:

if(pa2 == nullptr){

        if(pa1 == &frase[0]){
            cout << "Non ci sono asterischi. Non verrà eliminata nessuna parola.\n\n";
        }
        else{
            cout << "C'è un solo asterisco. Verrà eliminato unicamente l'asterisco.\n\n";
            for(; pa1 < &frase[size - 1]; pa1++){
                *pa1 = *(pa1 + 1);
            }
            *pa1 = ' ';
        }
    }

编辑4:现在该程序可以完全正常工作。我已经编辑了最后一个循环:

else{
        *pa2 = ' ';
        pa2+= 2;
        for(; pa1 < pa2 + 1 && pa2 < &frase[size]; pa1++, pa2++){
            *pa1 = *pa2;
            *pa2 = ' ';
        }
        *pa2 = ' ';
    }

感谢您的帮助和所有建议!我已经保留了代码,以帮助他人解决相同类型的问题。

最终编辑:请参阅NikosC。的帖子。他修改了程序的一部分并提高了效率,解决了大多数问题。再次感谢!

Nikos C.

循环中的逻辑不起作用。循环条件说:

for (int i = 0; i < size - 1 || stop == true; i++)

因此,此操作将持续到i < size - 1stop == true但是,您想要的是在时停止循环stop == true,而不是继续运行。因此,您需要:

for (int i = 0; i < size && !stop; i++)

请注意i < size,不是i < size - 1std::string::size()不包含终止\0符。

在循环内部,您具有:

if (frase[i] == '*') {
    if (*pa1 == '*') {
        pa2 = &frase[i];
        stop = true;
    }
    pa1 = &frase[i];
}

如果*找到a,则表示您正在检查是否pa1指向星号。但是,由于pa1初始化为null ,这将导致空指针取消引用相反,您应该做的只是测试是否pa1仍然为空指针。如果是这样,则意味着您还没有找到第一个*因此改为:

if (pa1 == nullptr) {
    // Since pa1 is still null, this is the first '*' we encountered.
    pa1 = &frase[i];
} else  {
    // pa1 was not null, so this means we just found the second '*'.
    pa2 = &frase[i];
    stop = true;
}

这种新逻辑允许您以不再需要的方式重写循环条件stop您可以简单地检查是否pa2不为null。如果仍然为空,则循环可以继续运行。

因此总体而言:

char* pa1 = nullptr; // The pointer which will "point" to the first *
char* pa2 = nullptr; // The pointer which will "point" to the second *
for (int i = 0; i < size && pa2 == nullptr; i++) {
    if (frase[i] == '*') {
        if (pa1 == nullptr)
            pa1 = &frase[i];
        else
            pa2 = &frase[i];
    }
}

(此外,更喜欢使用nullptr而不是NULL。它将防止使用时会被隐藏的某些类型的错误NULL。)

但是,您可以使用基于范围的for循环来进一步简化上述操作,这是对容器的所有元素进行迭代的推荐方式。您需要为迭代使用一个引用(auto&而不是auto),因为我们需要获取实际元素的地址,而不是元素副本的地址:

for (auto& i : frase) {
    if (i == '*') {
        if (pa1 == nullptr) {
            pa1 = &i;
        } else {
            pa2 = &i;
            break; // stop the loop since we found the second *
        }
    }
}

接下来,您将拥有尝试打印结果的代码:

if (pa2 == NULL) {
    if (pa1 == NULL) {
        cout << "Non ci sono asterischi. Non verrà eliminata nessuna parola.\n\n";
    }
    cout << "C'è un solo asterisco. Verrà eliminato unicamente l'asterisco.\n\n";
    for ( ; pa1 < &frase[size - 1]; pa1++) {
        *pa1 = *(pa1 + 1);
    }
}

这是行不通的,因为您试图取消引用,pa1即使它可能为null。在我看来,您想要做的只是简单地给出一条错误消息,指出没有星号,或者如果只有一个,则删除该星号:

if (pa1 == nullptr) {
    cout << "C'è un solo asterisco. Verrà eliminato unicamente l'asterisco.\n";
    return 0;
}

if (pa2 == nullptr) {
    cout << "Non ci sono asterischi. Non verrà eliminata nessuna parola.\n";
    pa2 = pa1;
}

对于最后一部分,要除去*text*字符串的这一部分,您只需要复制pa2到位置的字符pa1并调整其大小即可frase

while (pa2 < &frase[size]) {
    ++pa2;
    *pa1 = *pa2;
    pa1++;
}
frase.resize(size - 1 - (pa2 - pa1));
cout << "La frase dopo l'eliminazione è: " << frase << '\n';

本文收集自互联网,转载请注明来源。

如有侵权,请联系 [email protected] 删除。

编辑于
0

我来说两句

0 条评论
登录 后参与评论

相关文章