[已解决]VS2005中sprintf函数结果不正确

七彩云南

下面的代码:

1. /*------------------------------------------
   
2.    RANDRECT.C -- Displays Random Rectangles
   
3.                  (c) Charles Petzold, 1998
   
4.   ------------------------------------------*/
   
5. 
   
6. #include <windows.h>
   
7. #include <stdlib.h>           // for the rand function
   
8. #include <stdio.h>
   
9. #include <time.h>
   
10. 
    
11. HDC hdc;
    
12. long int times;
    
13. 
    
14. LRESULT CALLBACK WndProc (HWND, UINT, WPARAM, LPARAM) ;
    
15. void DrawRectangle (HWND) ;
    
16. 
    
17. int cxClient, cyClient ;
    
18. 
    
19. int WINAPI WinMain (HINSTANCE hInstance, HINSTANCE hPrevInstance,
    
20.                     PSTR szCmdLine, int iCmdShow)
    
21. {
    
22.      static TCHAR szAppName[] = TEXT ("RandRect") ;
    
23.      HWND         hwnd ;
    
24.      MSG          msg ;
    
25.      WNDCLASS     wndclass ;
    
26.      
    
27.      wndclass.style         = CS_HREDRAW | CS_VREDRAW ;
    
28.      wndclass.lpfnWndProc   = WndProc ;
    
29.      wndclass.cbClsExtra    = 0 ;
    
30.      wndclass.cbWndExtra    = 0 ;
    
31.      wndclass.hInstance     = hInstance ;
    
32.      wndclass.hIcon         = LoadIcon (NULL, IDI_APPLICATION) ;
    
33.      wndclass.hCursor       = LoadCursor (NULL, IDC_ARROW) ;
    
34.      wndclass.hbrBackground = (HBRUSH) GetStockObject (WHITE_BRUSH) ;
    
35.      wndclass.lpszMenuName  = NULL ;
    
36.      wndclass.lpszClassName = szAppName ;
    
37.      
    
38.      if (!RegisterClass (&wndclass))
    
39.      {
    
40.           MessageBox (NULL, TEXT ("This program requires Windows NT!"),
    
41.                       szAppName, MB_ICONERROR) ;
    
42.           return 0 ;
    
43.      }
    
44.      
    
45.      hwnd = CreateWindow (szAppName, TEXT ("Random Rectangles"),
    
46.                           WS_OVERLAPPEDWINDOW,
    
47.                           CW_USEDEFAULT, CW_USEDEFAULT,
    
48.                           CW_USEDEFAULT, CW_USEDEFAULT,
    
49.                           NULL, NULL, hInstance, NULL) ;
    
50.      
    
51.      ShowWindow (hwnd, iCmdShow) ;
    
52.      UpdateWindow (hwnd) ;
    
53.      
    
54.      while (TRUE)
    
55.      {
    
56.           if (PeekMessage (&msg, NULL, 0, 0, PM_REMOVE))
    
57.           {
    
58.                if (msg.message == WM_QUIT)
    
59.                     break ;
    
60.                
    
61.                TranslateMessage (&msg) ;
    
62.                DispatchMessage (&msg) ;
    
63.           }
    
64.           else
    
65.                DrawRectangle (hwnd) ;
    
66.      }
    
67.      return msg.wParam ;
    
68. }
    
69. 
    
70. LRESULT CALLBACK WndProc (HWND hwnd, UINT iMsg, WPARAM wParam, LPARAM lParam)
    
71. {
    
72.          char strTimes[256] = { 0 };
    
73.          static time_t starTime, endTime;
    
74. 
    
75.      switch (iMsg)
    
76.      {
    
77.          case WM_CREATE:
    
78.                   hdc = GetDC (hwnd) ;
    
79.                   starTime = time(NULL);
    
80. 
    
81.                   return 0;
    
82. 
    
83.      case WM_SIZE:
    
84.           cxClient = LOWORD (lParam) ;
    
85.           cyClient = HIWORD (lParam) ;
    
86. 
    
87.           return 0 ;
    
88. 
    
89.          case WM_CLOSE:
    
90.                   endTime = time(NULL);
    
91.                   sprintf(strTimes, "用时 = %d 秒\n矩形数 = %d\n", endTime - starTime, times);
    
92.                   MessageBox(hwnd, strTimes, "结果", MB_OK | MB_ICONINFORMATION);
    
93.                   ReleaseDC (hwnd, hdc);
    
94.                   DestroyWindow(hwnd);
    
95. 
    
96.                   return 0;
    
97.          
    
98.      case WM_DESTROY:
    
99.           PostQuitMessage (0) ;
    
100. 
     
101.           return 0 ;
     
102.      }
     
103.      return DefWindowProc (hwnd, iMsg, wParam, lParam) ;
     
104. }
     
105. 
     
