为什么非常大的If语句会导致堆栈溢出

【问题标题】：Why do very large If statements cause a stack overflow为什么非常大的If语句会导致堆栈溢出
【发布时间】：2016-04-04 19:45:43
【问题描述】：

我编写了一个跨越几千行的 if 语句，其中包含许多其他嵌套的 if 语句。当我没有注释掉很多它时，我得到一个堆栈溢出错误。我知道这是由于大小的原因，因为我发现这个特殊的 if 语句只能处理 6 个不同的代码块 [表示“if (mainmenu1.equipment1A) 的块] [每个 if ((mainmenu1.player1equipment && ( Player1.job == 1)) ||... block] 并且如果有 7 个没有注释掉哪些块，无论哪个 7，它都会给我以下错误。虽然很奇怪，这代码是我正在开发的游戏中代码的一部分，并且代码在 switch 语句中处于完全不同的游戏状态情况下，并且在启动时不应读取此代码。构建时我没有收到编译错误代码被注释掉了，但是在启动时我得到了这个错误。难道不是运行时错误吗？但在这种情况下，在程序读取代码之前我不应该得到错误。我将包含一个代码示例，总共大约 70,000 行，但它是重复的，所以想象一下下面的代码乘以 70。

SFML.exe 中 0x103B38B7 (ig7icd32.dll) 处的第一次机会异常：0xC00000FD：堆栈溢出（参数：0x00000000、0x01072000）。 SFML.exe 中 0x103B38B7 (ig7icd32.dll) 处未处理的异常：0xC00000FD：堆栈溢出（参数：0x00000000、0x01072000）。

SFML.exe 中 0x101AC0BE (ig7icd32.dll) 的第一次机会异常：0xC0000005：访问冲突写入位置 0x0107091C。 SFML.exe 中 0x101AC0BE (ig7icd32.dll) 处未处理的异常：0xC0000005：访问冲突写入位置 0x0107091C。

//equipment text
    if (mainmenu1.equipmentmenu2)
    {
        if (executeequipmenttextonce)
        {
            executeequipmenttextonce = false;

            if (mainmenu1.rightselected)
            {
            if ((mainmenu1.player1equipment && (Player1.job == 1)) || (mainmenu1.player2equipment && (Player2.job == 1)) || (mainmenu1.player3equipment && (Player3.job == 1)))
            {
                if (mainmenu1.equipment1A)
                {
                    if (!mainmenu1.knightweapon1displayed && mainmenu1.knightweapon1)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon1displayed = true;
                        textDisplay16.text.setString("Broadsword");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw1));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon2displayed && mainmenu1.knightweapon2)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon2displayed = true;
                        textDisplay16.text.setString("Ea");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw2));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon3displayed && mainmenu1.knightweapon3)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon3displayed = true;
                        textDisplay16.text.setString("Zangetsu");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw3));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon4displayed && mainmenu1.knightweapon4)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon4displayed = true;
                        textDisplay16.text.setString("Silver Fleurette");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw4));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon5displayed && mainmenu1.knightweapon5)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon5displayed = true;
                        textDisplay16.text.setString("Lawliet");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw5));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon6displayed && mainmenu1.knightweapon6)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon6displayed = true;
                        textDisplay16.text.setString("Luminous Edge");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw6));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon7displayed && mainmenu1.knightweapon7)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon7displayed = true;
                        textDisplay16.text.setString("Heaven's Saber");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw7));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon8displayed && mainmenu1.knightweapon8)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon8displayed = true;
                        textDisplay16.text.setString("Crystal Edge");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw8));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon9displayed && mainmenu1.knightweapon9)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon9displayed = true;
                        textDisplay16.text.setString("Triple Zoro");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw9));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon10displayed && mainmenu1.knightweapon10)
                    {
                        mainmenu1.equipment1A = false;
                        mainmenu1.knightweapon10displayed = true;
                        textDisplay16.text.setString("Elsword");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw10));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY1);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                }
                else if (mainmenu1.equipment2A)
                {
                    if (!mainmenu1.knightweapon1displayed && mainmenu1.knightweapon1)
                    {
                        mainmenu1.equipment2A = false;
                        mainmenu1.knightweapon1displayed = true;
                        textDisplay16.text.setString("Broadsword");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY2);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw1));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY2);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon2displayed && mainmenu1.knightweapon2)
                    {
                        mainmenu1.equipment2A = false;
                        mainmenu1.knightweapon2displayed = true;
                        textDisplay16.text.setString("Ea");
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY2);
                        textDisplayArray16.push_back(textDisplay16);
                        textDisplay16.text.setString(to_string(mainmenu1.knightw2));
                        textDisplay16.text.setPosition(view1.getCenter().x - window.getSize().x / 2 + equipmentPositionX1 + equipmentstockpositionx, view1.getCenter().y - window.getSize().y / 2 + equipmentPositionY2);
                        textDisplayArray16.push_back(textDisplay16);
                    }
                    else if (!mainmenu1.knightweapon3displayed && mainmenu1.knightweapon3)
                    {
                        mainmenu1.equipment2A = false;
                        mainmenu1.knightweapon3displayed = true;
                        textDisplay16.text.setString("Zangetsu");

【问题讨论】：

  • "我编写了一个跨越几千行的 if 语句"
  • ..但是看看那些潜伏在拐角处等待被发现的重构/设计模式！
  • 运行语句和运行代码。
  • 它在绘图代码中崩溃了，所以你给它喂了一些它不喜欢的东西。这很可能不是关于这段代码，而是一个副作用。正确复制您使用的对象中的元素？
  • “总共大约 70,000 行，但它是重复的” - 重复代码是所有反模式中最糟糕的一种。

标签： c++ runtime-error stack-overflow sfml


【解决方案1】：

从技术上讲，if 语句不会占用堆栈空间。

if 表达式中的函数调用可能会导致 StackOverflow。

在语句块中分配过多的局部变量可能会导致堆栈溢出。

在语句块中分配大变量可能会导致堆栈溢出。

不受限制的递归函数可能会导致堆栈溢出。

使用大量局部变量或参数的递归函数可能会导致堆栈溢出。

if 语句通常被评估为比较指令和分支语句。这些指令不占用堆栈空间。

您的错误可能位于其他地方。

使用调试器查找根本原因。

【讨论】：

  • 并考虑重新设计，这样您就不需要大量的if 语句。这似乎是 OP 最大的问题 - 庞大、不可维护且容易出错的复制粘贴代码。
  • @WakeUpToFlatEarth 查看gameprogrammingpatterns.com/component.html。很抱歉重复发布，但问题评论线程主要是关于没有提供解决方案的代码有多糟糕。
猜你喜欢
  • 2011-01-13
  • 1970-01-01
  • 2014-12-20
  • 1970-01-01
  • 2010-10-02
  • 2016-02-24
  • 2018-07-01
  • 2010-09-11
  • 1970-01-01
相关资源
最近更新 更多
热门标签
Java Python linux javascript Mysql C# Docker 算法 前端 SpringBoot Redis Vue spring 设计模式 .net core .net kubernetes c++ 数据库 数据结构 大数据 js 机器学习 微服务 Android Go 程序员 面试 JVM ASP.net core 云原生 人工智能 后端 PHP git CSS golang k8s Nginx Django mybatis 深度学习 多线程 React 架构 devops 爬虫 云计算 Spring Boot LeetCode