C言語：const修飾子の使い方など

Question

前提

文字列単位で英大文字を小文字に変換する関数を自作しましたが、今一つ自信がありませんのでご助言を賜りたいです。

実現したいこと

文字単位のtolower()関数の文字列版を作るのが目的です。

• 入力する文字列をconst char *型の引数とする。
• 出力する文字列をchar *char型の戻り値とする。

発生している問題・エラーメッセージ

いろいろstr関係のライブラリ関数の仕様を見ていて、なんとなく引数にconst修飾子をつけたのですが、出力文字列を受けるsoutの初期化でsout = sin;とすると、

異なるconst修飾子です。

という警告が出ました。そこでなんとなく、sout = (char *)sin;とキャストすると、警告なくコンパイルでき、実行結果も問題ありませんでした。
しかし、上記のように「なんとなく」で進んできたので気持ち悪く、今一つ自信がありません。
ポインタを返す関数を自作するのは初めてなので、いろいろ不安です。
オーソドックスというか、エレガントというか、問題のないコーディング流儀をご教授いただきたいです。
よろしくお願いいたします。

該当のソースコード

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

/* 文字列単位で英大文字を小文字に変換する関数 */
/* tolower()関数を使用する                    */
char *tolowerstr(const char *sin);

int main(void)
{
	char line[256];
	
	printf("文字列単位で英大文字を小文字に変換します。\n\n");
	printf("英文字列を入力してください。\n");
	fgets(line, sizeof line, stdin);
	line[strlen(line) - 1] = '\0';
	
	printf("↓\n");
	printf("%s\n", tolowerstr(line));
	
	return EXIT_SUCCESS;
}

/* 文字列単位で英大文字を小文字に変換する関数 */
/* tolower()関数を使用する                    */
char *tolowerstr(const char *sin)
{
	int i = 0;
	char *sout;
	sout = (char *)sin;
	
	while(sin[i]) {
		sout[i] = tolower(sin[i]);
		i++;
	}
	
	return sout;
}

実行結果

文字列単位で英大文字を小文字に変換します。

英文字列を入力してください。
ABC-Z
↓
abc-z

補足情報（FW/ツールのバージョンなど）

コンパイラ：Visual Studio 2022、OS：Windows10

Answer 1

英大文字を小文字に変換する関数

これだけだと，ふわっとしすぎていて話が不十分．
…っていうのが本質というか根っこのところなんじゃないかな．
その関数の機能というのが，

• 入力データそのものを書き換えてしまう
• 入力データとは別に，結果データを作る

のどっちなのか？

前者側なのであれば「 const とか言っていること」が間違っているという話になるのだし，
後者側なのであれば，じゃあ結果データってのはどこに作るのか？っていう話が当然あるわけで，それと照らし合わせれば「sout = sin;　とかいう実装は話としておかしくね？」っていう話になるよね．こっちの場合，入力データ側が const なのは正当だ．

Comments

[fana]

> ポインタを返す関数を自作するのは初めて

とか言ってるけど，そもそもその関数の戻り値は何なの？（何を指すポインタなの？）ってのがちゃんと定まっているのか？　っていう．
（入力データそのものを書きかえる関数なのであれば，戻り値は別に無くてもいいよね．「いやいや利便性のための戻り値なんです」とか言えるのであればそもそも「不安」なんて抱かないハズ）

[fana]

「入力データとは別に，結果データを作る」タイプの関数なのであれば結果の格納先が必要だよね．
少なくとも入力データの文字数を格納できるだけの出力バッファが要る．
じゃあそれは誰が用意するのか？
関数の外側で用意するのであれば，そのバッファのことを関数にどうにかして「ここに結果を入れてね」って知らせる必要があるよね．
そうではなくて関数が勝手に用意する？　どうやって？　「メモリを動的に確保して返すぞ」とかいうならその旨をしっかり定めないといかんでしょ．

[fana]

…つーわけで，引数やら戻り値やらのあるべき形ってのは，関数の仕様次第なのだから，
何はともあれ関数の仕様をしっかり固める（決める）ということをする　→　で，その仕様通りに実装する．
っていう順序でやるべき．
実装の段になってから「なんとなくキャスト」とか言い出す余地はそもそもないハズ．
（あと，キャストなんていう最上級危険物を「なんとなく」で扱っちゃダメです．）

[sanshirou]

fanaさん、ご回答ありがとうございます。ベストアンサーをつけてから見ていなくて、ご返事が遅くなってしまいました。

> 引数やら戻り値やらのあるべき形ってのは，関数の仕様次第なのだから，
> 何はともあれ関数の仕様をしっかり固める（決める）ということをする　→　で，その仕様通りに実装する．
> っていう順序でやるべき．
> 実装の段になってから「なんとなくキャスト」とか言い出す余地はそもそもないハズ．
> （あと，キャストなんていう最上級危険物を「なんとなく」で扱っちゃダメです．）

いろいろためになる知見をありがとうございました。まずは関数の仕様を明確にする、それからですね。よくわかりました。ありがとうございました。

Answer 2

そもそも、constってのは変化させないという修飾子なので、そこにconstをつけるのは間違いです

Comments

[sanshirou]

constは必要なかった、ということですかね。ありがとうございました。

[y_waiwai]

sin の内容を変換して変更するのですから、constをつけてはいけません

[Zuishin]

> 入力する文字列をconst char *型の引数とする。

条件を満たさなくなるので、この回答は誤りです。

Answer 3

strcpyみたいにchar* tolowerstr(chardst, char constsrc)にするか、strdupみたいにdstをmallocして返す関数にする話では。

Comments

[sanshirou]

ご回答ありがとうございます。しかしながら、意味が理解できませんでした。