106. void DrawRectangle (HWND hwnd)
     
107. {
     
108.      HBRUSH hBrush ;
     
109.      //HDC    hdc ;
     
110.      RECT   rect ;
     
111.      
     
112.      if (cxClient == 0 || cyClient == 0)
     
113.           return ;
     
114.      
     
115.      SetRect (&rect, rand () % cxClient, rand () % cyClient,
     
116.                      rand () % cxClient, rand () % cyClient) ;
     
117.      
     
118.      hBrush = CreateSolidBrush (
     
119.                     RGB (rand () % 256, rand () % 256, rand () % 256)) ;
     
120. 
     
121.      //hdc = GetDC (hwnd) ;
     
122.      FillRect (hdc, &rect, hBrush) ;
     
123.      //ReleaseDC (hwnd, hdc) ;
     
124.      DeleteObject (hBrush) ;
     
125.          times++;
     
126.          Sleep(1);
     
127. }     
     

复制代码

以上代码中 sprintf函数调用中的times, 用VS2005编译, 结果总是为0,  但是用VC6编译能得以正确结果(生成的矩形数), 问题出的哪里?

最佳答案

月排行榜 / 总排行榜

很cool的阳

2025-1-23 18:36:37

七彩云南 发表于 2025-1-23 17:40
怎么把DrawRectangle函数里的静态变量传给WinMain中的sprintf呢?

1. #include <windows.h>
   
2. #include <stdio.h>
   
3. 
   
4. // DrawRectangle 函数
   
5. int DrawRectangle() {
   
6.     static int staticVar = 0; // 静态变量
   
7.     return staticVar++;       // 返回静态变量的值并自增
   
8. }
   
9. 
   
10. // WinMain 函数
    
11. int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow) {
    
12.     int value = DrawRectangle(); // 获取 DrawRectangle 函数的返回值
    
13. 
    
14.     char buffer[256];
    
15.     sprintf(buffer, "Static Variable Value: %d", value); // 使用返回值
    
16.     MessageBox(NULL, buffer, "Message", MB_OK);
    
17. 
    
18.     return 0;
    
19. }

复制代码

跳转到最佳答案楼层

FishC

在您的代码中，问题的根源在于 `sprintf` 函数用于格式化字符串时，参数类型不匹配。在C语言中，`sprintf` 函数对于 `"%d"` 需要一个 `int` 类型的参数。然而，`times` 是 `long int` 类型（通常是 64 位），而 `time_t` 类型也是平台相关的。因此，您需要确保正确使用的格式化符号以匹配这些类型。

通过确保 `sprintf` 使用正确的格式化字符串可以解决此问题：

1. 将 `sprintf` 函数的格式化字符串更改为匹配 `long int` 类型。
2. 使用 `"%ld"` 来格式化 `long int` 类型的数据。


以下是修正后的代码：

1. char strTimes[100] = { 0 }; // 确保有足够空间存储结果字符串
   
2. static time_t starTime, endTime;
   
3. 
   
4. ...
   
5. 
   
6. case WM_CLOSE:
   
7.      endTime = time(NULL);
   
8.      sprintf(strTimes, "用时 = %ld 秒\n矩形数 = %ld\n", (long)(endTime - starTime), times);
   
9.      MessageBox(hwnd, strTimes, "结果", MB_OK | MB_ICONINFORMATION);
   
10.      ReleaseDC (hwnd, hdc);
    
11.      DestroyWindow(hwnd);
    
12.      return 0;

复制代码

这样调整后，`sprintf` 函数应该会正确地输出 `times` 的值。

另外，要确保字符串缓冲区 `strTimes` 的大小是足够的，以便能够存储格式化后的字符串。此处我将其设定为 100 字符长度，只是为了确保足够空间以避免缓冲区溢出问题，您可以根据实际需要调整。

如果还有其他问题，请告诉我！

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

七彩云南

但是我把times 定义为int  输出时使用 %d ,结果还是为0;

同样的把times定义为long long int, 输出时使用 %lld, 结果仍然是0;

以上情况出现在VSS2005中, VC6能得出正确结果.

FishC

七彩云南 发表于 2025-01-23 17:16
但是我把times 定义为int  输出时使用 %d ,结果还是为 ...

这个问题可能与 `sprintf` 在不同编译器版本中的实现或编译器设置有关。在Visual Studio 2005中，新的C运行库可能会引入一些行为上的差异。以下是一些建议的排查和修复步骤：

1. **确保时间计算的正确性**：由于您提到 `times` 总是为0，请确保在代码中 `times` 变量实际被增量。仔细检查相关逻辑，比如在 `DrawRectangle` 中增加 `times++` 代码。

2. **字符串缓冲区问题**：确保 `strTimes` 的大小足够大，以包含格式化后的结果。此缓冲区需要能够容纳最长的字符串。设定一个合理的缓冲区大小，如：

  

