为什么我应该在这里使用 scanf_s 而不是 scanf？

Question

交换值的程序：

#include <stdio.h>
#include <conio.h>

void swap(int *a, int *b);

void main()
{
    int a, b;
    printf("Enter two numbers: ");
    scanf_s("%d%d", &a, &b);

    printf("Before swap\n");
    printf("a = %d, b =  %d\n", a, b);
    swap(&a, &b);
    printf("After swap\n");
    printf("a = %d, b = %d", a, b);
}

void swap(int *a, int *b)
{
    printf("Enter the swapped numbers: \n");
    scanf("%d%d", a, b);
}

为什么我应该在这里使用scanf_s 而不是scanf？ 为什么会这样？

感谢您的帮助。

Answer 1

使用scanf，您可以扫描字符串：

char buffer[256];
scanf("%s", buffer);

但如果输入字符串大于目标缓冲区，它可能会导致缓冲区溢出：如strcpy，它将复制直到在输入中找到'\0'。所以你的编译器告诉你使用scanf_s 来指定缓冲区的大小，这样它就不会写过去：

char buffer[256];
scanf_s("%s", buffer, 256);

但在您的情况下，您只是输入单个整数，无论​​如何您都可以使用scanf。 scanf_s 不是标准函数，因此如果您的代码必须在 Visual Studio 之外编译，请务必小心。

更多：Difference between scanf and scanf_s

Answer 2

在发布的代码中，没有理由在swap 函数中使用scanf 或scanf_s。

scanf_s 是 Microsoft 引入的一个函数，作为 scanf 的所谓 更安全⁽¹⁾ 的替代品，这常常被毫无戒心的程序员粗心地使用，尤其是对于%s、%[ 和 %c 转换，导致安全漏洞。

scanf_s 被标准化为具有微妙不同语义的可选扩展。因此使用scanf_s 的程序是不可移植的。

Microsoft 修改了他们的 C 工具链以促使程序员使用 scanf_s，每当遇到 scanf 引用时都会发出警告或错误。这就是屏​​幕截图中显示消息的原因。

在您的程序中，scanf_s 和 scanf 的行为方式基本相同，因为转换 %d%d 对这两个函数具有相同的语义，但 scanf_s 会检测并处理 %d 的空指针参数（导致程序退出）而scanf 只会有未定义的行为（导致程序退出并出现分段错误）。

swap 函数可以这样写：

void swap(int *a, int *b) {
    int temp = *a;
    *a = *b;
    *b = temp;
}

还要注意main 应该有一个int 返回类型。

---

⁽¹⁾Andrew Henley 评论说这个主题非常敏感：scanf_s 是 Microsoft 作为 供应商特定的、非-portable, not-any-safer 替代scanf，为您的代码库带来vendor-lock-in。他们内部EEE 策略的图示。