sizeof(char)が必ず1でも、省略すべきではない
ではなく
malloc(strlen(s)+1)
と書くような糞コードばかり見て育った、
悪しき慣習を引きずった人は引退すべし
YOU死んじゃいなYO
こういうのが孤軍奮闘して、単に議論がダラダラと続いているだけなのに、最終的には
「宗教論争」と言い出して「どちらでもいい」ということにしてしまうんだよなぁ。
2chの片隅で、そんなことをしても、なにがどうなるってわけでもないんだけど、
弱い人は、精神防衛せずにはいられないんだろうなぁ。
それもいいかもね。
ただ、charとワイド文字の切り替えを意識してるコードだって知らない人がみたら、意図をつかみにくいかも。
切り替えがなくとも、
sの指す先の型を人間が手作業で書くのは良くないので、
ミス防止のためには、コンパイラに仕事させたほうがいいと思う。
malloc(strlen(s)+1)
のようなコードを書くこと自体が間違い。
最低でも関数でラップすべきだし、
C++ならstd::basic_stringを使うべきだろう。
char固定なら、いらないじゃん。
ぜんぜんOK
それにラッパー書かなくても、フツーはstrdup()使う。
char固定であることを人間が保証してやるなんて馬鹿馬鹿しい。
>>12
可哀想に。恵まれない職場にいるんだね。
別にいいじゃんねぇ
多バイト文字の文字数を求めるなら strlen を使っちゃダメだろ。
ええ? malloc()のラッパーって話じゃなくて、malloc(strlen(s)+1)のラッパーってことだよね?
職場のヘンなコードばっかり見てないで、いろいろコードを見たほうがいいと思われ。
strlen()使うんだから、char以外は使えないじゃん。
文字列複製用のメモリ確保だと思われるから、長さはバイト数でいいんじゃね?
mallocの引数は「バイト数」
strlenの返り値は「文字数」
保証は無い。(まだそんなマシン動いてるかどうかはともかく)
100%保障。
sizeof(char)が1でないコンパイラがあったら捨てたほうがいい。
sizeof( char ) != 1の環境は見たこと無いけど・・・
だからcharは1バイトだって。
何も書いてない時の ANSI 縛りは、暗黙の了解事項だろ。
そうでないとなんでもありありになっちゃうし。
ちがう人と交代したのかな?
箱の中に複数本の牛乳パックが入っている。
本数を問い合わせる関数があるとしよう。返り値は本数型だが、実はintのtypedefだ。
箱の中の牛乳パックを開けてタンクに入れようとする。
同じintのtypedefだからといって、
リットル型の引数に、本数型を渡していいものだろうか。
俺はダメだと思うね。
たとえ1本1Lと決まっていても、本数型からリットル型に変換する段は踏むべきだ。
hoge = (type*)malloc(sizeof(type)*size) ;
なんて書いてる人いるの?
初心者だって、
#define MALLOC(type, size) ((type)*)malloc(sizeof(type)*(size))
くらいして、
hoge = MALLOC(char, strlen(s)+1) ;
って書くだろ?
だいたい、いまどきCオンリーなんて、なにか理由があるときくらいなもので、
普通はC++だから、文字列クラスを使うだろう。
いまだにチマチマとstrlenとかstrcpyとかやってるから、
つまらないバグやセキュリティホールを作り込んでしまうんだよ。
strlen()のリターン値は文字数==バイト数で、malloc()引数もバイト数じゃん。
強いtypedefのあるD言語をお使いください。
> 初心者だって、 ・・・って書くだろ?
それ、たとえば誰が書いてるの?
職場の先輩とか?
>>31
これはちょっと全体的に反対。
＞#define MALLOC(type, size) ((type)*)malloc(sizeof(type)*(size))
そんな訳の分からないマクロ作るぐらいなら、素直にcalloc使うよ。
＞普通はC++だから、文字列クラスを使うだろう。
そんな普通は無いだろ。
名前欄消し忘れた・・・。
是非スルー頼む。orz
全体的に賛成だがcallocになるのか？ラッパー関数とかじゃなくて？
ｗｗｗ
たまたま 文字数==バイト数 なだけじゃん。
「規格上」文字数==バイト数。
たまたまじゃない。
Cではキャストは不要だがC++では必要。
まぁ、C++でmallocを使うこと自体がおかしいが。
たまたま「規格上」一致する組み合わせ
>>31のマクロだと「型と要素数を指定してメモリを獲得する」機能が欲しそうだったんで、
指定の仕方が近いcallocを持ってきただけだよ。
>>1やらの話なら、文字列長渡してメモリのアドレス返すラッパならアリじゃないかなぁ。
個人的には面倒だからやらないけども。
たまたまじゃないよ。
あつかってる文字セットが収まる範囲を1バイトにしてるんじゃん。
すでに_strdupがあるもんな。
「文字列の尻には\0があるから、+1文字分の領域が必要」
ということに対処するコードは一ヶ所だけにまとめておくべきで、
プログラム全体に散在させるべきではないよね。
それが後で変るかどうかは関係ない。
あちこちにmalloc(strlen(s)+1)と書けばいいと言う人間は、
構造化プログラミングを否定していると思う。
牛乳パック1本が1リットルと決まっているからといって、
リットル数を与えるべきところに本数を与えていいと思うか?
malloc(strlen(s) + sizeof(char)) って書くってこと?
恒久的にってのもだけど、論理的にも一文字1バイトだし。
(引数の型がTCHARとやらに変わるなど、思い込み)
無知なのに口だしてごめんなさい。
strlenがchar固定。sizeof(char)==1バイトが、規格から明らかならば、
sizeof(char)はいらないよね。
察するに>>40,>>43は「バイト数」と言う言葉と、「文字数」という言葉を
厳格に分けたいというところかな？
それに対して、１バイト==sizeof(char)は規格上定義されているから
分けて考える必要はないという反論。ってことですね。私はこっち派。
>>47,>>49
このスレで議論の中心になってるのは、+1ではなくて、
>>1の１行目、"sizeof(char)*" -charのサイズを掛けている-部分だと思うよ。
どこから、そういう発想がでてくるのだろ。
size_t CalcRequiredMemoryForString(const char* s) {
return sizeof(char)*(strlen(s)+1) ;
}
もっと細かく分けて
size_t ScanAscizStringLength(const char* s) {
return strlen(s)+1 ;
}
size_t CalcRequiredMemoryForString(const char* s) {
return sizeof(char)*ScanAscizStringLength(s) ;
}
とかでもいい。
そしたら、
char* s2 = malloc(CalcRequiredMemoryForString(s)) ;
と書けるようになるっしょ。
(1)はラッパを書く、(2)は書いたら弊害があるのか？ でダメ？
strlen()+1だけの関数とか、そんなのつかってるのオタクの職場くらいですよ。
壮大な釣り?
＞size_t ScanAscizStringLength(const char* s) {
string lengthはどう考えてもstrlen()の戻り値そのままだろ。
stringに形容詞がつくだけで長さが変わるなんて気色悪い。
getStringSize()って名前なら判らんでもないが。
で、malloc(sizeof(char) * stringSize)なら「あ〜あ、一一書いているよ」で済むが、
malloc(CalcRequiredMemoryForString(string))なんて書いていたら「戯け」の一喝だな。
# CalcRequiredMemoryForDoublePrecReal()なんてのも作るのか?
ふつう文字列クラス使うよな。
strlenとかstrcatとかstrcpyなんて直接使わないもの。
ふつうC++なので、やるとしてもテンプレート関数だな。
mallocとか使ってることから、ここはC限定ではないのかな？
>>53
そこまで書くんだったら >>49 程度で済ました方が良いと思う。それと、その関数名だと長すぎて使いたくない。
http://www.kouno.jp/home/c_faq/c7.html#8
一般的には、どっちでも良いってよ。
無知やうっかりミスによってバグを生じさせない
ということ。
+1が必要なことを知らない人もいるし、
+1を書き忘れることもあるし、
sizeとlengthを書き間違えることもある。
じゃぁどうすればいいか。
Cを捨ててC++で文字列クラスを使うべき。
C++でもmalloc使う人いるんだよねぇ。
> >>49 程度で済ました方が良いと思う。
strlen(s) + sizeof(char)
これこそ意味不明だねぇ。
1文字分ってことなら、
sizeof(char)
ではなく
sizeof(char)*1
としないとねぇ。
そうするなら
sizeof(char)*(strlen(s)+1)
になるわなぁ。
> その関数名だと長すぎて使いたくない。
いちいち手でタイプする人いるんだよねぇ。
ミスタイプしたりするからコピペが基本よ。
長い数式を書かない人なんだね・・・
まあ、常識的に考えてこんな関数つかわねーよな。
void *getBufByLen(int length)
{
return malloc(length + 1);
}
まぁこのぐらいのラッパなら良いんじゃないの？
strlen(str)+1がどのぐらい出てくるかにもよるんだろうけどさ。
どうしても「+1」をソースコードから出来るだけ消したいなら、俺なら
#define LEN2SIZE(x) ((x)+1)
でも作って、LEN2SIZE(strlen(str))で書くか。
したら、今度は副作用がどうとか言われんのかねぇ。
そこまでして+1を消すと、かえって読みにくくなるよ。
strlen(s) + 1 と直に書くのがいちばん素直なCのコード。
技巧に走りすぎてもよくない。
そうは思うけどさ。
たまに上司の好みで結構色々言われんのよ。
（俺も多分好みで妙なコード書いてるとは思うけど）
んで、そこで激論しても時間無駄だから、
まぁ良いかと思えることは従うことにしてる。
今のところ「+1消せ」は言われたことないけど、
そういう指示が出たらこうするかな、ってぐらいのモン。
ごもっともです。
ところで、もともとの質問主の環境は VC++6.0 みたいですね。
素直にC++の文字列クラスを使うのがいいのかな...
getBufByLenではなくallocBufByLenにしたほうがいいと思う。
>C++でもmalloc使う人いるんだよねぇ。
それは酷い。
Cだけなら sizeof(char)==1 が仕様で定義されてるけど
http://www11.big.or.jp/~magmell/science/watchfiles/semi01.jpg
この板では空気を読む必要は無い。
#include <stdio.h>
int main()
{
printf("%d\n",sizeof(char));
return 0;
}
をCでコンパイルしてみたか？
C++でもコンパイルしてみたか？
これはアブラゼミだね
CとC++でsizeof(char)は絶対１なのは仕様で保証してるんだけど・・・
ってことでしょ？
もう帰れ無能。
Cの規格での「文字」の定義は「1バイトに納まるビット表現」。
規格書ちゃんと嫁。
文字っつーかcharな。単に文字とか言うとまた噛み付いてくるで。
>単に文字とか言うと
規格に於ける定義の話をしてるんだが。
それならきっちり「文字型」といった方が誤解ないよねとか
文字とだけ言うとシングルバイト文字とマルチバイト文字を
ごっちゃにするやつが出てくるから 規格話するなら正確に
みたいあｎ
関係ない話なんだが、
sizeof演算子の結果の型はsize_tだけど、こういう場合はintにキャストしなくていいの？
C++だろうとmallocは使えるでしょう?
mallocしたものをdeleteしたりしなければ。
charが1バイトなのは当然として、
1を掛けるのを省略して書いて良いのかどうか
という問題だろう。
使えるけど、new があるのに
>mallocしたものをdelete
する危険を冒してまで態々使う意味が判らない。
>>86
「省略しちゃいけない」とする理由が
「文字数とバイト数は意味が違うから」であれば、
規格上は「文字」＝「1バイト文字」であり
「文字数＝バイト数」であるから
「省略しても問題ない」よね？という確認。
どうすべきかとなると宗教論争になりそうだ
C FAQにあるよ
> 「文字数＝バイト数」であるから
それは数値の一致であって、意味の一致ではない。
あくまでも「1バイト文字」の「文字数」なのだから。
省略してもプログラムは意図した通りに動くが、
プログラムが動けば何だっていいというのは間違い。
はあ…
>それは数値の一致であって、意味の一致ではない。
意味の一致だってばさ。頭悪いなあ。
「規格では、strlen() はバイト数を戻す」
これで納得した？
>>87
new使ったってその危険はdeleteとdelete[]を間違える危険にかわるだけじゃね？
みんな暇なの？
だろ?
格納するのに必要なバイト数
ではないのは確かだな。
アルファベット＝レターと非アルファベット記号＝キャラクターとストリングの意味からして、strlenが間違い。
strlenという関数名なんだろ。
100〜のスレッドの続きを読む