代码
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
int myCompare (const void * a, const void * b ) {
const char *pa = *(const char**)a;
const char *pb = *(const char**)b;
return strcmp(pa,pb);
}
int main() {
int i;
// const char *input[] = {"a","orange","apple","mobile","car"};
const char *input[1024];
input[0]="a";
input[1] = "orange";
input[2] = "apple";
input[3] = "apple";
input[4] = "mobile";
input[5] = "car";
int stringLen = sizeof(input) / sizeof(char *);
qsort(input, stringLen, sizeof(char *), myCompare);
for (i=0; i<stringLen; ++i)
printf("%d: %s\n", i, input[i]);
}
这段代码为什么会报 coredump
但是如果把input
变成const char *input[] = {"a","orange","apple","mobile","car"};
,那么就不会报错,求指点!!
1
littlewing 2018-12-24 19:05:34 +08:00
指针都没 malloc 不 core 才怪,先吧指针数组和数组指针弄清楚吧
|
2
May725 2018-12-24 19:19:26 +08:00 via iPhone
记住一点,程序崩溃与内存有关,在 c 语言上可以联想到指针错误,所以要查查指针相关的调用。楼上也直接回答了原因
|
3
Nasei 2018-12-24 19:22:38 +08:00 1
1024 这么大的数组, 你就给了 6 个值, 然后 stringLen 就错了
|
5
sl0000 2018-12-24 20:55:36 +08:00 1
1024 - 6 后面的 char * 都是 0x0, 但是 strcmp 的参数不能为 NULL
|
6
2exploring 2018-12-24 21:23:51 +08:00
一二楼都指出问题在哪儿了,三楼和五楼都还没抓住问题的本质?
|
7
sl0000 2018-12-24 21:27:54 +08:00
2exploring 什么本质?
|
8
sl0000 2018-12-24 21:28:40 +08:00
|
9
zhuangzhuang1988 2018-12-24 21:40:18 +08:00 1
好好的用 IDE 一看就知道 |
10
xi2008wang 2018-12-24 21:51:07 +08:00 via iPhone
大型打脸现场
|
11
2exploring 2018-12-24 21:59:25 +08:00
@sl0000 我收回上面说的话……
一开始看一楼说 malloc 我就认为应该给 input malloc,其实不应该,是我大意了…… 三楼说的很对,不过你说的有点问题,其它位置的值不一定为 null,而是任何值都有可能。 |
12
sl0000 2018-12-24 22:06:44 +08:00
接受! @2exploring
大型科普现场: 字符串 定义是从以第一个字节往后到'\0'为止. const char *input[] = {"a","orange","apple","mobile","car"}; 这种写法, 会把后面的空白元素都初始化 0x0 或者 0xcc...(这个指针大小也是不固定的). const char *input[1024]; 这种不会初始化; |
13
sl0000 2018-12-24 22:14:44 +08:00
额, const char *input[] = {"a","orange","apple","mobile","car"}; 这种写法元素就是 5 个.
上面应该说的是 const char *input[1024] = {"a","orange","apple","mobile","car"}; 为什么不能重新编辑!! |
14
tmy 2018-12-25 09:51:58 +08:00
const char *input[] = {"a","orange","apple","mobile","car"}这里 input 表示有 5 个指针的指针数组
const char *input[1024]; input[0]="a"; input[1] = "orange"; input[2] = "apple"; input[3] = "apple"; input[4] = "mobile"; input[5] = "car";这个表示 input 是一个有 1024 个指针的指针数组,前面 6 个有明确的指针,后面的指针是野指针,那么在你的比较函数里面 strcmp 用到了野指针,结果可能崩溃也可能不崩溃,你改为 for (i=0; i<6; ++i)这个后没有使用到野指针也就不会崩溃 |
15
aa514758835 2018-12-25 09:58:56 +08:00
你得先知道 const char *input[1024]; 这句话是什么意思,你定义了一个指针数组,是一个数组,内容全是空指针。你需要为指针 malloc 空间才可以往里面存值。如果你改为 const char input[1024]; 此时
|
16
aa514758835 2018-12-25 09:59:35 +08:00
@aa514758835 摁了个回车居然自动发出去了,我话还没说完呢。 此时是一个数组,开在了栈上,所以直接可以赋值,不会报错
|
17
stebest 2018-12-25 13:32:17 +08:00
确实是一各初学问题,然而楼上的大佬们都互相打脸 2333。
一楼的前半句就是对的,指针用多少写多少,申请那么多不给空间不初始化就调用,谁受得了。 但并不是指针数组和数组指针的区别。 |
18
2exploring 2018-12-25 22:56:24 +08:00
@aa514758835 const char *input[1024] 是一个指针数组你说对了,但内容并不全是空指针,也不是一定需要 malloc 才可以使用,只要给一个合法的指针值给它就可以用,比如 input[0] = "a","a" 就是一个合法的指针。楼主原始代码的问题在于没有确保 input[6..1023] 内容是合法的指针就使用了它们。至于你说的改为 const char input[2014],这还是原本的意思吗?还能完成原本的功能吗?
|
19
chenqh OP @2exploring 因为一开始不知道 input 到底有多大,所以给个 1024 的值,这是为了模拟
|
20
aa514758835 2018-12-26 09:31:16 +08:00
@2exploring 至于不 malloc 也能用的话,我以前是可以的,但是自从 vs 升到 2017 后,不开空间赋值就会报错了,估计其他编译器没事吧。最后那个 const char input[2014] 我确实说错了,忘了有 const,蛋疼
|
21
2exploring 2018-12-26 13:40:02 +08:00
@aa514758835 你在 20 楼中的编译报错是因为你把一个 const char* 类型赋给了 char* 类型,这在 C++ 中是不被允许的( C 好像没有此限制,你的源文件后缀是 cpp,想必是 c++ 了),并不能说明不 malloc 就不能用,你好好看一下编译器给的错误信息。
还有你说的改为 const char input[1024] 这个的问题也不在 const,而是你不明不白地把一个指针数组改为了 char 数组,我不知道你要怎么去使用这个 input ? |
22
aa514758835 2018-12-26 13:48:15 +08:00
@2exploring 看来我还得再多了解一下......
|