使用指针写入数组元素的问题

Question

第一个循环是完美的。二是不行。 “Z”有什么问题。 为什么循环不停止？ 当我在表格中给出“Z”时，循环没有反应。

#include <iostream>
using namespace std;

int main()
{
    char tab[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G',
                'H', 'I', 'J', 'K', 'L', 'M', 'N',
                'O', 'P', 'Q', 'R', 'S', 'T', 'U',
                'V', 'W', 'X', 'Y', 'Z' };

    char *wsk = tab;

    while(*wsk <= 'Y')
    {
        cout << *wsk << " ";
        wsk++;
    }
    cout << endl;

    wsk = tab;
    while(*wsk <= 'Z')
    {
        cout << *wsk << " ";
        wsk++;
    }
    cout << endl;
}

Answer 1

'Z' 是数组的最后一个元素。在（应该是）最后一次迭代结束时，wsk 递增到指向数组的最后一个元素之外。在下一次迭代中，*wsk <= 'Z' 通过越界指针间接访问，尝试访问不存在的对象，并且程序的行为未定义。

遍历整个数组的简单解决方案：

for(char c : tab)

使用指针的解决方案：

char* end = std::end(tab);
for(char* wsk = tab; wsk != end; wsk++)

您可以根据条件提前中断循环：

for(char c : tab) {
    std::cout << c;
    if (c >= 'Z')
        break;
}

---

如果你特别想要一个循环，当遇到某个值时结束，但该元素也必须被迭代，并且确定该值保证存在，并且由于某种原因无法知道数组的长度，那么你可以使用：

wsk = tab;
do {
    char c = *wsk++;
    std::cout << c;
} while(c < 'Z');

但请注意，如果数组中不存在该元素，这也会有 UB。

Answer 2

在输出'Z' 后，您递增wsk 使其指向数组的末尾。然后它到达while (*wsk <= 'Z')，因为*wsk 正在引用未定义的内存，所以任何事情都可能发生。

你需要while (*wsk < 'Z')，或者也许

for ( ; ; )
  {
  cout << *wsk << " " ;
  if (*wsk >= 'Z') break ;
  wsk++ ;
  }

如果你想在退出循环之前输出'Z'。

Answer 3

这是未定义的行为。

while(*wsk <= 'Z')

您可能希望循环直到找到Z，打印Z，然后退出循环，对吗？但这不是这里发生的事情。你点击了Z，但*wsk <= 'Z' 仍然是正确的，所以你会继续循环。

当您执行wsk++; 时，您现在超出了数组的范围！你刚刚踏入的这个广阔的开放记忆世界里有什么？谁知道！！

鉴于您想在此处执行的操作，您可以通过在数组中添加一个最大 char 值的终止字符来解决此问题：

char tab[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G',
            'H', 'I', 'J', 'K', 'L', 'M', 'N',
            'O', 'P', 'Q', 'R', 'S', 'T', 'U',
            'V', 'W', 'X', 'Y', 'Z', 0x7f };
//                                  ^^^^^^
//Add the largest char to the end of the array

请注意，如果您的数组中的字符全部为 <= 'Y'，这也将防止您的第一个循环导致未定义的行为。我们在这里基本上保证无论内容如何，​​您都不会越界。

Answer 4

使用vector 和range for 循环：

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

int main()
{
    vector<char> tab = { 'A', 'B', 'C', 'D', 'E', 'F', 'G',
                'H', 'I', 'J', 'K', 'L', 'M', 'N',
                'O', 'P', 'Q', 'R', 'S', 'T', 'U',
                'V', 'W', 'X', 'Y', 'Z', 0};


    for(auto it:tab){
        cout<<it<<" ";
        if(it>'Z')
            break;
        }
    cout << endl;
}

Live

Answer 5

和往常一样，问题是你选择了错误的循环。:)

因此，在循环中，您将在数组之外继续迭代。

这个条件

*wsk <= 'Z'

不会通过增加指针来阻止循环再进行一次迭代

wsk++;

在数组的最后一个元素之后访问内存。

改用do-while loop。

这是一个演示程序

#include <iostream>

int main() 
{
    char tab[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G',
                'H', 'I', 'J', 'K', 'L', 'M', 'N',
                'O', 'P', 'Q', 'R', 'S', 'T', 'U',
                'V', 'W', 'X', 'Y', 'Z' };

    for ( char c : tab )
    {
        const char *wsk = tab;

        do
        {
            std::cout << *wsk << " ";
        } while ( *wsk++ != c );

        std::cout << '\n';
    }       

    return 0;
}

它的输出是

A 
A B 
A B C 
A B C D 
A B C D E 
A B C D E F 
A B C D E F G 
A B C D E F G H 
A B C D E F G H I 
A B C D E F G H I J 
A B C D E F G H I J K 
A B C D E F G H I J K L 
A B C D E F G H I J K L M 
A B C D E F G H I J K L M N 
A B C D E F G H I J K L M N O 
A B C D E F G H I J K L M N O P 
A B C D E F G H I J K L M N O P Q 
A B C D E F G H I J K L M N O P Q R 
A B C D E F G H I J K L M N O P Q R S 
A B C D E F G H I J K L M N O P Q R S T 
A B C D E F G H I J K L M N O P Q R S T U 
A B C D E F G H I J K L M N O P Q R S T U V 
A B C D E F G H I J K L M N O P Q R S T U V W 
A B C D E F G H I J K L M N O P Q R S T U V W X 
A B C D E F G H I J K L M N O P Q R S T U V W X Y 
A B C D E F G H I J K L M N O P Q R S T U V W X Y Z