C言語なら俺に聞け 150

2019/02/06(水) 13:39:03.21

↑
次スレを作る時は上記1行をコピーして2行に増やして必ず1行目に入るようにしてください。

C言語の話題のみ取り扱います C++の話題はC++スレへ
質問には最低限の情報(ソース/コンパイラ/OS)を付ける
数行で収まらないソースは以下を適当に使ってURLを晒す
https://paiza.io/
https://ideone.com/
http://codepad.org/

C11
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1570.pdf

C99
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1256.pdf
http://kikakurui.com/x3/X3010-2003-01.html

C FAQ 日本語訳
http://www.kouno.jp/home/c_faq/

JPCERT C コーディングスタンダード
https://www.jpcert.or.jp/sc-rules/

※前スレ
C言語なら俺に聞け 149
https://mevius.5ch.net/test/read.cgi/tech/1540731704/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured

2019/02/06(水) 13:43:35.56

ミスってる
!extend:checked:vvvvv:1000:512
を書き込み内容にするためにもう一行書かなければならないのね

2019/02/06(水) 13:53:52.35

そうしろって書いてあるじゃん

2019/02/06(水) 14:08:04.08

違うな
シェバンに類似する特殊な一行目は書き込みすると失われる、だろ
だから次の手順説明のために同じものをもう一行増やす

しかも「必ず1行目に入るように」って書いてあるから二行に増やしても先頭に空白行があると受理されないハズだ
もう一行書くだけじゃダメだからな

2019/02/06(水) 14:19:35.82

>>1 乙です、ドンマイ

2019/02/06(水) 16:08:05.73

では、続きをしようか

986 名前：デフォルトの名無しさん (ﾜｯﾁｮｲ df01-vS77)[sage] 投稿日：2019/02/06(水) 04:57:27.49 ID:eUM592Vs0
ループから抜けるときに goto使ったらダメなんですか？

for (...) {
for (...) {
if (...) goto LOOP_EXIT;
}
}
LOOP_EXIT:

こういう感じで書いたら先輩に怒られました。
フラグ見てループ抜けるように修正しろと。
指示されたので従いますけど、どうしてダメなのかよく分かりません。

どなたか納得するような理由をご存知でしょうか？

2019/02/06(水) 16:09:43.98

若き日の俺は、gotoの他にフラグも無闇に使うなと教わったものだが
フラグはいいってことになったのは、いつからだ？　どういう経緯でだ？

2019/02/06(水) 16:21:58.32

Cにはラベル付きbreak文って無いんだね初めて知った

Javaのラベル付きbreak文
loop_exit: for(...) {
　for(...) {
　　if (...) {
　　　break loop_exit;
　　}
　}
}

まぁJavaでもあまり使ったことないけど…。

2019/02/06(水) 16:41:15.95

#define break goto
みたいなしょーもないことしても本質的に何も変わっちゃいねえ

2019/02/06(水) 18:02:39.46

gotoよりフラグがやばいよな。

2019/02/06(水) 19:00:59.18

goto があっても
いいじゃないか
C言語だもの

2019/02/06(水) 19:48:43.15

goto文そのものが悪いわけではない。いまだにgoto文の使用そのものがよくないというやつらのせいでおかしくなっている。

goto文で前のif文内に飛んでみたり、ループのなかに飛んでみたりと、乱用して人間には理解しづらいスパゲティプログラムを作るなという話なのに。

continue文だって実体はgoto文で、コンピュータにとってはメモリ上のマシン語を順番に実行しない場合はすべてメモリのアドレスのジャンプでgoto文。

2019/02/06(水) 20:21:21.69

センサーで取得した計測値を数値では無く文字列で返す関数を作ろうと思ってる。
どんな計測結果が得られるか事前には分からないため文字列のサイズは関数内で
mallocを使ってリサイズする予定。

このとき測定結果が格納された文字列を関数から渡してもらうのに、

a) 引数に文字列のポインタのポインタを渡す
void sokutei( char** buff )
{
・・・
*buff = (char*)malloc( sizeof(char)*length );
・・・
}

と、

b) 関数の返値に文字列のポインタを指定する
char* sokutei(void)
{
・・・
char* buff = (char*)malloc( sizeof(char)*length );
・・・
return (buff);
}

の二つの方法があるけど
一般的にはどっちの方法がお薦め？

2019/02/06(水) 20:30:53.85

>>12
機械語が分岐命令になる高級言語のステートメントを全否定というのは
構造化とはまったく違う話だな

2019/02/06(水) 20:31:48.22

>>13
別にどっちでも

2019/02/06(水) 20:34:02.27

>>15
そ、そう・・・

2019/02/06(水) 20:50:04.50

>>13
> センサーで取得した計測値を数値では無く文字列で返す関数を作ろうと思ってる。
これ自体が糞。
計測値は生で保持して、表示する時だけ文字列に変更しろ。
そうすればprintfだけで行けるだろ。

無駄に可変長のバッファなんて持とうとするから無駄に面倒になる。
初心者だから分からないのだと思うが、そもそも複雑にならないように全体像をまず組むんだよ。

2019/02/06(水) 21:37:14.88

