调用XInitThreads()后BGI程序仍崩溃？DDA画线算法异常排查

你遇到的这个XCB序列错误，核心问题出在XInitThreads的调用时机不对，再加上代码里DDA算法的逻辑错误，双重因素导致程序时不时崩溃。咱们一步步来修复：

问题根源分析 #

1. XInitThreads调用时机错误：这个函数必须在任何Xlib相关调用（包括BGI的initgraph）之前执行，而且只需要调用一次。你现在在main和drawLine里都调用了，反而会干扰X的线程初始化流程。
2. DDA算法逻辑错误：你初始化y = endY是完全错误的，应该是startY；另外step需要取绝对值，否则当坐标差为负数时，循环次数会出错。
3. 绘图循环中的delay可能触发线程冲突：BGI的delay在多线程环境下（即使你没显式开线程，Xlib本身是多线程的）可能会干扰事件队列处理。

修复后的完整代码 #

#include <stdio.h>
#include <graphics.h>
#include <math.h>
#include <X11/Xlib.h>

void drawLine(int startX, int startY, int endX, int endY) {
    // DDA Line Drawing Algorithm
    float deltaX, deltaY, step;
    float x = startX, y = startY; // 修复：初始化为起点坐标，用float避免整数截断误差
    deltaX = (float)(endX - startX);
    deltaY = (float)(endY - startY);

    // 取绝对值确定步长，确保循环次数正确
    if(fabs(deltaX) > fabs(deltaY)) {
        step = fabs(deltaX);
    } else {
        step = fabs(deltaY);
    }

    // 计算每步的增量
    float xIncrement = deltaX / step;
    float yIncrement = deltaY / step;

    // 先绘制起点
    putpixel((int)(x + 0.5), (int)(y + 0.5), 15);

    for(int i = 0; i < step; i++) {
        x += xIncrement;
        y += yIncrement;
        // 四舍五入后绘制像素
        putpixel((int)(x + 0.5), (int)(y + 0.5), 15);
        delay(50); // 缩短delay减少线程阻塞概率
    }
}

int main() {
    // 关键：XInitThreads必须在initgraph之前调用，且仅调用一次
    XInitThreads();

    // Graphics Initialization
    int gd = DETECT, gm;
    initgraph(&gd, &gm, NULL);

    // 获取用户输入
    int startX, startY, endX, endY;
    printf("Starting point:\nX = ");
    scanf("%d", &startX);
    printf("\nY = ");
    scanf("%d", &startY);
    printf("Ending point:\nX = ");
    scanf("%d", &endX);
    printf("\nY = ");
    scanf("%d", &endY);

    delay(1000);
    drawLine(startX, startY, endX, endY);

    // 等待用户输入再关闭图形窗口，避免窗口立即消失
    printf("Press any key to exit...\n");
    // 处理scanf留下的换行符，避免getchar直接读取换行
    while(getchar() != '\n');
    getchar();

    closegraph();
    return 0;
}

关键改动说明 #

• 调整XInitThreads调用时机：只在main开头调用一次，确保在initgraph之前执行，正确初始化X的线程支持。
• 修复DDA算法逻辑：
  • 初始坐标改为起点startX和startY，用float类型存储中间值，避免整数运算的截断误差。
  • step使用fabs取绝对值，确保循环次数始终是正的。
  • 提前计算每步的增量，让循环逻辑更清晰。
• 优化输入处理：处理scanf留下的换行符，避免getchar直接读取换行导致窗口立即关闭。
• 缩短delay时间：减少线程阻塞的概率，降低X事件队列冲突的可能。

编译注意事项 #

编译时需要链接BGI和X11相关库，比如：

gcc your_code.c -o dda_line -lbgi -lX11 -lXext -lXmu

这样修改后，程序应该能稳定运行，不会再出现XCB的线程崩溃问题了。

内容的提问来源于stack exchange，提问作者Abhishek Kuvalekar