文字列の変換や連結など

Question

下の例の通りの関数を作りたいと思っています。return文の箇所の間違いがわからないので、教えてください。

例
accum("abcd") -> "A-Bb-Ccc-Dddd"
accum("RqaEzty") -> "R-Qq-Aaa-Eeee-Zzzzz-Tttttt-Yyyyyyy"
accum("cwAt") -> "C-Ww-Aaa-Tttt"

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

char *accum(const char *source)
{
  int n;
  char str1[255];
  char a = '-';
  
  for(n = 0; n < (int)strlen(source); n++)
  {
   
    sprintf(str1,"%c",toupper(source[n]));
    
    if(n > 0)
    {
      for(int m =0; m < n; m++)
      {
        sprintf(str1,"%c",tolower(source[n]));
      }
    }
    
        sprintf(str1,"%c",a);
  }
  
   return str1;
}

エラーコード

solution.c:9:38: warning: 'sizeof source' will return the size of the pointer, not the array itself [-Wsizeof-pointer-div]
  for(n = 0; n < (int)(sizeof source / sizeof source[0]); n++)
                       ~~~~~~~~~~~~~ ^
solution.c:12:5: warning: implicitly declaring library function 'sprintf' with type 'int (char *, const char *, ...)' [-Wimplicit-function-declaration]
    sprintf(str1,"%c",toupper(source[n]));
    ^
solution.c:12:5: note: include the header <stdio.h> or explicitly provide a declaration for 'sprintf'
solution.c:25:11: warning: address of stack memory associated with local variable 'str1' returned [-Wreturn-stack-address]
   return str1;
          ^~~~
3 warnings generated.
2 warnings and 6 errors generated.

修正版

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

char *accum(const char *source)
{
  int n;
  char str1[255];
  char a = '-';
  
  for(n = 0; n < (int)strlen(source); n++)
  {
   
    sprintf(str1,"%c",toupper(source[n]));
    
    if(n > 0)
    {
      for(int m = 0; m < n; m++)
      {
        sprintf(str1,"%c",tolower(source[n]));
      }
    }
    
        sprintf(str1,"%c",a);
  }
  
   return str1;
}

修正版のエラー

solution.c:27:11: warning: address of stack memory associated with local variable 'str1' returned [-Wreturn-stack-address]
   return str1;
          ^~~~
1 warning generated.

Comments

[thkana]

エラーメッセージの部分もコードと同様に
```
```
で囲ってください。

[Merrifield]

エラーメッセージもコードのような書き方にするべきだったのですね。ありがとうございます。

Answer 1

修正版のエラー

solution.c:27:11: warning: address of stack memory associated with local variable 'str1' returned [-Wreturn-stack-address]
return str1;
^~~~
1 warning generated.

char str1[255]; は関数 accum のローカル変数で自動変数です。
関数に入った時に領域が確保され、関数から出た時に領域は無効になります。
その無効になった領域へのポインタが返されても、関数の呼び出し元では
正しい結果を参照することができません。

結果を入れてほしい領域は、関数の呼び出し元で用意し、関数を呼び出すときに
それも渡すようにしましょう。

#include <ctype.h>  // toupper, tolower
#include <string.h> // strlen
#include <stdio.h>  // printf

char *accum(const char *source, char *str1, int size)
{
	int n = strlen(source);
	if (n*(n+3)/2 > size) return NULL;
	char *p = str1;
	for (int i = 0; i < n; i++) {
		int c = (unsigned char)source[i];
		if (i > 0) *p++ = '-';
		*p++ = toupper(c);
		for (int j = 0; j < i; j++) *p++ = tolower(c);
	}
	*p = '\0';
	return str1;
}

int main(void)
{
	char buf[256]; 
	printf("[%s]\n", accum("abcd", buf, 256));
	printf("[%s]\n", accum("RqaEzty", buf, 256));
	printf("[%s]\n", accum("cwAt", buf, 256));
}

Comments

[Merrifield]

お答えいただきありがとうございました。
n*(n+3)/2 というのは、どういう意味なのでしょうか？

[Merrifield]

続けての質問で申し訳ないですが、
char *str = (char*)malloc(len * (len + 1));とすれば

return str;
のように戻り値としてと書くことができますか？

[kazuma-s]

"abcd" は 4文字、変換後の "A-Bb-Ccc-Dddd" は 13文字、文字列の終端の '\0' も含めると 14バイトの領域が必要です。n*(n+3)/2 は変換後に必要となるバイト数です。size がそれより小さいと変換結果を格納できません。

malloc で領域を確保すれば問題ありませんが、あとでどこかで free しないと、メモリをどんどん食いつぶしていきます。

[Merrifield]

理解できました。答えていただきありがとうございました。

[kazuma-s]

malloc(len * (len + 1)) では、len = 4 のとき、20バイト確保してしまいます。

[Merrifield]

そうですね。教えていただいてありがとうございます。

Answer 2

エラーメッセージを追いましょう。

warning: 'sizeof source' will return the size of the pointer,

source は、accum()の引数なので、const char * 配列のサイズは取得できません。ここでは、strlen()あたりを使うのが、よくある方法。

sprintf(str1,%c,toupper(source[n]));

こちらの警告では、stdin.hでは、int (char *, const char *, ...)となっていると言ってます。 2番目の引数を見ると、 %c となってます。ここは、const char * であるべき。
以下、ほぼ同様。

Comments

[Merrifield]

回答ありがとうございます。

const char * 配列のサイズは取得できないというのは、なぜなのでしょうか？

また、sprintfに関しての警告は、#include <stdio.h>を追加すれば問題ないでしょうか？