C言語なら俺に聞け 146
at TECH
[前50を表示]
450:デフォルトの名無しさん
18/07/26 17:01:26.31 Vf/lMr8pd.net
>>440
決まってる
451:デフォルトの名無しさん
18/07/26 17:03:12.87 0PbkpRUpH.net
C99には
The pointer returned if the allocation succeeds is suitably aligned
so that it may be assigned to a pointer to any type of object and
then used to access such an object or an array of such objects
in the space allocated (until the space is explicitly deallocated).
とあるので、どんなオブジェクトでもその場所に置けることは保証されている。
ただ、アラインメントがずれてるとアクセスできるけど遅くなるとかいう
変態環境でどうなるかとかは規定されていない。
（ふつうの実装ならまともにアラインメントされていると思うが）
これをどうしても合法的に確実にアラインメントしたい人向けなのが、aligned_alloc()。
452:デフォルトの名無しさん
18/07/26 17:08:47.42 VR2PsH3f0.net
>>443
そうなんだ
おれはてっきりもっと全然違う基準、例えば64バイトとか16KBとかで
アライメントしたいときaligned_allocの方を使うものかと思ってた
453:デフォルトの名無しさん
18/07/26 17:17:59.96 0PbkpRUpH.net
>>444
そりゃもちろんそういう用途にも使えるよ
454:デフォルトの名無しさん
18/07/26 19:04:38.15 Vf/lMr8pd.net
>>443
パフォーマンスはCの仕様で規定されてないから
そんな事を心配したら標準ライブラリも使えないぞ
memcpyが劇遅かもしれないとか
455:デフォルトの名無しさん
18/07/26 19:07:36.39 Vf/lMr8pd.net
普通の用途は>>444みたいなヤツ
zmmレジスタのサイズ
キャッシュのラインサイズ
HDDのセクタサイズ
FIFOのサイズ
なんかにしたい場合
456:デフォルトの名無しさん
18/07/27 00:14:41.96 J2z2B0JS0.net
家でC言語の独学してたけど眠くなって続かないから週末だけ学校にいこうと思う。関西でお勧めはどこですか？
457:デフォルトの名無しさん
18/07/27 07:12:56.09 GxaYOivnM.net
金の無駄遣い
458:デフォルトの名無しさん
18/07/27 09:20:14.17 UMN2C6wAd.net
大阪大学基礎工学部情報科学科
459:デフォルトの名無しさん
18/07/27 09:52:20.15 mngCBxZNM.net
勉強を成功させるには強い動機付けが必要
やる気も効率も上がらないのはまずそこに失敗してる
460:
18/07/27 20:31:18.69 dNDh2QHO0.net
>>451
動機付け、ですか…
そんなものは初めからありませんでした…
461:デフォルトの名無しさん
18/07/27 21:04:40.28 Gi4wb1wwM.net
個人的にはLinuxのデバドラから入るのがおすすめ
分かってくると俺の方が上手く書けるとか拗らせる
462:デフォルトの名無しさん
18/07/28 02:43:37.63 E0E9ZkiG0.net
動機を自分で意識的に付けようとしても中々出来ないのが普通なのでは？
何かそれなりのメリットがあると信じられないと何もしないのが普通の人間だろうし。
例えば何かをするとモテモテになるだとか、そういう自分にとってのメリットがあるならその何かをやる気は出るだろうけど、そんなのがないなら、あるいはもしあってもそれに気付けないなら普通はやる気は起きんでしょう。
463:デフォルトの名無しさん
18/07/28 07:47:22.78 CM3Pz0hR0.net
C言語、あるいはプログラムそのものの入門書で、文庫か新書サイズの良書はありますか？
464:デフォルトの名無しさん
18/07/28 08:07:15.42 xtvz/nLF0.net
C++ならこれならわかるC++があるが・・・
465:デフォルトの名無しさん
18/07/28 08:08:34.90 8G21bbxI0.net
はじめてのC
466:デフォルトの名無しさん
18/07/28 09:55:07.43 VMG9DnUG0.net
>>455
講談社ブルーバックスなら新書サイズでC本やプログラミング本があったはず。
ただ、書名は挙げられないし、良い入門書かどうかも分からん。
参考までに、本の寸法に制約があるのは何故かな?
467:デフォルトの名無しさん
18/07/28 10:28:26.03 CM3Pz0hR0.net
電車の中や、昼飯食べながら片手で読めるものがあれば、と思ったんです
パソコンいじりながら読むなら大きい方がいいんですけどね
468:デフォルトの名無しさん
18/07/28 11:19:05.91 fTj3LZGS0.net
Kindleで読めば？
469:デフォルトの名無しさん
18/07/28 12:06:22.12 s8FQvaoRM.net
さっさとkindleで読めアホ
470:デフォルトの名無しさん
18/07/28 14:09:45.43 xtvz/nLF0.net
マンガはタブレット端末で見るけど
参考書系は紙じゃないとなんか駄目だわ。全然集中できない。
471:デフォルトの名無しさん
18/07/28 16:49:06.40 XyATpAmoa.net
kindleとか電子書籍って参考書系読みにくいよね
漫画みたいにページそのままならいいのに
472:デフォルトの名無しさん
18/07/28 17:02:27.00 fTj3LZGS0.net
何年も前からiPadや電子ペーパー端末でプログラミングや法律の本読んでるけど特に不都合感じてないな
むしろ紙は持ち運びし辛いし検索もできないから不便だわ
473:デフォルトの名無しさん
18/07/28 19:13:56.62 sJabZ7tp0.net
すみません
この２行のコードを１行にするスマートな書き方ってありますか？
idx = idx << 12;
idx = idx ^ _data;
474:デフォルトの名無しさん
18/07/28 19:15:47.78 CM3Pz0hR0.net
kindleですか、参考にします。
ありがとうごさいました。
475:さまよえる蟻人間
18/07/28 19:31:07.68 g7V8p7HEd.net
>>465
idx = (idx << 12) ^ _data;
476:デフォルトの名無しさん
18/07/28 19:52:37.67 mee2LTKv0.net
>>467
うむ
477:デフォルトの名無しさん
18/07/28 20:05:12.26 0bzuzSMVM.net
初心者スレじゃないんだからidxがvolatileの場合を無視した回答は良くない
478:デフォルトの名無しさん
18/07/28 20:12:34.82 cv0xu0xkM.net
それを言うならvolatileでない場合を無視するのが問題では？
volatileであれば問題ないだろ
479:デフォルトの名無しさん
18/07/28 20:26:36.40 mee2LTKv0.net
わしここの住人じゃなくて通りすがりなんだけど、
>>467 だとなにか問題あるの？
480:デフォルトの名無しさん
18/07/28 21:15:36.33 DyF9e7oqM.net
>>471
もしもidxが送信データを書き込み受信データを読み出すためのレジスタだったら、
複数回書き変えること自体が重要だったり、読み込み時は異なる値を返却するかもしれない、
という割とどうでもいい話。
481:デフォルトの名無しさん
18/07/28 21:32:40.43 mee2LTKv0.net
>>472
C的に見れば無駄なコードが実は必要だったという、そういう話しかな
482:デフォルトの名無しさん
18/07/28 21:52:57.65 jbwKRPPF0.net
最適化で一瞬ﾚｼﾞｽﾀの値が変わらなくなったから
それがどうした
483:のという話だからな むしろ一瞬ﾚｼﾞｽﾀの値が変わるようなｺｰﾄﾞがそもそもぜんぜんﾀﾞﾒ
484:デフォルトの名無しさん
18/07/28 21:55:35.27 remUbofq0.net
そういう話になってくるとレジスタってなんや？って話になるからな？
485:デフォルトの名無しさん
18/07/28 22:00:32.66 jbwKRPPF0.net
ちゃんと確定値入れろよ
なんで計算中の値いれてんの
なんでそんな頭悪いことすんの
一行にしたいということは
間違いなく計算過程のｺｰﾄﾞだからな
486:デフォルトの名無しさん
18/07/28 22:02:55.51 aLrZyGnX0.net
通信プロトコルでCPUレジスタ使ってやり取りする様な方法ってあったっけ？
487:デフォルトの名無しさん
18/07/28 22:05:44.90 jbwKRPPF0.net
ﾒﾓﾘﾏｯﾌﾟﾄﾞI/Oなんか簡単に消える
488:デフォルトの名無しさん
18/07/29 00:51:03.42 ++N1yuL90.net
>>477
「レジスタ」って「CPUのレジスタ」だけじゃなくて「I/Oチップのレジスタ」って使い方もするよ
489:デフォルトの名無しさん
18/07/29 07:02:57.29 YD8oKrYq0.net
>>469を書くためにわざと>>465の質問しただろ
まあ確かにvolatileだと>>467ではダメ
明らかに初心者の質問に>>469の回答もどうかと思うが
idx = idx << 12; idx = idx ^ _data;
これで良い
490:デフォルトの名無しさん
18/07/29 07:15:13.10 YD8oKrYq0.net
質問というよりスレのレベルを見るための問題だな
>>474みたいな素人が見事に引っかかっちゃう
491:デフォルトの名無しさん
18/07/29 09:00:29.62 +KuOy2bC0.net
触らぬ質問に騒乱無し
492:デフォルトの名無しさん
18/07/29 09:09:25.31 7lOBGcOb0.net
実は本物の初心者で volatile などというものの存在すら知らず最初の回答で全く問題なかったのだが
知識があるものが知識がある故にあらゆる妄想を膨らませまくって勝手に争って大混乱になり、
最初に質問した者は怖くなって素早く逃亡。このスレで質問するのは止めようと固く心に誓う。←いまここ
493:デフォルトの名無しさん
18/07/29 10:14:38.25 Ga4xmacd0.net
明示してないvolatileなんて考慮する必要ないわ
494:デフォルトの名無しさん
18/07/29 11:40:21.87 LQAkWqzt0.net
知恵遅れのｸｿﾆｰﾄが
またﾃｷﾄｰなこといってるわ
2ちゃんねるはどこのｽﾚも同じだわ
ｸｿﾆｰﾄはどこにでもいる
やっぱりなｸｿﾆｰﾄ2ちゃんねるから
ｸｿﾆｰﾄの駆除が必要
495:デフォルトの名無しさん
18/07/29 11:43:48.42 LQAkWqzt0.net
まずﾌﾟﾛﾌｪｯｼｮﾅﾙなｶﾝﾍﾟｷなｵﾚの回答の
どこがおかしいか
だれも指摘できてないからな
496:デフォルトの名無しさん
18/07/29 11:51:38.06 LQAkWqzt0.net
ちなみに知恵遅れどものﾃｷﾄｰな回答のどこがおかしいか
ｵﾚはすべて回答してる
ここも同じだ
497:デフォルトの名無しさん
18/07/29 11:58:20.56 1hBCvnSx0.net
夏休み？
498:デフォルトの名無しさん
18/07/29 11:59:21.49 LQAkWqzt0.net
ﾌﾟﾛﾌｪｯｼｮﾅﾙの回答に
ｼﾛｳﾄが口をはさむ余地はない
ｸｿﾆｰﾄは毎日が夏休みだもんな
499:デフォルトの名無しさん
18/07/29 12:01:00.22 LQAkWqzt0.net
ﾏｼﾞでなｸｿﾆｰﾄどもは身の程知らずなわけ
ｸｿﾆｰﾄがまともな社会生活を送ってる社会人に勝てると思ってんの
500:デフォルトの名無しさん
18/07/29 12:08:05.09 Rgaf8fSN0.net
プロフェッショナルかなんか知らんけど組み込み系は苦手のようだな
501:デフォルトの名無しさん
18/07/29 12:09:43.67 LQAkWqzt0.net
な知恵遅れがｵﾚのﾚｽのどこがおかしいか指摘できない
指摘しようがないからな
なぜならｽｷのないｶﾝﾍﾟｷなﾚｽだからな
どこのｽﾚでも知恵遅れが偉そうにしてるのは同じだ
502:デフォルトの名無しさん
18/07/29 12:09:51.76 7lOBGcOb0.net
>>486
何処で回答してんの？
503:デフォルトの名無しさん
18/07/29 12:11:10.71 LQAkWqzt0.net
>>476>>474 でおもいっきり回答してるわ
知恵遅れはなにが書いてあるか理解できない
504:デフォルトの名無しさん
18/07/29 12:12:34.88
505: ID:7lOBGcOb0.net
506:デフォルトの名無しさん
18/07/29 12:14:12.01 LQAkWqzt0.net
それが回答になってないなら
ｵﾏｴのｵﾂﾑに問題があるか
ただの仕事未経験のｸｿﾆｰﾄがなにも分からずﾃｷﾄｰなこといってるだけだからな
ただｸｿﾆｰﾄである確度が極めて高い
507:デフォルトの名無しさん
18/07/29 12:15:16.00 7lOBGcOb0.net
>>496
はいはい。おだいじに。
508:デフォルトの名無しさん
18/07/29 12:15:53.84 LQAkWqzt0.net
はい
ｸｿﾆｰﾄ図星
509:デフォルトの名無しさん
18/07/29 12:17:35.80 7lOBGcOb0.net
やはり気温40℃超えは人をおかしくするには十分な気温だったようだな。
猛暑も酷なことをするもんだ。合掌。
510:デフォルトの名無しさん
18/07/29 12:19:13.56 LQAkWqzt0.net
ｸｿﾆｰﾄは家からでないから関係ないだろ
電気代も親に払ってもらってるからな
ｸｰﾗｰの電気代もﾊﾟﾁｮｺﾝの電気代もな
親が払ってくれてんだぞ
511:デフォルトの名無しさん
18/07/29 12:33:12.40 Rgaf8fSN0.net
組み込み系の周辺チップの制御レジスタの仕様書見たことある？
制御レジスタの特定ビットのRead/Write動作そのものに意味があるんだよ
直に制御レジスタをアクセスしないと駄目
最適化でCPUレジスタの中で値を書き換えても無意味、周辺チップは何の動作もしない
volatileはそんな時にコンパイラの最適化を抑制するために使用する
例えば、
idx = (idx << 12) ^ _data;
の例で言えば、(idx<<12)の部分で実際にidxのアドレスにマップされたメモリの値が変更される
その後に最終的に、(idx << 12) ^ _dataの値が上書きされる
idxがvolatile宣言されていない場合は、途中の評価結果はCPUの内部レジスタだけで最適化されてマップされたメモリの値は最終評価結果の書き込みしか保証されない可能性がある
組み込みの制御レジスタの例としてPICのデータシートのリンクを貼っておく
ビット毎にR/Wでチップの機能を制御している様子が分かる
URLﾘﾝｸ(www.microchip.co.jp)
512:デフォルトの名無しさん
18/07/29 12:46:01.14 LQAkWqzt0.net
最適化で、一瞬変わってたのが変わらなくなったらどうすんのとかいってるｱﾎなこといってる知恵遅れに
その一瞬だけ変わってたｺｰﾄﾞが正しいｺｰﾄﾞとか
ﾎﾝｷで思ってんのといってるだけだからな
そもそもvolatileついてなかったら
そのあと書き変わる保証すらどこにもない
そもそも保証がないｺｰﾄﾞのなんの保証をするわけ？
知恵遅れがいうことはさっぱり理解できないわ
513:デフォルトの名無しさん
18/07/29 12:47:08.34 XJOvENJB0.net
C++相談室 part137 あたりに書き込もうと思った悪態を
間違えてこちらに投稿したのに気づいたが引っ込みがつかなくなった、
という感じがしないでもない。
今までの自分の態度をかえりみると「誤爆しました、すまん」と
書くことも照れ臭いと思ってる、かも知れない。
おそらく俺の勘違い(嫉妬に基づく邪推)だろうけどネ。
514:デフォルトの名無しさん
18/07/29 12:48:26.49 LQAkWqzt0.net
あっちもこっちも
ｸｿﾆｰﾄがえらそうにしてるからな
2ちゃんねるからｸｿﾆｰﾄの駆除は必要
515:デフォルトの名無しさん
18/07/29 12:50:28.71 LQAkWqzt0.net
むしろあっちでもこっちでも
ｵﾚは間違った回答は一切してないからな
すべてｶﾝﾍﾟｷなﾚｽだ
知恵遅れがﾃｷﾄｰなこといってるのは
あっちもこっちも同じ
516:デフォルトの名無しさん
18/07/29 12:57:17.34 Rgaf8fSN0.net
>>475
そりゃvolatileも万能ではないよ
最終的に確認するのは実機デバッグしかない
アセンブラで直接叩くしか解決できない場合もある
ICEで直接追跡しながらプログラムコードとチップの動作を確認するのが組み込みシステムの開発
ただしvolatileすら使わないのでは無駄に書き替え工数が膨れ上がる
最低限の対応はあらかじめ取っておくべき
517:デフォルトの名無しさん
18/07/29 12:57:38.10 Rgaf8fSN0.net
間違えた
>>504
518:デフォルトの名無しさん
18/07/29 13:10:28.41 x0a0gOqsM.net
>>504
このスレの誰もが真っ先にｷﾐを駆除したがってるのに、天才のはずのｷﾐが気づいてない？おかしいねぇ
519:デフォルトの名無しさん
18/07/29 13:14:35.44 LQAkWqzt0.net
ｸｿﾆｰﾄのたまり場から
ｸｿﾆｰﾄを駆除するだけだからな
部屋にいる人間の数より
南京虫やﾀﾞﾆやﾉﾐの数のほうが多い
そういうｽﾚだ
ﾊﾞﾙｻﾝ焚かないとなまともな人間がよりつかなくなる
その南京虫やﾀﾞﾆやﾉﾐみたいなｵﾏｴを駆除するといってるわけ
わかった？
520:デフォルトの名無しさん
18/07/29 13:26:52.38 OHU95624d.net
>>506
今時、
SFRにアクセスする道具は整ってる
Cで書ける
標準の環境でない場合だけ心配すればいい
521:デフォルトの名無しさん
18/07/29 14:46:05.01 u6NN385t0.net
>>506
volatileの機能は明確に定義されていて
万能だとか頭の悪そうな話は必要ない
実機でデバッグしてコンパイラのバグが発覚することはあって
そのときの状況によってはアセンブラを使うこともあるが
アセンブラで「叩く」って言葉遣いは変だぜ
石を叩くのはCでもできるしな
ところで、おまえさんのところでは
JTAGじゃなくICEって言うのか
522:デフォルトの名無しさん
18/07/29 15:04:54.06 Rgaf8fSN0.net
>>511
インサーキットエミュレータ(In Circuit Emulator：ICE)だな
もっともC言語でプログラム組んでたのは7,8年前までだけどな
当時はルネサス変更前のNECの78K0や東芝のTLCS780、松下(パナソニック)のMN101なんかで組み込み開発をやってた
523:デフォルトの名無しさん
18/07/29 15:12:25.34 h14Y8Y+4d.net
volatileだって書いた通りに動く
コンパイラが信用できないときは逆アセンブルして確認する
実機デバッグしなければならないのは書いたやつが信用できないからで
volatileだからじゃないね
あとjtagはインターフェースのことでデバッガ自体は
ICEと呼ぶかな、うちも
524:デフォルトの名無しさん
18/07/29 15:40:41.02 OHU95624d.net
volatileをつけてコンパイラが(一見)正しいコードを生成しても
CPUが順番を変えて実行したりする
マルチコア、OoO時代は
同期コードを自分で書くのは色々な意味でやめた方が良い
SFRアクセスには使えるが
525:デフォルトの名無しさん
18/07/29 15:52:36.85 OHU95624d.net
>>513
本来の意味のICEなんて絶滅寸前
526:デフォルトの名無しさん
18/07/29 17:56:25.83 VgN+++E0a.net
夏はアイスだよね～
527:デフォルトの名無しさん
18/07/29 17:57:28.24 +KuOy2bC0.net
割り込み禁止って今のOSだと無意味？
528:デフォルトの名無しさん
18/07/29 18:45:23.92 Ga4xmacd0.net
>>517
普通に使ってると思うけど？
529:デフォルトの名無しさん
18/07/29 19:38:18.03 u6NN385t0.net
ユニプロセッサからマルチプロセッサへパラダイムシフトするにあたって
１つのCPUだけ割り込み禁止してもどうたらって話だろ
530:デフォルトの名無しさん
18/07/29 20:17:45.56 VgN+++E0a.net
>>517
それはOS上で動くプログラムの受け取るシグナルの話？
それなら特定のやつ以外は禁止したり自分でハンドら書いたりできるわけだが。
そういう話ではない？
531:デフォルトの名無しさん
18/07/29 20:20:49.84 LQAkWqzt0.net
もしかして
ﾊｰﾄﾞｳｪｱ割込とｿﾌﾄｳｪｱ割込の違いすら分かってないのが
ﾚｽしてんのか
532:デフォルトの名無しさん
18/07/29 20:25:27.65 LQAkWqzt0.net
そもそもﾃﾞﾊﾞｲｽがCPUより超遅いことが分かってないようなのが
一瞬で値が変わってたのが、それが変わらなくなったらどうすんのとかいってるﾚﾍﾞﾙだからな
ﾎﾝﾄなｺｲﾂﾗの程度がｲﾛｲﾛと知れるわ
533:デフォルトの名無しさん
18/07/29 20:25:43.86 QevJM+ENd.net
いい加減秋田
534:デフォルトの名無しさん
18/07/29 20:35:45.02 2vtczMQ4M.net
>>519
どの割込みの話ししてるのか良くわからんけど、デバイスからの割り込みって予め指定したプロセッサが受け取る実装が多いぞ
535:デフォルトの名無しさん
18/07/29 20:44:07.76 YD8oKrYq0.net
良くわからん人は素直に>>514の通りにしなさい
ここで一言で語れるような内容じゃないから
536:デフォルトの名無しさん
18/07/29 20:44:37.09 YD8oKrYq0.net
特に半角君
537:デフォルトの名無しさん
18/07/29 20:47:29.08 LQAkWqzt0.net
低学歴知恵遅れがﾄﾞﾔ顔で書いてるOoOの問題なんか
分かった上でｵﾚはｶﾝﾍﾟｷなﾚｽしてるからな
538:デフォルトの名無しさん
18/07/29 20:48:20.64 LQAkWqzt0.net
もうねﾄﾞﾔ顔でﾚｽしてるﾔﾂの知能ﾚﾍﾞﾙが
致命的に低いﾜｹ
539:デフォルトの名無しさん
18/07/29 20:50:38.73 YD8oKrYq0.net
完璧なヤツが遅いとか速いとか
ハード割り込みとかソフト割り込みとか
まったくとんちんかんwww
書けば書くほど痛い
540:デフォルトの名無しさん
18/07/29 20:51:15.80 LQAkWqzt0.net
はい低学歴のﾚｽ頂きました
541:デフォルトの名無しさん
18/07/29 20:52:18.31 LQAkWqzt0.net
ﾚｽで簡単に判定できる
ｺｲﾂはｸｿﾆｰﾄ
ｺｲﾂは低学歴
超簡単
542:デフォルトの名無しさん
18/07/29 20:55:32.94 u6NN385t0.net
ソフトウエア割り込みとかバカかｗ
センズリ野郎
543:デフォルトの名無しさん
18/07/30 16:36:28.63 3BqaNEe76.net
キーボードから実数値を入力
3件のみを値が多い順に出力
最初にデータの件数(3件以上で、最大で100件とする)を入力
100件を超える場合、「エラー」と出力
ソートを使うみたいです。よろしくお願いします。
544:さまよえる蟻人間
18/07/30 17:03:31.87 hdlApzlBd.net
int i, num, array[100];
printf("Number of numbers: ");
scanf("%d", &num);
for (i = 0; i < num; ++i)
{
printf("array[%d]: ", i);
scanf("%d", &array[i]);
}
...
545:さまよえる蟻人間
18/07/30 17:04:07.39 hdlApzlBd.net
比較関数をqsortに渡せ。
546:デフォルトの名無しさん
18/07/30 17:56:36.84 n1wK/LR00.net
アレ? 蟻のarrayはint配列だけど、問題で要求してるのは実数値じゃね?
547:デフォルトの名無しさん
18/07/30 18:03:34.93 ehNSvi3pM.net
値が大きい順なのか
値の出現回数が多い順なのか
548:さまよえる蟻人間
18/07/30 18:12:03.23 6ITjo5O70.net
arrayをdoubleかfloatの配列にしてくれ。すまん。
549:デフォルトの名無しさん
18/07/30 18:38:36.45 xHVHgAPAa.net
夏休みの宿題か・・・
550:デフォルトの名無しさん
18/07/30 18:47:55.30 6X4H9pr00.net
floatならscanf/printfともに%f
doubleならscanfは%lfでprintfは%f
551:デフォルトの名無しさん
18/07/30 18:57:52.63 xHVHgAPAa.net
100件入力しても最初の3件だけを大きい順ではなく多い順に並べると・・・
変な宿題だな。読み間違えてないか？
552:デフォルトの名無しさん
18/07/30 19:07:25.93 Ei1MMj+i0.net
入力データに重複を許し、その出現数をカウントしろとか？
553:デフォルトの名無しさん
18/07/30 19:26:58.25 aSIGzv/D0.net
上位3件
554:デフォルトの名無しさん
18/07/30 19:27:37.61 aSIGzv/D0.net
だろ
普通に考えて
555:デフォルトの名無しさん
18/07/30 19:28:21.43 xHVHgAPAa.net
3件しかデータがないのに多い順となるとその内2件が同じ値でなければできない。
3件とも違うとか、あるいは3件とも同じなら並べかえる必要がない。
更に配列の先頭の二つが同じなら並べ変える必要がない。
それは既に多い順になっている。または全て同じ値だ。
とすると二番目の値と三番目の値を比較し、同じだったら最初の値と三番目の値を入れ換えれば良い。
三つとも全て同じ値の時に無駄な入れ換え処理をすることになるが、この程度はよかろう。
556:デフォルトの名無しさん
18/07/30 19:29:45.17 xHVHgAPAa.net
>>543
知らんがな。
557:デフォルトの名無しさん
18/07/30 19:37:02.99 aSIGzv/D0.net
「3件のみを」は「出力」にかかる
「値が多い」だから出現回数順
「多い順」のソート対象は書いてないが
普通に考えれば入力した全数
これがおれの解釈
558:デフォルトの名無しさん
18/07/30 19:43:33.82 WDgnZ3L+0.net
出現頻度の上位3位分を出せ？
typedef double VAL_T;
struct {
VAL_T val;
int count;
} data[100] = { 0 };
線形探索して カウントアップして data[] を count をキーにソートして
data[0].val, data[1].val, data[2].val, を出力する
559:デフォルトの名無しさん
18/07/30 19:44:19.20 v5YMtZ310.net
整数値ではなくて実数値で出現回数って些か問題に無理がないか？
560:デフォルトの名無しさん
18/07/30 19:56:43.56 xHVHgAPAa.net
>>547
その解釈が恐らく正しいと思うが、元の文章は素直に読むとそうならんよなあ。
561:デフォルトの名無しさん
18/07/30 19:58:59.40 WDgnZ3L+0.net
値の大きい順に上位３位なら
VAL_T data[100];
入力値をつっこんで data をソートして data[0], data[1], data[2], を出力する
int sortcmp(const void* c1, const void* c2)
{
VAL_T d1 = *(const VAL_T*)c1;
VAL_T d2 = *(const VAL_T*)c2;
if (d1 > d2) return -1;
else if (d1 < d2) return 1;
return 0;
}
・・・・
qsort(data, num, sizeof(VAL_T), sortcmp);
for (i=0; i<3 && i<num; i++)
printf("%d: %lg\n", i+1, (double)data[i]);
・・・・
562:デフォルトの名無しさん
18/07/30 20:08:25.88 aSIGzv/D0.net
>>550
そういう解釈も可能
563:デフォルトの名無しさん
18/07/30 20:10:45.97 aSIGzv/D0.net
色々な解釈が可能な文
仕様書からコードにするときには
複数の解釈が無いか気にした方が良い
564:デフォルトの名無しさん
18/07/30 20:17:02.90 kECQyXHfp.net
すみません。値が大きい順です。お願いします。
565:デフォルトの名無しさん
18/07/30 20:21:05.16 Ei1MMj+i0.net
良い試験問題だなｗ
566:デフォルトの名無しさん
18/07/30 20:23:08.08 aSIGzv/D0.net
ずこーっ
上位3個でqsortとか使うのはアホ
567:デフォルトの名無しさん
18/07/30 20:25:48.38 r0zEs9EBa.net
ああ。sortコマンドとheadコマンド使いたくてうずうずする。
568:デフォルトの名無しさん
18/07/30 20:30:04.49 r0zEs9EBa.net
上位三つなら全部チェックすりゃあ良いんじゃないか?
ループとif文3つか?
569:デフォルトの名無しさん
18/07/30 20:39:55.63 WDgnZ3L+0.net
なんやソート使えという制限はないのか
570:デフォルトの名無しさん
18/07/30 22:01:25.27 WHL4n1uA0.net
ｺｺではquickselectというｱﾘｺﾞﾘｽﾞﾑを使いなさい
というのが正解
やっぱりなこのｽﾚは浅はかなｸﾙｸﾙﾊﾟｰしかいないわ
571:デフォルトの名無しさん
18/07/30 22:06:32.60 Ei1MMj+i0.net
>>560
上位3名に入れそうだね？
572:デフォルトの名無しさん
18/07/30 22:17:34.44 aSIGzv/D0.net
quickselectもアホ
上位3個なら原始的なアルゴリズムが一番速い
573:デフォルトの名無しさん
18/07/30 22:18:31.28 aSIGzv/D0.net
速くて簡単でリソースも使わない
574:デフォルトの名無しさん
18/07/30 22:19:07.01 Z5guDo9v0.net
課題みたいだし
・バブルソートを書けるか
・保持する要素は3つでいいことに気付けるか
をみる定型問題だろね
575:デフォルトの名無しさん
18/07/31 08:53:15.13 CiY/UjgCK.net
アルゴリズムもわかってない
まず、問題解決のための適切なアルゴリズムの選択ができない
クイックソートは明らかに適切ではない
ここではクイックセレクトが適切
確かにこのスレには
まともな教育を受けたのはいない
基本的になにも知らなすぎる
白痴に近い
576:デフォルトの名無しさん
18/07/31 09:25:34.54 quzd6t9e0.net
クイックセレクトってどんなアルゴリズムなん？
元はクイックソートだけどソートしないで選択するだけって感じ？
577:デフォルトの名無しさん
18/07/31 09:39:58.84 +/oTEV5Ja.net
>>566
無駄な比較をしない感じ
578:デフォルトの名無しさん
18/07/31 11:52:18.67 Sty7sRvnH.net
>>566
パーティショニングして目的の順位を含む方だけ再帰
579:デフォルトの名無しさん
18/07/31 12:06:11.93 TL2AM5+0M.net
あー、そういうことか
580:デフォルトの名無しさん
18/07/31 12:43:01.65 xftY0DAGd.net
明らかに上位3個の抽出には向かない
581:デフォルトの名無しさん
18/07/31 12:45:03.91 xftY0DAGd.net
最大値を探すのにクイックセレクトを使うアホはいない
そういうこと
3個でも同じ
582:デフォルトの名無しさん
18/07/31 13:12:03.24 kZi5wZ5a0.net
入力毎に判定。
3プールのmin以下なら無視。
maxより大きいなら順位シフト。
mid以下ならmin上書き。
それいがいはmidをminにしてmid上書き。
583:デフォルトの名無しさん
18/07/31 13:17:53.66 xftY0DAGd.net
そういう原始的なアルゴリズムが最良
584:デフォルトの名無しさん
18/07/31 13:21:36.88 xftY0DAGd.net
最悪パターンは小さい順に並んだ時だが
これでもせいぜい数倍になるだけ
クイックセレクトの最悪値は酷い
もちろん典型例でも原始的アルゴリズムの方が速い
元データも変更不要
メモリも3個分だけ
585:デフォルトの名無しさん
18/07/31 13:29:26.70 NZiCEZHW0.net
1,ソート対象...data[5, 3, 12 ,8 ,6]
2,メモリをソート対象の最大値個数分確保...a[12]
3,ループでa[5]=data[5], a[3]=data[3]...と全部入れる
4.詰めてオシマイ
史上最速ソートだと思わないか？ループ1回で終わるどｗ
586:デフォルトの名無しさん
18/07/31 13:45:59.26 j6W8yzMq0.net
>>575
バケットソートだね～
値の重複がある場合はリストをつなぐ必要があるし
今回の問題は要素が実数値だから不適になっちゃうけど
587:デフォルトの名無しさん
18/07/31 13:49:32.57 hTiFVqvS0.net
質問内容によるとソート対象は実数値なんだが
588:デフォルトの名無しさん
18/07/31 13:49:52.96 NZiCEZHW0.net
>>576
あ、そういう名前で実際あるんだね・・・
素晴らしいアイディアだと思ったのに(´・ω・｀)
589:デフォルトの名無しさん
18/07/31 14:44:47.46 riS9l2jZ0.net
値をいれる必要はない。
各要素を出現回数のカウンタにすれば重複数も記録できる。
だがこれができるのは入力が整数で表せる場合だけ。
590:デフォルトの名無しさん
18/07/31 14:46:44.33 UZkSYp0La.net
ま、しかし、普通の数値で100しかデータがないと今時のPCだとどんなアルゴリズム使っても人間が感じられないぐらいのスピードで実行しちゃうと思う。
わざとsleepとかするなら別だが。
591:デフォルトの名無しさん
18/07/31 14:57:40.88 PCFGvven0.net
今は熱対策が必要だな
592:デフォルトの名無しさん
18/07/31 17:49:39.11 xftY0DAGd.net
>>580
桁数が異常に長い入力とか
チープなマイコンとか
まあ色々と考えられる
593:デフォルトの名無しさん
18/07/31 17:55:01.37 UZkSYp0La.net
あー。1万桁のデータが100個とか。w
594:デフォルトの名無しさん
18/07/31 17:58:48.84 zWbJ6Mhya.net
スレッドの引数に渡す構造体を↓みたいにして、
クラスのメンバ関数ポインタを渡そうとしたんですがエラーになります。
info.pfunc = test.callの部分はどう書くのが正解ですか？
typedef struct
{
void (*pfunc)();
} TInfo;
class TestClass
{
public:
void Call(void)
{
cout << "呼んだ" << endl;
}
};
main()
{
TestClass test;
TInfo info;
info.pfunc = test.call;
}
595:デフォルトの名無しさん
18/07/31 18:03:58.43 Yh1Yhi2x0.net
最近はC言語にクラスがあるの？
596:デフォルトの名無しさん
18/07/31 19:15:37.92 NZiCEZHW0.net
static void Call()
597:デフォルトの名無しさん
18/07/31 20:20:43.19 4WdKzGL90.net
#include <stdio.h>
typedef struct tag_baka_t t_baka_t;
typedef int (*baka_func_t)(t_baka_t const* pt_baka);
typedef struct tag_baka_t {
char* text;
baka_func_t function;
} t_baka_t;
int call_baka1(t_baka_t const* ct_baka) { fprintf(stdout, "baka1:%s\n", ct_baka->text); return 1; }
int call_baka2(t_baka_t const* ct_baka) { fprintf(stdout, "baka2:%s\n", ct_baka->text); return 2; }
int call_baka3(t_baka_t const* ct_baka) { fprintf(stdout, "baka3:%s\n", ct_baka->text); return 3; }
int call(t_baka_t const* ct_baka) {
return ct_baka->function(ct_baka);
}
int main(int argc, char* argv[]) {
t_baka_t baka1 = { "boo", call_baka1 };
t_baka_t baka2 = { "foo", call_baka2 };
t_baka_t baka3 = { "woo", call_baka3 };
fprintf(stdout, "called baka1 -> returned %d\n", call(&baka1));
fprintf(stdout, "called baka2 -> returned %d\n", call(&baka2));
fprintf(stdout, "called baka3 -> returned %d\n", call(&baka3));
return 0;
}
C言語の構造体ならｺﾚでいける
ｺﾚならﾊﾞｶでも分かるﾊｽﾞ
C++の構造体でも同じように書ける
598:さまよえる蟻人間
18/07/31 20:29:50.71 qpn2fVfCd.net
ばかばっか
599:デフォルトの名無しさん
18/07/31 20:29:50.72 V319xmHP0.net
>>584
節子それc言語やないで
600:デフォルトの名無しさん
18/07/31 20:33:19.91 j6W8yzMq0.net
>>584
完全にC++ですやん
URLﾘﾝｸ(ideone.com)
とりあえずstd::functionの使いどころかなとは思うけど
601:デフォルトの名無しさん
18/07/31 21:00:04.51 YzKZx0BTa.net
>>590
これっぽいですね、ありがとうございます
オブジェクトポインタと関数ポインタを別スレッドに渡して、
スレッドでその関数を定期的にコールすることをしたかったんです
602:デフォルトの名無しさん
18/07/31 23:06:10.75 ljgZb1RQ0.net
次からはC++の質問はC++のスレへ。
603:デフォルトの名無しさん
18/08/01 16:42:35.83 9yb7YrdQ0.net
クイックソートのwikipediaに書いてあるc言語のソースに、
【value_type tmp, pivot = med3(a[i], a[i + (j - i) / 2], a[j]); /* (i+j)/2 ではオーバーフローしてしまう */】
と書いてあるのですが、「(i+j)/2ではオーバーフローしてしまう」のオーバーフローする場合がまったくわかりません。
教えていただけますか。
604:さまよえる蟻人間
18/08/01 17:40:34.16 TGp/sEuFd.net
i, jがどんな値かによる。
i == 0x7fffffff && j == 0x7fffffff
で、i + jがどんな整数値になるか考えたまえ。
605:デフォルトの名無しさん
18/08/01 18:09:35.03 9yb7YrdQ0.net
ありがとうございます。理解出来ました。
下限上限への理解がまだまだでした…
606:デフォルトの名無しさん
18/08/01 18:20:26.56 edgaccID6.net
>>593
unsigned char i=128,j=128,k;
k=i+j;
607:デフォルトの名無しさん
18/08/01 19:46:12.42 xh0kKqMn0.net
>>584が578のやり方でtest.Callのアドレスを渡せないのはなんでなん？理由が知りたい。
608:デフォルトの名無しさん
18/08/01 20:21:01.54 jfFEYUCE0.net
>>597
まず瑣末だけど、 test.call じゃなくて &Test::Call でメンバ関数のポインタを取らなきゃいけないってのはあるけど それはいいとして
ざっくり言えば「メンバ関数の型はクラスの型を含んでて、普通の関数とは型違うから」ってとこかねー
# g++
エラー: cannot convert ‘void (TestClass::*)()’ to ‘void (*)()’ in assignment
info.pfunc = &TestClass::Call;
スレチなので 解答が不足ならば「C++相談室」か「スレ勃てるまでもないC/C++の質問はここで」辺りで願います
詳しい人いるだろうし
609:デフォルトの名無しさん
18/08/01 20:28:43.32 xh0kKqMn0.net
>>598
そのざっくりな説明だけでもわかりやすい。なるほどありがとう。
610:デフォルトの名無しさん
18/08/01 20:34:52.96 xh0kKqMn0.net
ググったら安定のロベールさん出てきた。買った方がいいなこれ基礎力でえらい差が出てしまう気がしてきた
611:デフォルトの名無しさん
18/08/01 20:52:58.87 WDXOLOCva.net
C++の話はC++スレでやれ。
612:デフォルトの名無しさん
18/08/01 21:33:20.86 jJEMbL0b0.net
なんだ結局ロベールのステマかよしょうもねえな
613:デフォルトの名無しさん
18/08/01 22:23:36.11 ArWxfDp8a.net
>>598
分かりやすい説明
614:デフォルトの名無しさん
18/08/02 00:25:44.59 6TeXhDWV0.net
ｵｰﾊﾞﾌﾛｰ以前に
ﾎﾟｲﾝﾀ同士の足し算なんかしないからな（Cでは普通のやりかたではできない仕様になってる）
ﾎﾟｲﾝﾀ同士の足し算なんかしたらおもいっきりｵｰﾊﾞｰﾌﾛｰすることがあるからな
つまり簡単にｵｰﾊﾞｰﾌﾛｰはおきうる
ﾎﾟｲﾝﾀにはｱﾄﾞﾚｽが入ってるからな
やってることは
基準になるｵﾌｾｯﾄ（ここでは左端）に距離を足すだけといっていい
距離は通常ﾎﾟｲﾝﾀの引き算で求める
距離は左端と右端の距離の半分の距離に
基準になるｵﾌｾｯﾄに足す
すると中間点になる
つまり中間点固定
しかしﾋﾟﾎﾞｯﾄが中間点固定だと最悪のｹｰｽの場合
ｸｲｯｸｿｰﾄはものすごく遅くなる
で、ﾋﾟﾎﾞｯﾄを乱数で選択することで
その最悪のｹｰｽを回避するというやりかたもある
実際の現実のﾃﾞｰﾀでそんな最悪のｹｰｽはほぼないといっていい
615:デフォルトの名無しさん
18/08/02 00:29:26.39 c3/ck1cgd.net
すいません初心者です、質問なんですが・・・
for文を使うときにfor(i = 1 ;i <= 5 ;i++ )でiが5より増えたときにループを抜けるようにiを1ずつ増やすというのは分かるんですが
何故ループするときにfor文のi = 1の代入の部分は無視されるんでしょうか・・・
616:デフォルトの名無しさん
18/08/02 00:39:59.52 6TeXhDWV0.net
そういう制御文だからな
そういう仕様だ
for ( [ﾙｰﾌﾟ前に行う演算]; [ﾙｰﾌﾟが処理される条件]; [次のﾙｰﾌﾟが処理される前に行う演算] )
i = 1;
whlle (i <= 5) {
・・・
i++
}
と同じ
どれもすべて入れても入れなくてもいい
for (;;) こう書くと無限ﾙｰﾌﾟになる
617:デフォルトの名無しさん
18/08/02 00:43:35.36 c3/ck1cgd.net
なるほどそういう仕様だったんですね・・・
丁寧にありがとうございますこれでようやく眠れる
618:デフォルトの名無しさん
18/08/02 01:18:26.80 4HyEhcJnM.net
>>606
continueがあるとちょっと書き換えが必要だけどね
619:デフォルトの名無しさん
18/08/02 06:45:33.31 S4Pms+eVM.net
>>608
書き換え？
i = 1;
whlle (i <= 5) {
・・・
goto CONTINUE;
・・・
CONTINUE:
i++
}
ってやるだけだろ
620:デフォルトの名無しさん
18/08/02 07:13:00.94 PxPZ0mdd0.net
それが書き換え
ってのが>>608の主張だろ
621:デフォルトの名無しさん
18/08/02 09:43:21.84 NSwKZwuS0.net
for文でのcontinueの挙動を最近知ったばかりで自慢したかったんだろ
622:デフォルトの名無しさん
18/08/02 09:48:48.46 Bu8RCR+yd.net
if
for
while
do while
goto
break
continue
このくらいしか無いんだから全部覚えとけ
なぜかgotoの使い方を知らんヤツが多い
多重ループを抜けるのにわざわざフラグを使うアホとか
623:デフォルトの名無しさん
18/08/02 10:51:33.79 NSwKZwuS0.net
使い方を知らんというより
絶対使うなと刷り込まれているのが多い
なぜだと聞いても答えられないやつ全員
624:デフォルトの名無しさん
18/08/02 12:55:13.95 Cp7uV85Za.net
gotoは悪という教義のある宗教の信者だから仕方がない。
625:デフォルトの名無しさん
18/08/02 13:54:18.40 4fjj7SMLd.net
大体使わなくて済むし
使う必要がある時に
さりげなく使うもの
626:デフォルトの名無しさん
18/08/02 14:17:14.48 TfLreE+60.net
全く使わない 脱出用途なら使う 飛び込みでも使う
627:デフォルトの名無しさん
18/08/02 16:25:02.06 SFe9zIxEa.net
脱出時だけ使ってるな。
関数の途中で複数エラーが出る可能性がある場合とかは関数の最後の方にgotoさせてそこでまとめてメモリ解放などをする。
他の言語だと例外投げて try catch でなんとかするようなやつだな。
628:デフォルトの名無しさん
18/08/02 18:05:39.94 QNpgl+2ZM.net
飛び込みはやめーや
629:デフォルトの名無しさん
18/08/02 18:57:08.60 yzfiTIk96.net
脱出はreturnにすればいい
630:デフォルトの名無しさん
18/08/02 19:01:34.31 xZR22mJ60.net
returnが複数あるのを嫌がる宗派があるからなあ
631:デフォルトの名無しさん
18/08/02 19:11:54.12 1EegAYN5d.net
>>619
宗教の信者さん？
632:デフォルトの名無しさん
18/08/02 19:12:56.06 jJ76BOdj0.net
初期化(1) if (エラー) goto E1;
初期化(2) if (エラー) goto E2;
初期化(3) if (エラー) goto E3;
：
return おっけー
E3: (3) の後始末
E2: (2) の後始末
E1: (1) の後始末
return エラーじゃ
633:
18/08/02 19:15:38.12 hFhSdR7O0.net
>>620
確かに、misra が全力でそれを嫌うのは、なにか理由があってのことでしょうか？
634:デフォルトの名無しさん
18/08/02 19:20:08.81 zwgkA1lD0.net
>>622
俺もこれ派だわ
635:デフォルトの名無しさん
18/08/02 19:26:51.94 1EegAYN5d.net
○○派とか言うと他の方法を知らないように感じる
臨機応変に色々の方法を使い分けるのが良い
636:デフォルトの名無しさん
18/08/02 19:33:21.75 xZR22mJ60.net
コーディング規約を決める偉い人の信仰心に依存する、か
637:デフォルトの名無しさん
18/08/02 19:35:06.17 zwgkA1lD0.net
>>625
あとよく使うのがagainだな
ループでagainしたい時に上に飛ぶやつ
638:デフォルトの名無しさん
18/08/02 19:41:55.73 1EegAYN5d.net
コーディング規約を偉い人が決めるような会社は嫌だな
639:デフォルトの名無しさん
18/08/02 19:43:01.73 1EegAYN5d.net
偉い人はもっと大きな事を考えて
640:デフォルトの名無しさん
18/08/02 20:10:50.62 Cjo1mv3c0.net
エラー起こす前にちゃんと検査しろ。
641:デフォルトの名無しさん
18/08/02 20:30:01.75 1NEFWSVpM.net
retryとoutは使うわ
後始末不要ならその場でreturn
後始末必要なら最後にreturn
まあこの辺の書き方より関数名、変数名、変数の数を気にして欲しいと思う今日この頃
642:デフォルトの名無しさん
18/08/02 22:24:28.26 NSwKZwuS0.net
Cのgotoが関数から出られない、という時点で
悪しきgotoの大部分が封じられているんだけど
せっかくそうなって�
643:｢るのに１関数で数千行なんてやらかす 何というか吹っ切れたやつが暗黒面に墜ちていると とんでもねえんだよ
644:
18/08/02 22:30:24.78 hFhSdR7O0.net
>>632
＞Cのgotoが関数から出られない、という時点で悪しきgotoの大部分が封じられている
ところがどっこい、setjmp()/longjmp() というものがありまして…
645:デフォルトの名無しさん
18/08/02 22:43:04.03 xZR22mJ60.net
longjmpでイスカンダルまで飛びたいんですけど
何故か地球に戻ってしまいます
646:デフォルトの名無しさん
18/08/03 02:01:51.21 TY6Sib0f0.net
イスカンダルのトーフ屋
647:デフォルトの名無しさん
18/08/03 05:37:20.02 o9dqmONZ0.net
孫悟空はsetjmp/longjmpの使い手
648:デフォルトの名無しさん
18/08/03 09:55:27.36 UvsVeV1Q0.net
組込み人は状態変数でワープする。
649:デフォルトの名無しさん
18/08/03 10:02:42.04 8VyIMXmS0.net
スタック上の戻りアドレスを自分で操作して、好きな場所にワープさせてた奴いたっけな
650:デフォルトの名無しさん
18/08/03 10:12:18.20 9KGEdKKU0.net
>>633
longjmpで悪しきgotoを
やって見せてくれ
ideoneで
651:デフォルトの名無しさん
18/08/03 14:40:50.79 9KGEdKKU0.net
悪しきgotoの大部分を封じるときに
巻き添えで封じられた悪しくないgotoを救う
アイテムがlongjmpだろうがよ
652:デフォルトの名無しさん
18/08/03 19:36:07.75 4c00FTp80.net
gotoの効率の良さは取立ての生うにの味
653:デフォルトの名無しさん
18/08/03 20:15:47.20 X+nM2ZWla.net
gotoは良く切れる刀のようなもの。
その怪しい魅力に取り憑かれダークサイドに堕ちた者は数知れず。
654:デフォルトの名無しさん
18/08/03 20:42:44.92 6lqoN/6FM.net
昔のBASICはその鋭い刃しかなかったってことか
655:デフォルトの名無しさん
18/08/03 20:43:55.37 I1YZhHYF0.net
GOSUBとかなかった？
656:デフォルトの名無しさん
18/08/03 20:49:49.72 8VyIMXmS0.net
fortranからの伝統芸
657:デフォルトの名無しさん
18/08/03 21:11:54.24 1LNshpyd0.net
その昔サブルーチンの存在を知らなかった頃
gotoだけでスタックオーバーフロー的な
破綻をさせないで組んだのは結構自分の糧になってる
大人になってコーディング規約を作る側になった時
脱出のみ使用可能ってルールを作ったら
頑なに拒否するメンバーがいて仕事を外れてもらったことがる
宗教じみた反応でちょっと怖かった
658:デフォルトの名無しさん
18/08/03 22:31:36.61 yp/iEsny0.net
gotoなんかよほどの理由がない限り使わない
適切に構造化できないｱﾎが使うとﾛｸなことにならない
原則禁止でいい
659:デフォルトの名無しさん
18/08/03 23:20:08.07 yp/iEsny0.net
>>622なんかではgoto使うｹｰｽなんかほぼない
普通にｺｰﾄﾞ書いてればgotoなんかなくなる
後始末の仕方が判定できないｺｰﾄﾞを書くと
初期化が終わったあと、その初期化がむしろ不要になった場合その後始末が大変になる
知恵遅れほどいちいち不要なgotoを使いたがるからな
#define ERROR_INIT_AHO0 -1
#define ERROR_INIT_AHO1 -2
#define ERROR_INIT_AHO2 -3
#define ERROR_INIT_AHO3 -4
typedef struct tag_aho { char* p1; int fd2; FILE* fp3; } t_aho_t;
int 初期化(t_aho_t** ppt_aho) {
t_aho_t* pt_aho;
*ppt_aho = NULL;
if ((pt_aho = malloc(sizeof(t_aho_t)) == NULL) { return ERROR_INIT_AHO0; }
*ppt_aho = pt_aho;
if ((pt_aho->p1 = malloc()) == NULL) { 後始末(pt_aho); return ERROR_INIT_AHO1; }
if ((pt_aho->fd2 = open()) < 0) { 後始末(pt_aho); return ERROR_INIT_AHO2; }
if ((pt_aho->fp3 = fopen()) == NULL) { 後始末(pt_aho); return ERROR_INIT_AHO3; }
return 0;
}
void 後始末(t_aho_t* pt_aho) {
if (pt_aho == NULL) return;
if (pt_aho->p1 != NULL) { free(pt_aho->p1); }
if (pt_aho->fd2 > 2) { close(pt_aho->fd2); }
if (pt_aho->fp3 != NULL) { fclose(pt_aho->fp3); }
free(pt_aho);
}
660:デフォルトの名無しさん
18/08/03 23:26:42.29 Fao22p7ya.net
gote使う前にそのループを関数化で分ける
661:デフォルトの名無しさん
18/08/03 23:29:40.43 yp/iEsny0.net
正解！
662:デフォルトの名無しさん
18/08/03 23:34:32.92 yp/iEsny0.net
>>648の簡単な例なら最初のmallocのあと
pt_aho->p1 = NULL;
pt_aho->fd2 = -1;
pt_aho->fp3 = NULL;
と初期化しておくだけでどう後始末すれば判定できる
663:デフォルトの名無しさん
18/08/03 23:44:52.51 8VyIMXmS0.net
>>651
後始末ってエラー時だけじゃなく終了時にも使いたいから
未使用かどうかを分かる様にしとくと色々便利だよね
664:デフォルトの名無しさん
18/08/04 00:39:50.00 iUNKVgUH0.net
Linux は、goto でエラー処理をまとめてる
return も、2箇所以上で使っている。
関数の冒頭で、処理しないものを、return ではじいたりしてる
MISRA-C のルール違反
トヨタなんか、MISRA-C で、5/100点とか聞くw
ルールの95% に違反w
665:デフォルトの名無しさん
18/08/04 04:04:56.84 2zclIjyW0.net
>>648
それだと毎回その関数用の構造体を作らねばならなくなっていまいち面倒ではないか？俺は>>622とその方式組み合わせたような感じにしてるよ。
int func()
{
int r = -1;
char *p1 = NULL;
int fd2 = -1;
FILE *fp3 = NULL;
if ((p1 = malloc()) == NULL) goto func_exit;
if ((fd2 = open()) == -1) {
r = -2;
goto func_exit;
}
if ((fp3 = fopen()) == NULL) {
r = -3;
goto func_exit;
}
/* 処理 */
// ここで p1, fd2, fp3 を使い終わった場合は func_exit より後でメモリ開放や
// クローズされないようにするために -1 や NULL を代入する。
r = 0; // 成功
func_exit:
if (p1) free(p1);
if (fd2 != -1) close(fd2);
if (fp3) fclose(fp3);
return r;
}
666:デフォルトの名無しさん
18/08/04 05:10:46.07 YA1hWK5Y0.net
>>647
適切に構造化できないｱﾎさん乙
667:デフォルトの名無しさん
18/08/04 05:53:52.28 I5yMWHTla.net
>>654
goto使わず普通に関数呼べばいいだけでは
668:デフォルトの名無しさん
18/08/04 07:40:44.39 YA1hWK5Y0.net
int func()
{
int r = 1;
//無駄な初期化はしない
char *p1;
int fd2;
FILE *fp3;
p1 = malloc(); if(p1 == NULL) goto
669:malloc_failure; fd2 = open(); if(fd2 == -1) goto open_failure; fp3 = fopen(); if(fp3 == NULL) goto fopen_failure; r = 0; // 成功 //フラグの類は極力使わない fclose(fp3); fopen_failure: close(fd2); open_failure: free(p1); malloc_failure: return r; }
670:デフォルトの名無しさん
18/08/04 08:12:49.13 mQQzn2Q+0.net
gotoを使った方がシンプルで見易いコードになるにも関わらず
頑なに使わない信者さん
>>612の振りで始まったにも関わらずたくさん出てくるね
671:デフォルトの名無しさん
18/08/04 08:15:22.02 mQQzn2Q+0.net
わざわざ専用構造体と専用後始末関数と専用エラーコードをつくってわざわざ複雑にしてるアホもいるし
変数名で自己紹介してるから自覚はあるんだろうけど
672:デフォルトの名無しさん
18/08/04 08:22:18.05 mQQzn2Q+0.net
初期化と後始末を包んだ関数もいるね
十数行で済むシンプルな関数に
何十行使うつもりだか
673:デフォルトの名無しさん
18/08/04 08:32:14.97 cyc8bEPyM.net
Ok, so I'm looking at your code and your code is pure and utter *shit*.
674:デフォルトの名無しさん
18/08/04 08:42:16.32 +vznLLf60.net
goto hell
675:デフォルトの名無しさん
18/08/04 09:32:53.84 AH7fLvgn0.net
>>653
MISRAの話だとそういう制御の話がよく取り上げらてて俺もほとんど守ってないけど、データ変換の話はよく考えられててなかなか為になるよ
676:デフォルトの名無しさん
18/08/04 09:33:52.60 AH7fLvgn0.net
>>656
最初の一文も読めないよのかよ…
> それだと毎回その関数用の構造体を作らねばならなくなっていまいち面倒ではないか？
677:デフォルトの名無しさん
18/08/04 10:21:26.55 CpwGeL+S0.net
ｱﾎは後始末しないのもわかったわ
初期化がおわったあと
ﾌｧｲﾙあけたらずーっとあけっぱなし
ﾋｰﾌﾟ作ったら作りっぱなし
なにがおきようが関数抜けたらずーっと放置
まずその部分で完結したｺｰﾄﾞが書けない
だいたいどんな知恵遅れがｺｰﾄﾞ書いてんのか分かった
ｱﾎが自己紹介のｺｰﾄﾞ書いてるしな
次ページ最新レス表示スレッドの検索類似スレ一覧話題のニュースおまかせリスト▼オプションを表示暇つぶし2ch
1904日前に更新/274 KB
担当:undef