尝试从字符串中删除后缀但没有成功

Question

我正在尝试用下一种方式删除后缀，但最终输出与输入相同

    #include <stdio.h>
    #include <string.h>
    #include <stdlib.h>
    
    #define SUFFIX ".doc"
    
    static void removeSuffix(char **outNewFileName, const char *inFileName)
    {
        *outNewFileName = strdup(inFileName);
        outNewFileName[strlen(inFileName) - strlen(SUFFIX)];
    }

    int main ()
    {
        char *fileName = "tmp.doc";
        char *outnewFileName  = NULL;
        
        removeSuffix(&outnewFileName, fileName);

        free(outnewFileName);
    
        return 0;
    }

例如，如果 fileName 是 tmp.doc outnewFileName 也是 tmp.doc

Answer 1

你很亲密！您的 strlen(inFileName) - strlen(SUFFIX) 表达式找到了终止新字符串的正确位置，但您实际上并未对该表达式执行任何操作。

要在该位置终止字符串，请将那里的char 的值设置为零（即nul 终止符）：

static void removeSuffix(char** outNewFileName, const char* inFileName)
{
    *outNewFileName = strdup(inFileName);
    (*outNewFileName)[strlen(inFileName) - strlen(SUFFIX)] = '\0';
}

在这段代码中，我们使用(*outNewFileName) 来指代新的字符串数组，使用方括号中的偏移量来指代需要更改为nul 的实际字符。 （圆括号是必需的，因为[] 运算符的优先级高于* 运算符。）

---

注意：正如 cmets 中所指出的，如果您启用了完整的编译器警告，您会看到这一点（来自 clang-cl）：

警告：表达式结果未使用 [-Wunused-value]

请随时要求任何进一步的澄清和/或解释。

Answer 2

对于初学者，总是尝试编写更通用的函数。

这个函数声明

static void removeSuffix(char **outNewFileName, const char *inFileName);

不好。它依赖于魔术字符串文字".doc"。所以你不能使用该功能删除其他后缀。

第二次通过引用传递第一个参数（指针）会产生一个问题，是否应该在分配新地址之前释放指针。

函数定义也没有意义

static void removeSuffix(char **outNewFileName, const char *inFileName)
{
    *outNewFileName = strdup(inFileName);
    outNewFileName[strlen(inFileName) - strlen(SUFFIX)];
}

看来你的意思

static void removeSuffix(char **outNewFileName, const char *inFileName)
{
    *outNewFileName = strdup(inFileName);
    ( *outNewFileName )[strlen(inFileName) - strlen(SUFFIX)] = '\0';
}

但是这种方法是不正确的。传递的字符串可以没有必须删除的后缀。在这种情况下，该函数将返回一个无效字符串。或者传递的字符串可以多次包含与后缀相同的组合。而且用户可以传递一个长度小于后缀长度的字符串。

所以这个函数声明和它的整体定义是错误的和不好的。

该函数可以通过以下方式声明和定义，如下面的演示程序所示。

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

char * removeSuffix( const char *s1, const char *s2 )
{
    char *p = NULL;
    
    if ( *s2 != '\0' )
    {
        for ( char *current = ( char * )s1; ( current = strstr( current, s2 ) ) != NULL; ++current )
        {
            p = current;
        }
    }       
    
    if ( p == NULL || *( p + strlen( s2 ) ) != '\0' ) p = ( char * )( s1 + strlen( s1 ) );
    
    size_t n = p - s1;
        
    p = malloc( n + 1 );
        
    if ( p )
    {
        memcpy( p, s1, n );
        p[n] = '\0';
    }

    return p;
}

int main(void) 
{
    char *fileName = "tmp.doc.doc";
    char *suffix = ".doc";
    
    char *outnewFileName  = removeSuffix( fileName, suffix );
    
    if ( outnewFileName ) puts( outnewFileName );
    
    free( outnewFileName );
    
    return 0;
}

程序输出是

tmp.doc

如您所见，该函数确实从字符串"tmp.doc.doc" 中删除了包含符号".doc" 的两个组合的后缀。

Answer 3

我宁愿：

char* strrrstr(const char *haystack, const char *needle)
{
    const char *r = NULL;
    size_t nlen = strlen(needle);

    if(haystack && needle)
    {
        while (1) 
        {
            char *p = strstr(haystack, needle);
            if (!p)
                break;
            r = p;
            haystack = p + nlen;
        }
    }
    return (char *)r;
}


#define SUFFIX ".doc"

static char *removeSuffix(const char *str, const char *suffix)
{
    char *newString = strdup(str);
    char *pos;
    if(newString)
    {
        pos = strrrstr(newString, suffix);
        if(pos) *pos = 0;
    }
    return newString;
}

int main ()
{
    static char *fileName = "tmp.doctmp.doc";
    char *outnewFileName;
    
    outnewFileName = removeSuffix(fileName, SUFFIX);

    printf("%s\n", outnewFileName);

    free(outnewFileName);

    return 0;
}

您应始终阅读警告并将警告级别设置为尽可能高。如果你这样做了：

<source>:36:23: warning: statement with no effect [-Wunused-value]

   36 |         outNewFileName[strlen(inFileName) - strlen(SUFFIX)];

      |         ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

解释问题。

Answer 4

"...从字符串中删除后缀没有成功"

如下：

outNewFileName[strlen(inFileName) - strlen(SUFFIX)];

正在修改的变量需要表示为(*outNewFileName)，所以上面改成：

(*outNewFileName)[strlen(inFileName) - strlen(SUFFIX)];

此外，在这种方法中，可以通过数学运算得出数组索引，但很难正确执行，并且不是从字符串中截取文件扩展名的唯一方法。以下是替代方法，例如，不需要使用 #define 值，并且不需要数组索引计算。 （但确实需要. 文件扩展分隔符）...

char *new = strdup(filename);//after getting a new string at least as long as original...
if(new)//use strtok
{
    char *tok - strtok((*outNewFileName), ".");
    if(tok)
    {
        strcpy((*outNewFileName), tok);//tok contains string w/o extension.
        ...

编辑以解决 cmets：

另一种比前一种更灵活的方法是创建一种能力，以指定哪个分隔符来标识文件扩展名，并将其作为参数传递以返回字符串的部分直到并包括最后一次出现 /em> 的分隔符。例如：

strcpy((*outNewFileName), return_base_filename((*outNewFileName), '.'));

其中return_base_filename()定义为：

//returns string upto last occurance of delimiter.
char * return_base_filename(char *in, char delimiter)
{
    char *ptr = in;
    char *ptrKeep = NULL;
    
    while(*ptr)
    {
        if(*ptr == delimiter) 
        {
            ptrKeep = ptr;
        }
        ptr++;
    }
    in[ptrKeep - in]=0;
    return in;
}

请注意，此方法适用于以下文件名/目录形式：

• "base.ext"
• "base1.base2.base3.ext"
• "C:\\one\\two\\three\\with.this.file.txt"

第 3 次需要 2 次调用 'return_base_filename(,)'，第一次使用 '\' 作为分隔符，第二次使用 '.'。

char buf[] = {"C:\\one\\two\\three\\with.this.file.txt"};
strcpy(buf, return_base_filename(buf,'\\'));
 strcpy(buf, return_base_filename(buf,'.'));