将子字符串复制到字符串

Question

我正在尝试从用户那里获取 2 个字符串，第二个将是“针”复制到第一个字符串 例如： 字符串1（用户输入）：八高运费 字符串 2（用户输入）：八个 输出：八高fREIGHT

for example i want to print: toDAY is a good DAY

在堆栈中复制多个针时遇到问题 我曾尝试使用 while (*str) {其余函数与 str++} 我想要一些解释

  #define _CRT_SECURE_NO_WARNINGS
    #define N 101
    #include <stdio.h>
    #include <stdlib.h>
    #include <string.h>
    #include <conio.h>

    void replaceSubstring(char* str, char* substr);
    void main() {
        int flag = 1;
        char str[N], substr[N];
        //char* str_ptr = &str, * substr_ptr = &substr; //creating pointer for the sake of while

        while (flag) {
            printf("\nEnter main text: ");
            gets_s(str,N);
            if (!str)
                flag = 0;
            printf("\nEnter sub-text: ");
            gets_s(substr,N);
            if (!str)
                flag = 0;
            replaceSubstring(str, substr);
            printf("%s",str);
            }
        printf("\nExited. (press any key to exit)");    
    }
    void replaceSubstring(char* str, char* substr) {
        int lensbstr;
        str = strstr(str, substr);
        _strupr(substr); //cnvrt to UPPERCASE
        lensbstr = strlen(substr); //length of the mutual string
        if (str)
            strncpy(str, substr, lensbstr);
    }

Answer 1

这看起来像是一个编程练习，所以我不会只给你答案。不过，我会给你一些提示。

两个大问题：

• 您没有可以替换第二个和以后的实例的循环。
• 您将子字符串大写...不是子字符串的副本。第二次通过 replaceSubstring 只会匹配子字符串的大写版本。

几个小问题/样式cmets：

• str 是一个数组，所以它的值总是非零，所以“if(!str)”永远不会为真。
• strncpy 几乎从来都不是正确的答案。它可以在这里工作，但你不应该养成使用它的习惯。它的行为是微妙的，很少是你想要的。在这里使用 memcpy 会更快更明显。
• 您将子字符串大写并测量其长度，即使您没有找到它，因此不需要这些结果。
• 虽然使用 int 作为标志是可行的并且是传统方式，但新版本的语言具有 stdbool.h、“bool”类型以及“true”和“false”常量。使用这些几乎总是更好。
• 您似乎打算在用户为第一个字符串输入空字符串时停止。那么在这种情况下你为什么要第二个字符串呢？似乎您想要一个无限循环和中间的“中断”。