免费注册 查看新帖 |

Chinaunix

  平台 论坛 博客 文库
最近访问板块 发新帖
查看: 1818 | 回复: 5
打印 上一主题 下一主题

我自己写了个SUBSTR,大家看下是否有问题。 [复制链接]

论坛徽章:
0
跳转到指定楼层
1 [收藏(0)] [报告]
发表于 2006-09-05 22:40 |只看该作者 |倒序浏览
  1. #include <stdio.h>
  2. #include <assert.h>
  3. char* substr(char *str,int start,int length){
  4.     if (start < 1)
  5.         return NULL;//illegal
  6.     if (length < 1)
  7.         return NULL;//illegal
  8.     if ((start + length - 1) > strlen(str))
  9.         return NULL;//illegal
  10.     assert( str != NULL);
  11.     char *p;
  12.     char *q;
  13.     int i = 1;
  14.     p = (char*)malloc(length + 1); //+1 for '\0'
  15.     q = p; //save the addr
  16.     str--;
  17.     while ((*str) != '\0')
  18.     {
  19.         if (i >= start && i<= (start + length - 1))
  20.         {
  21.           *p = *str;
  22.            p++;
  23.         }
  24.         str++;
  25.         i++;
  26.     }
  27.     *p = '\0';
  28.     p = q;
  29.     return p;
  30. }
  31. int main(){
  32.     char* t;
  33.     t = substr("12345",1,1);
  34.     if (t == NULL){
  35.         printf("substr is illegal");
  36.     }else{
  37.         printf("[%s]\n",t);
  38.     }
  39.     free(t);   
  40. }
复制代码

论坛徽章:
0
2 [报告]
发表于 2006-09-05 22:52 |只看该作者

这个函数有两种写法,不知道哪个合理。或者大家写这个函数是怎么写的?

1。按楼上的写法,内存分配在substr函数中进行。
2。或者内存分配在函数外完成,把指针传入。函数变为:


  1. void substr(char *desc,char *str,int start,int length){
  2. }
  3. 调用的时候:
  4. 在MAIN函数中
  5. char* t;
  6. t = (char*)malloc(length + 1);
  7. substr(t,"12345",1,1);
复制代码

这样也可以,但不知道哪种好,哪种更合理。

论坛徽章:
0
3 [报告]
发表于 2006-09-05 23:34 |只看该作者
这样写可能更好!
char  *substr(char *desc,const char *src,int offset, int n_size)
{
        //这段不变
        if (offset< 1)
                return NULL;//illegal
        if (n_size< 1)
                return NULL;//illegal
        if ((offset+ n_size- 1) > strlen(src))
                return NULL;//illegal
        assert( src!= NULL);
        //-------------------
        char *pc =desc;
        int n_StrCount=0;
        while(n_StrCount<offset)
        {
               ++src;
               n_StrCount++;
        }
            
                n_StrCount=0;
        while (n_StrCount<n_size)
        {
              *pc++=*src++;
                       n_StrCount++;
        }
    *pc='\0';
  return(desc);
}

//注意
desc必须先分配好..不能指空地址NULL
如在main()定义为如下格式
char t[n_size];
或者
char *t;
  t=new char[n_size+1];
  //(或者)
  t=(char*)malloc(sizeof(char)*n_size+1)

   char *t1=new char[n_size+1];
  //调用如下
  t1=substr(t,"I love china",2,4);
//将得到
  t1="love"
  t  ="love"

[ 本帖最后由 bain8181 于 2006-9-5 23:41 编辑 ]

论坛徽章:
0
4 [报告]
发表于 2006-09-06 09:25 |只看该作者
我讲2点:
C 语言下标从 0 开始。
你判断 start < 1 是不是准确?
难道就不允许 从开始取?

((start + length - 1) > strlen(str)) 时 是取可取部分还是非法?到底怎么样更合理?
如果你认为非法,那么后面的while代码都是垃圾,只需要一个 memcpy 即可。

论坛徽章:
0
5 [报告]
发表于 2006-09-06 09:39 |只看该作者
q = p; //save the addr
str--;
while ((*str) != '\0')

红色部分是个笔误吧

论坛徽章:
0
6 [报告]
发表于 2006-09-06 10:17 |只看该作者
你这个函数与strndup()相似,
只是多一个起始位置参数, 和一些参数除错功能,
完全可以简化:

char *substr(const char *str, int start, int length)  {
    判断参数;
    ...

    return strndup(str+start, length);
}
您需要登录后才可以回帖 登录 | 注册

本版积分规则 发表回复

  

北京盛拓优讯信息技术有限公司. 版权所有 京ICP备16024965号-6 北京市公安局海淀分局网监中心备案编号:11010802020122 niuxiaotong@pcpop.com 17352615567
未成年举报专区
中国互联网协会会员  联系我们:huangweiwei@itpub.net
感谢所有关心和支持过ChinaUnix的朋友们 转载本站内容请注明原作者名及出处

清除 Cookies - ChinaUnix - Archiver - WAP - TOP