DWORD disk_serialINT[MAX_PATH + 1];
GetVolumeInformationA(NULL, NULL, NULL, disk_serialINT, NULL, NULL, NULL, NULL);
char* disk_serialANSI;
sprintf(disk_serialANSI, "%d", disk_serialINT);
std::string HDDserial = disk_serialANSI;
这是我获得硬盘序列号的代码段,但是问题是,每次程序执行时,值都是不同的。有人可以解释吗?
已解决:
DWORD disk_serialINT;
GetVolumeInformationA(NULL, NULL, NULL, &disk_serialINT, NULL, NULL, NULL, NULL);
std::string HDDserial = std::to_string(disk_serialINT);
谢谢。
这两行将为您提供undefined behavior:
char* disk_serialANSI;
sprintf(disk_serialANSI, "%d", disk_serialINT);
您声明了一个指针变量,但实际上并未使其指向任何地方。未初始化的局部变量具有不确定的值(实际上,它似乎是随机的),并且通过使用该未初始化的指针,您不知道where sprintf
调用将写入。
由于您使用C ++进行编程,因此有两种解决方案。
老式的方法是使disk_serialANSI
成为字符数组,其大小足以容纳数字(包括字符串终止符)。另一种方法是手动为指针分配内存,然后在完成使用后再次释放该内存。
使用std::ostringstream
格式化数据并获得std::ostringstream
。
使用std::string
直接转换为字符串。
在Joachim所说的之上,您没有正确传递序列号。您应该将指针传递给单个值。
std::to_string
在您的代码中,这与您在此处所做的事情很复杂:
Boost Lexical cast
忽略Joachim的答案所涉及的未初始化变量DWORD disk_serialINT;
GetVolumeInformationA(NULL, NULL, NULL, &disk_serialINT, NULL, NULL, NULL, NULL);
,您正在传递指向sprintf(disk_serialANSI, "%d", disk_serialINT);
格式字符串的指针。一旦将disk_serialANSI
更改为单个值,情况会更好。但是,您要将无符号值传递给单个格式的字符串。
是时候放弃这些陈旧的C格式化功能,而使用C ++标准库在整数值和文本之间进行转换了。
最后要强调的一点是,您必须检查对Win32 API的调用的返回值。您不知道函数调用是否成功。您不得以为是。这些全部包含在文档中:"%d"
此程序将打印包含当前目录的卷的序列号:
disk_serialINT
如果有人来查看此代码,这里的解决方案就是我的看法:
问题1
https://msdn.microsoft.com/en-us/library/windows/desktop/aa364993.aspx
#include <Windows.h>
#include <iostream>
int main()
{
DWORD disk_serialINT;
if (!GetVolumeInformationA(NULL, NULL, NULL, &disk_serialINT, NULL,
NULL, NULL, NULL))
{
std::cout << "Failed: " << GetLastError() << std::endl;
return 1;
}
std::cout << "Current directory volume serial numnber: " << std::hex
<< disk_serialINT << std::endl;
return 0;
}
值的数组。DWORD disk_serialINT[MAX_PATH + 1];
宏(并向其添加1)而不了解其用途。DWORD
类型为unsigned时为变量MAX_PATH
命名)。INT
实际上是volume序列号时将其命名)。问题2
DWORD
disk_serial
变量的地址)。问题#3
GetVolumeInformationA(NULL, NULL, NULL, disk_serialINT, NULL, NULL, NULL, NULL);
问题4
disk_serialINT
char* disk_serialANSI;
。sprintf(disk_serialANSI, "%d", disk_serialINT);
数组,或保留指针但分配内存,然后释放它)。>>disk_serialANSI
。]char
的安全版本。问题5
%08lX
换句话说,您所有典型的初学者程序员都犯了错误。希望他们在此期间能学到更多,而不仅仅是在Stack Overflow上找到问题的答案。
[David Heffeman上面已经提供了C ++解决方案,所以为了回答完整,这里是等效的C解决方案:
sprintf