C言語の設計ミスった危険な関数トップ10決めようぜ (215ﾚｽ)
上下前次1-新
82(2): 2019/12/16(月)16:51 ID:arH1mwfY(2/13)調 AAS
void print_data(void) {
for (i = 1; i <= 3; i++) {
for (j = 1; j <= 3; j++) {
printf("%d, %d\n", i, j);
if (i * j > 3) return;
}
}
}
print_data();
outer より後に処理が続く場合
printf("complete\n");
83: 2019/12/16(月)16:55 ID:arH1mwfY(3/13)調 AAS
"短い処理"をいちいち関数にしなくて済むから、break 2とかはあってもいいが、
長い処理は関数にしてないとろくにテストできないんだからgotoなんていらない
84: 2019/12/16(月)19:03 ID:4DgO9d+i(1)調 AAS
じゃあ>>57もいらないね
85: 2019/12/16(月)19:09 ID:lsrBHsJG(1)調 AAS
gotoの基本的な使い方くらい知っておいた方が良いかと
ループを抜ける為だけに関数を分けないといけなくなるし
挙げ句の果てに言語の欠陥とか言い出す
86: 2019/12/16(月)20:06 ID:3ha1TSe5(1)調 AAS
今のC言語ではネストしたループから抜けるのはフラグを使うかgotoしかないからたいていgoto使ってるけどgotoなんて野蛮なものはできたら使いたくないから他の言語で実装されてるラベル付きbreakなりレベル付きbreakが欲しいって話だろ
今更gotoの基本的な使い方ガーとか言ってる奴はどんだけ低レベルなんだよw
87(1): 2019/12/16(月)20:09 ID:xr9KOlE0(1/17)調 AAS
あれ？
returnで抜ける為に関数に分けるんじゃなかったの？
88: 2019/12/16(月)20:21 ID:xr9KOlE0(2/17)調 AAS
gotoは野蛮とかいう歪んだ教育のせいで
わざわざ変数を定義したりわざわざ関数を分けたりして
わざわざ見にくいコードにしちゃう
生でポインタを扱う野蛮なC言語なんか使わないで
上品な言語を使った方が良いよ
89: 2019/12/16(月)20:28 ID:jL52ryYk(1/2)調 AAS
gotoじゃなくてgodowntoなら誰も文句言わなかった
90: 2019/12/16(月)20:47 ID:LjjtFzBx(1/4)調 AAS
自分は小規模な字句解析みたいなのでもgoto使うことがある。
ラベルで状態を表してgotoで遷移する。
goto使わなかったら状態を変数に入れて、
ループの中でswitchで場合分けするようなコードになって辛い。
91: 2019/12/16(月)20:52 ID:DbnQVwfE(1)調 AAS
ループの中のswitchでループを抜けるのに
gotoを使うんじゃ？
92: 2019/12/16(月)21:41 ID:wRV+mRkt(1)調 AAS
次の状態を返す関数ポインタでループすればいんじゃないの？
93: 2019/12/16(月)21:45 ID:jhycq5nX(1)調 AAS
そんな野蛮なことはしません
94: 2019/12/16(月)22:12 ID:arH1mwfY(4/13)調 AAS
>>87
returnで抜けるためというか、
1. gotoで抜けたい
2. 関数にするか
3. あ、そもそもコードが複雑なんだな
4. 関数にして正解だった
こんな感じかな。
gotoで抜けたいと思う＝コードが複雑になってるという警鐘だと思ってる。
95: 2019/12/16(月)22:22 ID:xr9KOlE0(3/17)調 AAS
で>>82みたいな意味不明な関数の分け方をすると
>>57は、より複雑な記述をしたいっていう主張だし
gotoに親を殺された人の言うことは支離滅裂
理由が後付けだからそうなる
96: 2019/12/16(月)22:27 ID:arH1mwfY(5/13)調 AAS
> で>>82みたいな意味不明な関数の分け方をすると
なんで意味不明？
そもそもその処理に意味をもたせてから言おうよｗ
処理に意味をもたせると、関数にしたほうが良いと思うからさ
97: 2019/12/16(月)22:32 ID:arH1mwfY(6/13)調 AAS
> 理由が後付けだからそうなる
質問の方が先なのに、理由を先に言えるわけがないｗ
Q. 多重ループから抜けるのに、gotoは必要ですか？
A. 必要ありません。多重ループが必要になるぐらい複雑なら関数にしますから
これが普通だから、後付にはならないよね。
98: 2019/12/16(月)22:37 ID:xr9KOlE0(4/17)調 AAS
じゃ>>57も当然不要ですね
99: 2019/12/16(月)22:38 ID:arH1mwfY(7/13)調 AAS
そんなにくり返し言わなくても、最初から不要だしｗ
100: 2019/12/16(月)22:39 ID:xr9KOlE0(5/17)調 AAS
じゃあ>>57はどういうつもりで書いたんでしょうかね？
101: 2019/12/16(月)22:40 ID:LjjtFzBx(2/4)調 AAS
gogoサイコーっす
102: 2019/12/16(月)22:41 ID:LjjtFzBx(3/4)調 AAS
間違えたw goto
103: 2019/12/16(月)22:42 ID:xr9KOlE0(6/17)調 AAS
話を戻して
・ループを抜けた後に関数の後処理が必要な場合
・ループを抜けた後にローカル変数を使った処理をする場合
・他、意味的に関数内で処理したい場合
はどうするの？
104(1): 2019/12/16(月)22:44 ID:arH1mwfY(8/13)調 AAS
> ・ループを抜けた後に関数の後処理が必要な場合
前処理を関数の中でするな
> ・ループを抜けた後に関数の後処理が必要な場合
戻り値を返せ
>・他、意味的に関数内で処理したい場合
自分で具体例が思いつかないなら出すな
105: 2019/12/16(月)22:45 ID:arH1mwfY(9/13)調 AAS
コピペミスったな
>・ループを抜けた後にローカル変数を使った処理をする場合
戻り値を返せ
106(1): 2019/12/16(月)22:48 ID:xr9KOlE0(7/17)調 AAS
>>104
なぜ？
107: 2019/12/16(月)22:52 ID:arH1mwfY(10/13)調 AAS
>>106
なにが？
ってか、もう少し具体例を書けよ。
それみれば設計がおかしいという結論になる
例えて言うなら、パーサーの処理の中でネットワークにアクセスしたい時困るから
ネットワークモジュールはグローバルにアクセスできるようにしておくべき
みたいなことを言ってるようなもんだよ。それは設計がおかしい。
108(1): 2019/12/16(月)22:54 ID:xr9KOlE0(8/17)調 AAS
int func(int n){
ロック
ループ
アンロック
return 結果;
}
109: 2019/12/16(月)22:55 ID:xr9KOlE0(9/17)調 AAS
int func(int n){
ループ
値を加工
return 値
}
110: 2019/12/16(月)22:58 ID:arH1mwfY(11/13)調 AAS
>>108
「ループ」という言い方がおかしい。
ループで「実装」しているのであって
やっているのはなにかの「処理」だろう？
int func(int n){
ロック
処理
アンロック
return 結果;
}
「処理」という名前をつけたくなるならば、
そこは独立した関数にするべきだろう
111: 2019/12/16(月)23:02 ID:xr9KOlE0(10/17)調 AAS
double sigma(const double *a){
ループで総和と2乗和と数を計算
標準偏差を計算
return 標準偏差
}
112(2): 2019/12/16(月)23:04 ID:xr9KOlE0(11/17)調 AAS
int func(int n){
int a[10] = {0};
ループ
aから値を計算
return 値
}
113: 2019/12/16(月)23:06 ID:xr9KOlE0(12/17)調 AAS
いくらでも思い浮かぶ
114: 2019/12/16(月)23:07 ID:LjjtFzBx(4/4)調 AAS
全部関数に分けろって言われて終わりそう
115: 2019/12/16(月)23:09 ID:xr9KOlE0(13/17)調 AAS
分ける理由がない
116: 2019/12/16(月)23:10 ID:xr9KOlE0(14/17)調 AAS
>>112を分けてみて
117: 2019/12/16(月)23:17 ID:xr9KOlE0(15/17)調 AAS
関数分けってのは意味、機能、汎用性、...で分けるべきであって
単にループを抜けるのにreturnを使いたいから分けるとかを考えるべきじゃない
118(1): 2019/12/16(月)23:23 ID:xr9KOlE0(16/17)調 AAS
素直にgotoの使い方を覚えた方が良いよ
意地はってないで
119(1): 2019/12/16(月)23:42 ID:arH1mwfY(12/13)調 AAS
>>118
だから意味とか汎用性とかで分けると
自然にループに抜けるときにreturnを使えば良くなるんだよ
120: 2019/12/16(月)23:45 ID:jL52ryYk(2/2)調 AAS
gotoでもreturnでも読めれば良い
但し1関数50行以上は読めないんで宜しく
121: 2019/12/16(月)23:48 ID:arH1mwfY(13/13)調 AAS
>>112
だから「ループ」と書くなっちゅうねんｗ
関数を設計するならこんな感じやろ？
int func(int n){
int a[10] = {0};
memset_random(a, sizeof(a)); // a配列をランダムな値で埋める
int chksum = chksum(a, sizeof(a)); // a配列のチェックサムを計算
return 値
}
関数の中でループしてようが何して用が関係ないわ
122: 2019/12/16(月)23:55 ID:xr9KOlE0(17/17)調 AAS
sizeof(a) とか書くレベルの相手してたのか
アホらしい
123: 2019/12/17(火)00:04 ID:JvBVW/sd(1/4)調 AAS
>>119
じゃあ普通のbreakもいらないね
まさか使ってないよね
124: 2019/12/17(火)06:41 ID:YMIUnRyv(1)調 AAS
まあgotoあればbreakは要らんわなw
そんな言語は使いたくないけど
125: 2019/12/17(火)07:00 ID:JvBVW/sd(2/4)調 AAS
gotoもbreakもいらないって
126: 2019/12/17(火)07:10 ID:JvBVW/sd(3/4)調 AAS
gotoもbreakも無い
ループを抜けるときはreturn
switchを抜ける時もreturm
関数は単一の処理しか出来ない
こんな言語はイヤだね
127: 2019/12/17(火)07:23 ID:JvBVW/sd(4/4)調 AAS
後に退けなくなったアホの作る言語
128: 2019/12/17(火)07:39 ID:odSFZTU+(1)調 AAS
gotoの使い方
http://www7b.biglobe.ne.jp/~robe/cpphtml/html03/cpp03005.html
http://www9.plala.or.jp/sgwr-t/c/sec06-6.html
http://tomeapp.jp/archives/1029
http://opencv.jp/opencv-2svn/c/core_utility_and_system_functions_and_macros.html
129: 2019/12/17(火)08:49 ID:QeraReD1(1/2)調 AAS
たとえ相手がアホであっても、しつこく絡むのもまたアホだと思うんだが…
130: 2019/12/17(火)12:33 ID:vDeQfs2U(1)調 AAS
自分より下の人間を叩くしか能がないんだろ
ID:arH1mwfYなんてスルーでいいと思うんだがそれしか叩けないからいつまでたっても終わらないw
131: 2019/12/17(火)13:15 ID:GBggCGQK(1)調 AAS
雑談スレだしネタも無いから良いんじゃね？
132(2): 2019/12/17(火)18:32 ID:x9hYOthl(1)調 AAS
String型が無いって死ぬほど不便なんだが
133: 2019/12/17(火)19:06 ID:p6v4J1KY(1)調 AAS
他の言語を使ってください
134: 2019/12/17(火)19:18 ID:6LhBKKEf(1)調 AAS
>>132
自分で似たようなものを作ればいいだろ。汎用的なものがあっても使い勝手や性能で何かしら不満が出るのだから、自分の要求に合致するものを自分で作りな。
135: 2019/12/17(火)20:52 ID:QeraReD1(2/2)調 AAS
特定の用途においてローカルラベルが役に立つことも無くはないかもしれないな。
自分でそれをやるかどうかは別にして。
いずれにせよ、gcc では式の中で変数を宣言できるのだから
ローカルラベルが可能になっても驚かないよ。
136: 132 2019/12/18(水)23:56 ID:9RDvo6To(1)調 AAS
typedefを使用
https://pastebin.com/iX2DDpRA
137: 2019/12/19(木)00:12 ID:vqVcyk4J(1)調 AAS
PHPはbreakにネスト数指定できるよな
138(1): 2019/12/19(木)09:03 ID:vjQjJ4nK(1)調 AAS
C言語の開発者によるgoto文の使い方を対象とした実証研究の結果、「goto文は無害だと考えられる」
https://developers.srad.jp/story/15/02/14/2017207/
139(1): 2019/12/19(木)16:51 ID:86m15ms9(1)調 AAS
longjmp についても調査してほしいな。
特に例は挙げないが、誰もが使っているようなプログラムでも結構使われてる。
140: 2019/12/19(木)16:57 ID:+cpLTGtZ(1)調 AAS
static
141: 2019/12/25(水)01:15 ID:k3ZI/Zl3(1)調 AAS
>>139
それは代わりがない。
142: 2020/07/09(木)10:18 ID:tNMImStE(1)調 AAS
>>7
大発明なんbだけどな、
リスクも大きい
143(1): 2020/07/09(木)10:22 ID:vrNDocOm(1)調 AAS
発明って言うか当時はそれが普通だったろ
144: 2020/07/09(木)10:51 ID:I92CxMlL(1)調 AAS
longjmpは「設計上のミス」ではないんでは。
むしろそういう動作をする目的で設計・導入されたように思えるけど
（longjmpに関するRationaleを読んでないので当てずっぽうw）
145: 2020/07/09(木)16:31 ID:KHuqFwsC(1)調 AAS
>>5
エスティーアールトゥーケー
146(2): 2020/07/09(木)21:58 ID:Zo8nB2NL(1)調 AAS
例外は setjump、longjumpて実装されてるんじゃないのかな？
違うのかな？
147(1): 2020/07/10(金)22:03 ID:AfLodKTd(1)調 AAS
>>143
普通じゃない
当時はPascalの文字数+文字列が普通
148(1): 2020/07/10(金)22:26 ID:U1HXCT7S(1)調 AAS
returnが1変数しか対応していないこと。
ポインター祭りになって可読性がウンコになる
149: 2020/07/10(金)23:11 ID:7RoGj1wV(1)調 AAS
>>148
可読性がうんこなのはお前の能力が足りないから
150: 2020/07/11(土)00:22 ID:UdJEarb2(1)調 AAS
いろんな言語たらい回しにされるって不幸だよね。
こういうノーガキ垂れる奴がウチの職場にいなくて良かった〜ヽ(´▽｀)/
151(1): 2020/07/11(土)10:50 ID:J0INoyBc(1)調 AAS
>>147
OSのAPIは文字数じゃなくて
文字列byte列+デリミタ'$'
みたいなのが多かったぞ
152: 2020/07/11(土)12:01 ID:2M7rjl8q(1)調 AAS
>>151
それはCP/Mの文字列出力がそうなっていただけ
入力はC言語のfgets()なんかみたいにデリミタの '$' を付加したりしないよ
153(1): 2020/07/11(土)23:02 ID:wT5X8b6q(1/2)調 AAS
MASM もそうだけど、$ は実際にメモリに配置されるのですか？
それとも、db(アセンブラ命令)で長さを省略するためのものですか？
154: 2020/07/11(土)23:04 ID:wT5X8b6q(2/2)調 AAS
すみません、>>153 です
送信した瞬間に自分の発言の矛盾に気がつきました…
155(1): 2020/07/13(月)19:12 ID:Xa4f7njE(1)調 AAS
>>146
longjumbと例外に関して
すばらしい文書があるからお読みなさって。
http://www.nurs.or.jp/~sug/a/super/longjmp.htm#sec36
156(1): 2020/07/13(月)19:20 ID:WBkWHxcT(1)調 AAS
そっちは Not Found ですよ
http://www.nurs.or.jp/~sug/soft/super/longjmp.htm
157: ◆QZaw55cn4c 2020/07/13(月)20:53 ID:p7kPKe7n(1)調 AAS
>>156
ありがとうございます、昔に読んだのに URL を失ってしまって…ずっと探していたんです
158: 2020/07/13(月)21:24 ID:jVM2RLmn(1)調 AAS
>>155
ありがとうございます >>146 です
拝見いたします
159: [139] 2020/07/14(火)09:59 ID:obK0XQNS(1)調 AAS
ありがとうございます。いや、longjmp をdisるような意図は無かったんだけど。
「シグナルとコールバック」も勉強になります。
http://www.nurs.or.jp/%7Esug/soft/super/signal.htm
この辺りは man を読んでも使い方が解りにくいので…
160: 2020/07/23(木)11:48 ID:3i8DjqKb(1)調 AAS
_sが付いてるやつ使えよ
161: 2021/05/21(金)17:59 ID:lDwsy5iU(1/2)調 AAS
関数じゃないけどこれやろ
If the unsigned type has conversion rank greater than or equal to the rank of the signed type, then the operand with the signed type is implicitly converted to the unsigned type.
うんこ
162(1): 2021/05/21(金)18:32 ID:lDwsy5iU(2/2)調 AAS
プログラマを信じろと言うけどさ
じゃあ暗黙の型変換とか余計な節介やめてくれねーかな
163: 2021/05/21(金)18:36 ID:HkPQV5tP(1)調 AAS
cでまともに算数出来れば一人前
164: 2021/07/18(日)03:12 ID:a6CF+eHw(1)調 AAS
暗黙が嫌なら明示しろよばかじゃないの？
165: 2023/01/02(月)19:09 ID:N1W2ThUk(1)調 AAS
>>162
暗黙の型変換はワーニング出るようにできなかったっけ？
166: 2023/05/10(水)10:41 ID:zbvSAEuT(1)調 AAS
>>138
まとめがおかしい
啓蒙活動の結果gotoの有害な使い方は極めて少なくなった
というべきだろう
167: 2023/08/04(金)19:33 ID:+XXFTri2(1)調 AAS
ken thompsonがcreat()の最後のeを省いたのは失敗だったと
直すとしたらそこを直したいと
168: 2023/09/27(水)15:27 ID:0JeuaQMM(1)調 AAS
失敗は成功への近道だと思って取り組んでみよう。
169: 2023/09/30(土)09:21 ID:z8ssgezC(1)調 AAS
(”ロ”;)ｹﾞｹﾞｯ!!
170(1): 2023/10/03(火)12:21 ID:QaeBAOHS(1)調 AAS
型省略でintと看做すのはミスだろ
171: 2023/10/04(水)22:29 ID:pmZG4qtZ(1)調 AAS
>>170
K&Rの頃との互換性の為じゃないかな。
Ｃは、兎に角モノグサの為の様な仕様が多いよね。
一度馴染むと、BASICですら面倒くさい。
172: 2023/10/05(木)17:05 ID:WXXGTjkD(1)調 AAS
まんどくせ
173: 2023/10/05(木)19:40 ID:KP6Kmsv6(1)調 AAS
ポインタ渡し全般かな
174: 2023/10/05(木)22:17 ID:nm5oakcJ(1)調 AAS
Cの型はある意味pythonより弱い
175: 2023/10/07(土)11:48 ID:QSI1NZ2Z(1)調 AAS
ある意味というか、Pythonは一応強い型付けでしょ。
176(1): 2023/10/07(土)11:51 ID:SZDGKzpf(1)調 AAS
Cに型なんかあって無い様なもんだしな
177: 2023/10/07(土)17:38 ID:2aizzJpQ(1)調 AAS
>>1
fgets知らんのかな
getsなんて使わない
どうしても使う時はfgets(buf,size,stdin)とすればいいだけ
178: 2023/10/08(日)10:11 ID:CrdCteTP(1)調 AAS
>>176
Cはアセンブラで言うところのアドレッシングが充実
179: 2023/10/09(月)15:00 ID:gMk1hFfQ(1)調 AAS
マクロが真っ黒
180: 2023/10/15(日)02:11 ID:LZ9c8wO/(1)調 AAS
∑(ﾉ▼ο▼)ﾉ ｵｫｵｫｵ!!
181: 2023/10/16(月)09:47 ID:kgcCjrnK(1)調 AAS
DDoS回避したみたいね
上下前次1-新書関写板覧索設栞歴
あと 34 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.013s