voidにするならポインタ返した方がいいと思うよ。
正しさでいうと、既存のライブラリ使ってシリアライズするのが正しい。

2019/02/06(水) 22:39:39.17

関数の戻り値は処理の成否のみを返す。
正常終了なら０、異常終了なら非ゼロ。
入出力は引数で渡す。

と、決めて統一するのが良いと思う。

2019/02/06(水) 22:43:12.36

助言ありがとう
確かに関数の戻り値は処理の成否のみに限定するのはスッキリしていいかも

> 文字列で返すのが糞

確かにその通りで(；^ω^)

2019/02/06(水) 23:32:22.18

重要なのは統一するってこと。

標準ライブラリには戻り値に二つの意味を持たせているのが多いから、個人的に好きじゃない。

fopen() 成功時はファイルポインタを返し、失敗時はNULLを返す。

strtol() 成功時は変換結果を返し、失敗時は0を返す。
これじゃ変換結果が0なのか、失敗して0なのか分からない。
で、errnoを見ろってことになってる。

なんでこうなってるかというと、戻り値に処理の成否と結果の二つを持たせているから。
だったら、分ければいいじゃんって話。

2019/02/06(水) 23:38:05.98

>>20
多分相場が分かってないのだと思うけど、
正しく初心者の自覚があるのなら、ある程度縛りプレイで行った方がいい。
C言語なんて現役で50年来てるし、そのセンサーだって民生品なら誰かが同様に使ってる。
それでもそれ用のライブラリがないのなら、それはそもそもそういう使い方をする奴がこれまで居なかったということ。
つまり、そのやり方が間違っている、と見ていい。

初心者が自前でライブラリみたいな物をいきなり組まなければならない時点で設計が間違ってる。
それは君らがよく誤用している「無能な働き者」に該当する。
（ただし繰り返すが、Webでの多くは誤用されているので注意）

2019/02/07(木) 00:06:12.90

副作用がメインならともかく、欲しいものを素直にreturnすればいいよ。

2019/02/07(木) 00:29:49.70

strtol()は変換できなかった時の戻り値が曖昧だから0やらLONG_MAX、LONG_MINに固定してるだけだと思ってたんだけど

2019/02/07(木) 00:29:57.91

バカかコイツラNaNが出たらどーすんだよ
文字列が正しいだろJK

2019/02/07(木) 05:23:13.49

>>20
C言語の戻り値って、エラーのみで返して、正の値って無いから、C++だな、C++言語に行くように

2019/02/07(木) 06:44:41.17

>>19
それやだ
んなことするくらいならsetjmpだ

2019/02/07(木) 07:00:15.05

>>13
ほんとうはaのときは引数にエラーがいるし、bのときは戻り地を
エラーにしないといけないからあまり変わらない

2019/02/07(木) 17:48:23.42

くっそ初心者なんだがガウスの消去法でn次連立方程式を解くようにしたいんだがn次を入力させてからやるにはどうやりゃいいんだ？

2019/02/07(木) 17:57:54.86

行列の格納先に配列のかわりにmallocで動的に確保する。

2019/02/07(木) 21:03:55.08

n以外はどうやって入力させるつもりだったの？

13 (ﾜｯﾁｮｲ 82ab-/WZR) · 2019/02/07(木) 23:25:09.01

a)の方法。空の文字列を渡そうとすると失敗するっぽい

char** str;
sokutei( str );

これやっても有意な文字列はstrに返ってこなかった。
もしこれやるならb)の方法じゃ無いとダメかも↓

cha* str;
str = sokutei();

2019/02/07(木) 23:38:06.10

初心者だから間違ってるかもしれんけど
外でメモリ確保してないからじゃない？
文字列を関数内で作ってたら関数処理終わった後は文字列がそのまま残るかは保証されなかったような気が
外で配列宣言して関数に渡して、関数内でそこにmemcpyさせるとか？

2019/02/07(木) 23:41:01.38

>>32
それやるなら、こうじゃね?

char* str;
sokutei(&str);

2019/02/07(木) 23:59:08.20

あぁmallocしてんのね…
aは何でchar**なんだろう

2019/02/08(金) 00:24:25.67

ちゃ

2019/02/08(金) 07:47:48.49

>>29
C99以後ならVLAという手もある
int n;
scanf("%d", &n);
double mat[n][n + 1];

**デフォルトの名無しさん** (ｽﾌｯ Sd22-QV4H) · 2019/02/08(金) 08:37:48.99

>>34
あぁなるほど

char **str;
sokutei(str); // 関数の引数にstrのアドレスがコピーされるだけ

だから関数内で代入してもコピーに再代入してるだけか

char *str;
sokutei(&str); // str自体のアドレスを渡す

これなら外のstrにmallocで確保したアドレスが入るのね

2019/02/08(金) 09:08:14.73

渡した引数を書き換えてくれるインターフェースは
scanf() の使い方と同じくするしかない

2019/02/08(金) 13:03:58.19

Cに未来はあるのか

◆QZaw55cn4c (ﾜｯﾁｮｲ e247-3ywI) · 2019/02/08(金) 21:04:27.25

>>37
VLA は廃れていく方向のダメな子じゃないですか？c++ からも linux からも追放されたようですし