1. char strTimes[100];

复制代码

3. **调试输出**：在 `WM_CLOSE` 之前，输出或打断点检查 `times` 的值，以确保在 sprintf 之前它是正确的。例如，您可以在调试模式下通过直接输出到控制台或通过其他方法来检查 `times` 的值。

4. **调用 _set_SSE2_enable(0)**：在一些特定旧系统上，问题可能与浮点运算相关，可以尝试调用 `_set_SSE2_enable(0)`。但大多数情况下这与您的问题无关。

5. **Visual Studio 设置**：检查编译器设置，确保没有设置特定选项从而导致对 `sprintf` 行为的影响。

以下是一些调试代码的例子：

1. #include <stdio.h>
   
2. 
   
3. // 在 WM_CLOSE 之前打印 times
   
4. fprintf(stdout, "times = %d\n", times);
   
5. 
   
6. // 或者在调试器中设置断点查看 times 的值

复制代码

如果问题仍然存在，请逐步验证每一步以找到根源。确保没有遗漏初始化或者条件竞争等潜在错误。

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

很cool的阳

1. 使用局部静态变量
   
2. 将 times 定义为局部静态变量，而不是全局变量。这样可以确保 times 的值在函数调用之间保持不变。
   
3. 修改代码如下：
   
4. void DrawRectangle(HWND hwnd)
   
5. {
   
6.     static long int times = 0;  // 定义为局部静态变量
   
7.     HBRUSH hBrush;
   
8.     RECT rect;
   
9. 
   
10.     if (cxClient == 0 || cyClient == 0)
    
11.         return;
    
12. 
    
13.     SetRect(&rect, rand() % cxClient, rand() % cyClient,
    
14.                      rand() % cxClient, rand() % cyClient);
    
15. 
    
16.     hBrush = CreateSolidBrush(
    
17.                 RGB(rand() % 256, rand() % 256, rand() % 256));
    
18. 
    
19.     FillRect(hdc, &rect, hBrush);
    
20.     DeleteObject(hBrush);
    
21.     times++;
    
22.     Sleep(1);
    
23. }
    
24. 然后在 WM_CLOSE 消息中，将 times 的值传递给 MessageBox：
    
25. case WM_CLOSE:
    
26.     endTime = time(NULL);
    
27.     sprintf(strTimes, "用时 = %d 秒\n矩形数 = %d\n", endTime - starTime, times);
    
28.     MessageBox(hwnd, strTimes, "结果", MB_OK | MB_ICONINFORMATION);
    
29.     ReleaseDC(hwnd, hdc);
    
30.     DestroyWindow(hwnd);
    
31.     return 0;

复制代码

七彩云南

很cool的阳 发表于 2025-1-23 17:20

怎么把DrawRectangle函数里的静态变量传给WinMain中的sprintf呢?

七彩云南

我用:

1. sprintf(strTimes, "矩形数： %d\n", times);

复制代码

替换

1. sprintf(strTimes, "用时 = %d 秒\n矩形数 = %d\n", endTime - starTime, times);

复制代码

即只传 times 一个变量给 sprintf函数时, 可以正确输出 times 的值, 但传2 个变量时  times 的值总是0.

很cool的阳

七彩云南 发表于 2025-1-23 17:40
怎么把DrawRectangle函数里的静态变量传给WinMain中的sprintf呢?

1. #include <windows.h>
   
2. #include <stdio.h>
   
3. 
   
4. // DrawRectangle 函数
   
5. int DrawRectangle() {
   
6.     static int staticVar = 0; // 静态变量
   
7.     return staticVar++;       // 返回静态变量的值并自增
   
8. }
   
9. 
   
10. // WinMain 函数
    
11. int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow) {
    
12.     int value = DrawRectangle(); // 获取 DrawRectangle 函数的返回值
    
13. 
    
14.     char buffer[256];
    
15.     sprintf(buffer, "Static Variable Value: %d", value); // 使用返回值
    
16.     MessageBox(NULL, buffer, "Message", MB_OK);
    
17. 
    
18.     return 0;
    
19. }

复制代码

七彩云南

真见鬼了.

把两个参数交换下位置, 先输出矩形数, 后输出时间
即把sprintf改为:

1. sprintf(strTimes, "矩形数 = %d \n时间 = %d 秒\n", times, endTime - starTime);

复制代码

可以得到正确输出.

七彩云南

增加一个中间变量保存 endTime - starTime, 用于保存时间变量的值, 再把这个变量作为sprintf参数:
即:

1. int shijian;
   
2. 
   
3. shijian = endTime - starTime;
   
4. sprintf(strTimes, "时间 : %d 秒\n矩形数 ： %d\n", shijian, times)

复制代码

可以正确输出.

另:GCC11.4.0上没有这个问题, 可以正确执行并正确输出.