#include<stdio.h> #include<string.h> int main() { char s1[100],s2[100]; scanf("%s%s",&s1,&s2); int i=0,c=0; int l1=strlen(s1),l2=strlen(s2); for(i=0;i<=l1-l2;i++) { if(strncmp(s1+i,s2,l2)==0)c++; } printf("%d",c); return 0; }这段代码有什么问题

这段代码存在以下问题：

- 在使用 scanf 读取字符串时，应该使用 %s 而不是 &s1 和 &s2。
- 在使用 strncmp 函数时，应该将第一个参数改为 s1+i，而不是 s1。
- 在使用 strncmp 函数时，应该将第三个参数改为 l2+1，以包含字符串末尾的空字符。
- 在最后输出结果时，应该换行以避免输出在同一行。

改正后的代码如下：

#include <stdio.h>
#include <string.h>

int main() {
    char s1[100], s2[100];
    scanf("%s %s", s1, s2);
    int i = 0, c = 0;
    int l1 = strlen(s1), l2 = strlen(s2);
    for (i = 0; i <= l1 - l2; i++) {
        if (strncmp(s1 + i, s2, l2 + 1) == 0) {
            c++;
        }
    }
    printf("%d\n", c);
    return 0;
}

相关问题

编写串的堆式存储，并且不写#include <string.h>这句

以下是一个使用堆式存储的串的示例代码，实现了串的赋值、拼接、比较和输出等基本操作，不使用<string.h>库函数：

#include <stdio.h>
#include <stdlib.h>

#define MAX_SIZE 100 // 字符串最大长度

typedef struct {
    char *str; // 字符串指针
    int len; // 字符串长度
} String;

// 初始化字符串
void initString(String *s) {
    s->str = (char *)malloc(MAX_SIZE * sizeof(char)); // 分配初始空间
    s->len = 0;
}

// 释放字符串空间
void freeString(String *s) {
    free(s->str);
    s->len = 0;
}

// 赋值操作
void assignString(String *s, char *c) {
    int i, len = 0;
    while (c[len] != '\0') { // 计算字符串长度
        len++;
    }
    if (len > MAX_SIZE) {
        printf("String too long!\n");
        exit(1); // 报错退出
    }
    for (i = 0; i < len; i++) {
        s->str[i] = c[i]; // 逐个字符复制
    }
    s->str[len] = '\0'; // 添加结束符
    s->len = len;
}

// 拼接操作
void concatString(String *s, char *c) {
    int i, j, len = 0;
    while (c[len] != '\0') { // 计算字符串长度
        len++;
    }
    if (s->len + len > MAX_SIZE) {
        printf("String too long!\n");
        exit(1); // 报错退出
    }
    for (i = s->len, j = 0; j < len; i++, j++) {
        s->str[i] = c[j]; // 逐个字符复制
    }
    s->str[i] = '\0'; // 添加结束符
    s->len += len;
}

// 比较操作
int cmpString(String *s1, String *s2) {
    int i;
    for (i = 0; i < s1->len && i < s2->len; i++) {
        if (s1->str[i] < s2->str[i]) {
            return -1;
        } else if (s1->str[i] > s2->str[i]) {
            return 1;
        }
    }
    if (s1->len < s2->len) {
        return -1;
    } else if (s1->len > s2->len) {
        return 1;
    }
    return 0;
}

// 输出操作
void printString(String *s) {
    printf("%s\n", s->str);
}

int main() {
    String s;
    initString(&s);
    assignString(&s, "hello");
    concatString(&s, " world");
    printString(&s);
    freeString(&s);
    return 0;
}

注意在本例中，赋值和拼接操作使用了逐个字符复制的方式，而比较操作则使用了逐个字符比较的方式，效率相对较低。在实际应用中，可以根据具体情况选择使用库函数或自己实现。

#include <stdio.h> #include <string.h> void myrevstr(char s1[], char s2[]); //将字符串s2反转后存放在s1中 void myrevstr(char s1[], char s2[]) { int pos = 0; char *p = s2+strlen(s2)-1; while (p>= (1) ) { s1[pos] = *p; (2) ; pos++; } (3) ; } int main() { char s1[80]; myrevstr(s1, "HelloW"); printf("%s", s1);//输出为WolleH return 1; }

完善程序如下：

#include <stdio.h>
#include <string.h>

void myrevstr(char s1[], char s2[]); //将字符串s2反转后存放在s1中

void myrevstr(char s1[], char s2[]) {
    int pos = 0;
    char *p = s2 + strlen(s2) - 1;
    while (p >= s2) {
        s1[pos] = *p;
        p--;
        pos++;
    }
    s1[pos] = '\0';
}

int main() {
    char s1[80];
    myrevstr(s1, "HelloW");
    printf("%s", s1);//输出为WolleH
    return 0;
}

其中，需要注意的点有：

1. 在 `while` 循环中，判断条件为 `p>=s2`，即指针 `p` 没有超过字符串 `s2` 的起始地址。
2. 在循环中，先将 `p` 所指向的字符赋值给 `s1` 中的对应位置，再让指针 `p` 向前移动一位。
3. 循环结束后，需要在 `s1` 的末尾添加一个空字符 `\0`，以表示字符串的结束。