Cでの文字列パディング

StringPadRight（ "Hello"、10、 "0"）-> "Hello00000"を実行することになっているこの関数を作成しました。

char *StringPadRight(char *string, int padded_len, char *pad) {
    int len = (int) strlen(string);
    if (len >= padded_len) {
        return string;
    }
    int i;
    for (i = 0; i < padded_len - len; i++) {
        strcat(string, pad);
    }
    return string;
}

それは機能しますが、いくつかの奇妙な副作用があります...他の変数のいくつかが変更されます。どうすればこれを修正できますか？

printfがパディングを行うことを知っておくと役立つ場合があります。フォーマット文字列として％-10sを使用すると、10文字の長さのフィールドに入力が正しくパディングされます。

printf("|%-10s|", "Hello");

出力します

|Hello     |

この場合、-記号は「左揃え」を意味し、10は「フィールド内の10文字」を意味し、sは文字列を揃えていることを意味します。

Printfスタイルのフォーマットは多くの言語で利用可能であり、Web上に多くの参照があります。これは、フォーマットフラグを説明している多くのページの1つです。いつものように、WikiPediaのprintfページも役に立ちます（ほとんどの場合、printfがどれだけ広く普及したかについての歴史のレッスンです）。

'C'の場合、カスタムのパディングが必要な場合は、malloc（）や事前フォーマットを必要としない[s] printfの代替（より複雑な）使用法があります。

秘訣は、％sに '*'長さ指定子（最小および最大）と、潜在的な最大長までパディング文字で埋められた文字列を使用することです。

int targetStrLen = 10;           // Target output length  
const char *myString="Monkey";   // String for output 
const char *padding="#####################################################";

int padLen = targetStrLen - strlen(myString); // Calc Padding length
if(padLen < 0) padLen = 0;    // Avoid negative length

printf("[%*.*s%s]", padLen, padLen, padding, myString);  // LEFT Padding 
printf("[%s%*.*s]", myString, padLen, padLen, padding);  // RIGHT Padding 

「％*。* s」は、左または右のパディングの必要性に応じて、「％s」の前または後に配置できます。

[####Monkey] <-- Left padded, "%*.*s%s"
[Monkey####] <-- Right padded, "%s%*.*s"

PHP printf（ここ）は、％s形式で、一重引用符（ '）の後にカスタムパディング文字を使用して、カスタムパディング文字を指定する機能をサポートしていることがわかりました。
printf("[%'#10s]\n", $s); // use the custom padding character '#'
生成：
[####monkey]

入力文字列にすべてのパディング文字を保持するのに十分なスペースがあることを確認する必要があります。これを試して：

char hello[11] = "Hello";
StringPadRight(hello, 10, "0");

hello最後にヌルターミネータを説明するために、文字列に11バイトを割り当てたことに注意してください。

「Hello」を渡した引数は、定数データ領域にあります。char * stringに十分なメモリを割り当てていない限り、他の変数にオーバーランしています。

char buffer[1024];
memset(buffer, 0, sizeof(buffer));
strncpy(buffer, "Hello", sizeof(buffer));
StringPadRight(buffer, 10, "0");

編集：スタックから定数データ領域に修正されました。

わかりました、理にかなっています。だから私はこれをしました：

    char foo[10] = "hello";
    char padded[16];
    strcpy(padded, foo);
    printf("%s", StringPadRight(padded, 15, " "));

ありがとう！

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

using namespace std;

int main() {
    // your code goes here
    int pi_length=11; //Total length 
    char *str1;
    const char *padding="0000000000000000000000000000000000000000";
    const char *myString="Monkey";

    int padLen = pi_length - strlen(myString); //length of padding to apply

    if(padLen < 0) padLen = 0;   

    str1= (char *)malloc(100*sizeof(char));

    sprintf(str1,"%*.*s%s", padLen, padLen, padding, myString);

    printf("%s --> %d \n",str1,strlen(str1));

    return 0;
}

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

int main(void) {
    char buf[BUFSIZ] = { 0 };
    char str[] = "Hello";
    char fill = '#';
    int width = 20; /* or whatever you need but less than BUFSIZ ;) */

    printf("%s%s\n", (char*)memset(buf, fill, width - strlen(str)), str);

    return 0;
}

出力：

$ gcc -Wall -ansi -pedantic padding.c
$ ./a.out 
###############Hello

関数自体は私にはうまく見えます。問題は、文字列にその数の文字を埋め込むのに十分なスペースを割り当てていないことである可能性があります。将来size_of_string、関数に引数を渡すことでこの問題を回避し、長さがサイズより大きくなりそうなときに文字列を埋めないようにすることができます。

このスレッドで元の質問を形成する関数で間違いなく間違っていることの1つは、パラメーターとして渡された文字列リテラルの末尾に余分な文字を連結していることです。これにより、予測できない結果が生じます。関数の呼び出し例では、文字列リテラル「Hello」がプログラムにハードコードされているため、おそらくその末尾に連結すると、危険なほどコードが上書きされます。元の文字列よりも大きい文字列を返したい場合は、動的に割り当てることを確認し、完了したら呼び出し元のコードで削除する必要があります。

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


void padLeft(int length, char pad, char* inStr,char* outStr) {
    int minLength = length * sizeof(char);
    if (minLength < sizeof(outStr)) {
        return;
    }

    int padLen = length - strlen(inStr);
    padLen = padLen < 0 ? 0 : padLen;

    memset(outStr, 0, sizeof(outStr));
    memset(outStr, pad,padLen);
    memcpy(outStr+padLen, inStr, minLength - padLen);
}