コーディングスタイルにこだわるスレ

**デフォルトの名無しさん** · 2007/10/28(日) 15:59:01

コーディングスタイルについて熱く語れ

**デフォルトの名無しさん** · 2011/02/07(月) 12:32:07

>>666
いや、俺は組み込み屋なんだが。
幸いにして関わったプロジェクトでは C++ コンパイラが使えたんで、
何の問題も無く使えている（ような気がする？）。
もちろん OS のある環境とまったく同じというわけにはいかないけど、
少なくともスマートポインタを使わない理由は無い。現に使えている。

なんだろうな？
まぁもうちょっと調べてみるよ。

**デフォルトの名無しさん** · 2011/02/07(月) 12:39:41

スマートポインタは何を解決してくるの？
deleteの書き忘れ？
deleteを書き忘れちゃうようなコードを書く人のためのもの？

**デフォルトの名無しさん** · 2011/02/07(月) 13:33:50

しょぼい組み込みだとコンパイラがないとか
コンパイラの信頼性とか、そういう世界なのかな。

**デフォルトの名無しさん** · 2011/02/07(月) 14:26:40

>>668
マーフィーの法則によるとdeleteを書き忘れない人は存在しない

**デフォルトの名無しさん** · 2011/02/07(月) 14:35:02

()や{}を書く時セットで両方記入しておいて、
カーソル移動して中身を後から書いてる。
同様に、new書いたらdeleteを直後に書く。

直後じゃなかったとしても、生成と破棄はセットで設計してるから大丈夫。
対になるものはいつも対応を取りながら進めていく。

**デフォルトの名無しさん** · 2011/02/07(月) 14:57:55

>>667
ちなみにターゲットマシンのCPUは何でございましたでしょうか？

**デフォルトの名無しさん** · 2011/02/07(月) 17:30:18

>>671
だからそれを人力でやってるのが駄目なんだっつーの。
そりゃ普通そう書くよ。

**デフォルトの名無しさん** · 2011/02/07(月) 23:04:20

>671
変なフレームワークや仕様を強要されてる場合、newとdeleteを
別々のクラス/関数/スレッド/etcで実行せざるを得なかったりして
必ずしも1対1にできるとは限らないと思う。

**デフォルトの名無しさん** · 2011/02/07(月) 23:29:24

>>671
そのやりかただと return や例外やその他いろいろ破綻する可能性が残る。

>>668 スマートポインタはこういった問題を防ぐためにある。

**デフォルトの名無しさん** · 2011/02/07(月) 23:31:54

>>672
ARM7,9,11 と PowerPC 系が少し。

**デフォルトの名無しさん** · 2011/02/08(火) 01:00:25

> newとdeleteを別々のクラス/関数/スレッド/etcで実行せざるを得なかったりして

そういうときはまず設計を改めたいものだ。
生成期間の管理があやふやになってスマポに頼るような時、
すでに半分棺桶に体突っ込んでるようなもの。どの道破綻する。

**デフォルトの名無しさん** · 2011/02/08(火) 01:23:28

>>676
ゲーム? のはずないか。
そのあたりは c++ いけるしな。

**デフォルトの名無しさん** · 2011/02/08(火) 07:58:22

>>677
> そういうときはまず設計を改めたいものだ。
メッセージ作って他のスレッドに投げてそのスレッドがさらにまた…
ってな事はしないのか？

**デフォルトの名無しさん** · 2011/02/08(火) 14:14:39

スマポが嫌われるのは、多分
・どれを weak_ptr にするかでどうせ後々面倒が起こって、それじゃ delete 忘れ修正の手間と大差ない
・余分なメモリアロケーションが発生するのが嫌だ
のどちらかが大きいのではなかろうか。

参照カウンタではなくリンクリストで実装された侵入型ならどちらも回避できる。
が、リンクリストでの実装は参照カウンタに比べて遅いのだとか。

スマポが delete 忘れを解決するというが、たとえばコンストラクタで new したものを
delete する程度なら、調査追跡も楽だから、スワポなんて無くていい。

スワポがあると助かるのは、例えば、寿命の管理が面倒な場合など。
例えば、何らかのキューにオブジェクトを突っ込む必要があるような場合、突っ込んだ
側にとってはもう不要だが、キューから取り出す側はまだ delete されると困る、という
ような場合。

他には例えば、 Factory メソッドで生成するとか、new するオブジェクトと delete する
オブジェクトが違う場合などは、delete 忘れの追跡が面倒になることもあるから、スワポ
が便利。

スワポが必要になる状況が無いとか、ハンドルとかのスワポ以外の対策があるなら、
別に無理に使う必要は無い。

>>679
俺はスマポ反対ではないが、そういうときにリソースをやりとりして
誰の持ち物か分からなくなるようなのは行儀悪すぎるだろう。

**デフォルトの名無しさん** · 2011/02/08(火) 14:32:50

>>680
> ・どれを weak_ptr にするかでどうせ後々面倒が起こって、それじゃ delete 忘れ修正の手間と大差ない
> ・余分なメモリアロケーションが発生するのが嫌だ

なら std::auto_ptr や boost::scoped_ptr を使わない理由は無いな。

> スマポが delete 忘れを解決するというが、たとえばコンストラクタで new したものを
> delete する程度なら、調査追跡も楽だから、スワポなんて無くていい。

そのやりかただと代入やコピーコンストラクタや例外やその他いろいろ破綻する可能性が残る。

> スワポがあると助かるのは、例えば、寿命の管理が面倒な場合など。

スワポってなぁに？

> 俺はスマポ反対ではないが、そういうときにリソースをやりとりして
> 誰の持ち物か分からなくなるようなのは行儀悪すぎるだろう。

boost::scoped_ptr なら移動不可能な単一所有権。
std::auto_ptr (std::uniqe_ptr) なら移動可能な単一所有権。
boost::shared_ptr なら共有される所有権。

単一所有権なら誰の持ち物かは明白。
共有される所有権が誰の持ち物か分からなくなるのは、まぁ当然だよね。

**デフォルトの名無しさん** · 2011/02/08(火) 14:33:54

s/std::uniqe_ptr/std::unique_ptr/

**デフォルトの名無しさん** · 2011/02/08(火) 15:16:56

> 他には例えば、 Factory メソッドで生成するとか

createみたいなメソッドはもう格上げして、
使う側からしたら　(new, create) <--> delete
という扱いにすればその心配はないかも。

deleteの追跡とやらは、createを呼び出した側での話しになる。

**デフォルトの名無しさん** · 2011/02/08(火) 21:24:29

>>658
> なんせ未だにカラム制限のあるコーディング規約もあるくらいだ。
Linux カーネルとか *BSD とか Solaris とかのの話か?

**680** · 2011/02/08(火) 23:55:19

>>681
> なら std::auto_ptr や boost::scoped_ptr を使わない理由は無いな。
scoped_ptr の類まで嫌う人たちがいるのかな？

> スワポってなぁに？
市況2板を見た後だったので FX 脳になってた。

**デフォルトの名無しさん** · 2011/02/09(水) 00:31:41

>>684
んにゃ、某大手家電メーカーの特定部署。1行80カラム以内で
コメントは(行単位でないなら)41カラムだかどこだかより前に出現してはいけない。
インデントも確か特殊だった記憶がある。

**デフォルトの名無しさん** · 2011/02/09(水) 10:18:54

あー、そういう環境じゃしょうがないよね。

**デフォルトの名無しさん** · 2011/02/13(日) 00:26:14

>>686
富士通みたいな所だな。

**デフォルトの名無しさん** · 2011/02/22(火) 12:38:52.11

某fj2で、携帯作ってるトコなんかだと
自動的にテスト仕様書を生成するために

//:　unkがmoreそうな場合はleaveする
　if (iUnk < iMore) {
　　User::Leave(EGodBlessYou);
　}

みたいに、「//:」を0カラム目にしてコメントを書け
という規定があるね

**デフォルトの名無しさん** · 2011/02/27(日) 23:18:33.78

プリプロセスの#includeとかが行頭にないとコンパイラがスルーしたことならあったな。
もう直ったのかshc

**デフォルトの名無しさん** · 2011/03/02(水) 16:28:45.71

20年くらい前かな
関数名のプリフィックスを2文字、機能単位で3文字、ユニークな部分を3文字に制限された
チームがあった

**デフォルトの名無しさん** · 2011/03/02(水) 20:36:20.17

うっせーバカ！現在進行形だ！

**デフォルトの名無しさん** · 2011/03/06(日) 19:22:45.45

今まさに、boostが使えない（or入れるのに手間がかかる）プロジェクトに居るんだが、
std::auto_ptrのみで、このスレで紹介されたような状況を対応する方法はあるのだろうか？

**デフォルトの名無しさん** · 2011/03/06(日) 19:35:04.41

>>693
コピーしたときの妙な特性さえ把握してれば普通に使える奴だよ std::auto_ptr は。
何が心配なの？

**デフォルトの名無しさん** · 2011/03/10(木) 22:21:32.27

>>694
すまん、書き方が悪かった。
一番聞きたかったのは、boost::sheard_ptrでないと解決できないような問題は存在する（多いのか）？
大抵の問題はstd::auto_ptrを駆使すれば問題を解決できるのか。

boost::sheard_ptrはboostが使える環境でないと使えない。（あたりまえだけど）
で、組込み系だと使えるライブラリが制限されていることは多いんで、
boostがあれば解決、だと、現実解にならない場面が多いのよ。

**デフォルトの名無しさん** · 2011/03/11(金) 00:39:48.77

仮に boostがあれば解決だったとして、
boostが使えない環境で boost::sheard_ptr 相当の、
あるいは必要な部分だけの機能を自作することは現実的ではないの？

**デフォルトの名無しさん** · 2011/03/11(金) 01:27:42.99

>>695
そんなもん、「作るものによる」としか言えないに決まってるじゃねーか。
心配している「状況」を挙げないと。

**デフォルトの名無しさん** · 2011/03/11(金) 19:06:21.02

>>695
スレ違い。
本を読んで自分で考えろ。その後まだ質問があれば、適切なスレで質問するといい。

**デフォルトの名無しさん** · 2011/03/20(日) 01:23:32.02

std::auto_ptr はコンテナ（vectorとか）との相性が悪い。

**デフォルトの名無しさん** · 2011/06/05(日) 05:34:53.68

横に長いソースはやめてください
フルHDのモニタでエディタ全画面表示にしてもラップするって酷いお

**デフォルトの名無しさん** · 2011/06/05(日) 06:57:16.18

フォントがでかすぎるんじゃねぇの?

**デフォルトの名無しさん** · 2011/06/06(月) 00:11:56.44

>>701
お前に俺の気持ちは分からない

**デフォルトの名無しさん** · 2011/06/06(月) 01:25:22.03

しらんがな

天使 ◆uL5esZLBSE · 2011/07/03(日) 22:33:58.14

２０１１年、Ruby,Perl,PHP,Pythonって並べたときにさ
ここで、Ruby以外を選ぶ奴ってマジでなんなんだろうな

本当にバカだな

**デフォルトの名無しさん** · 2011/07/03(日) 23:46:31.28

巣に帰れ

**デフォルトの名無しさん** · 2011/09/17(土) 23:31:20.71

C++のコンストラクタ初期化子のインデントについて

CHoge() : hoge1(0), hoge2(0), hoge3(0), ...

この初期化子を1行1メンバで並べるときインデントはどうしてますか？
VS2008だと半角スペースが2つ入るんだけどMSはそういうスタイルなのかしら

**デフォルトの名無しさん** · 2011/09/18(日) 00:35:41.14

VS って自動で埋め込まれるインデントの
量（空白３つ分とか）をユーザーが設定することはできないの？

**デフォルトの名無しさん** · 2011/11/22(火) 12:56:04.40

どうなの？

**デフォルトの名無しさん** · 2011/11/22(火) 16:45:32.56

>>700
同意だわ。読みづらい。
最近は異常に識別名が長すぎるんだよな。

**デフォルトの名無しさん** · 2012/06/02(土) 18:48:09.93

場合によって、敢えて横長にする人もいるから何とも。
右側は別に文脈を読む上では重要じゃないときとか、似たようなことを連続で書く時とか。

uy · 2012/08/14(火) 23:40:41.70

test

**デフォルトの名無しさん** · 2012/10/14(日) 16:45:51.75

スペースやインデントは
コーディングスタイルに含めるべきではない。

これらはコードの文字の長さによって
変わるもので、決まったルールなんか作れない。
短ければ改行入れないが、
ながければ改行入れるでしょう？

**デフォルトの名無しさん** · 2013/10/17(木) 22:14:47.56

スペース・タブ混在インデントは絶滅して欲しい。
一々ソース書いた奴のタブ幅設定の推測しなきゃならんとか腐りすぎだ。
スペースインデントは表示が崩れないだけまだマシだが、
2文字インデントとか2・3・3文字インデントとか変なルールで読むことを強要するあたりやっぱ腐ってる。
って訳でインデントはタブ文字にして、インデント幅はタブ幅設定で好きに設定してくれ。
桁揃えはインデント分をタブ文字、桁揃え分をスペースだと崩れにくい。

**デフォルトの名無しさん** · 2013/10/31(木) 13:15:45.62

整形ツール使えば気にならなくなるし

**デフォルトの名無しさん** · 2013/10/31(木) 13:25:22.86

今作ってるツールの設定ファイルで空白・タブ混在させてるのを思い出した
よく考えたら絶対文句言われるな作りだなｗ

**デフォルトの名無しさん** · 2013/12/14(土) 18:37:49.80

if文が深くなるの避けるために否定にして抜けるのってあり？

if A:
____spam()
____if B:
________egg()
を
if not A:
____continue

if not B:
____continue
spam()
egg()
にするの
if A and Bはなしとして

**デフォルトの名無しさん** · 2013/12/14(土) 23:59:10.77

A=true,B=falseの時の動作が違うことないかそれ

**デフォルトの名無しさん** · 2013/12/15(日) 05:35:07.74

>>716
そうやって間違う元だから、無闇と否定するのはよくない。
真理値表でも作れば間違えないだろうけれど、読む方が間違う元だ。
だからと言って、一々コメントに真理値表を書くのも間抜けだしね。

たまにどや顔でドモルガンの定理を使って書き換える奴いるけど、
そういう奴もしばしば書き換えに失敗しているよ。

**デフォルトの名無しさん** · 2013/12/15(日) 14:03:29.97

>>716
その if が、エラーチェックみたいなやつなら、普通にそうしてる。

**デフォルトの名無しさん** · 2014/03/12(水) 21:56:07.66

>>713
俺がこのスレを開いたときに考えたことと
丸っきり同じ事が書いてあってびっくりした
願わくば今のソース全部再フォーマットしたいわ

**デフォルトの名無しさん** · 2015/02/11(水) 00:06:57.40

ネストしたfor文を意図的に抜けたい時、for文の繰り返し条件のところに脱出フラグも書くのってアカン？
for (初期値; 繰り返し条件 && 脱出フラグ; インクリメント) {
}

**デフォルトの名無しさん** · 2015/02/11(水) 07:01:25.43

>>721
いったん「インクリメント」のところまで戻らせないといけないから、break 脱出い比べてイマイチ感

**デフォルトの名無しさん** · 2015/02/11(水) 10:11:15.36

>>722
for文ネストしててもbreakがいいの？
…
if(脱出条件) {
boolean 脱出フラグ = true;
break;
}
}
if(脱出フラグ) break;
}
if(脱出フラグ) break;
}

俺的には↑これ読みにくいと思ってるんだけど

**デフォルトの名無しさん** · 2015/02/11(水) 10:21:37.13

あ、すまん >>723これスコープが変だわ
boolean宣言はループ入る前にfalse入れとく

**デフォルトの名無しさん** · 2015/02/11(水) 14:03:08.03

その例なら goto で脱出するのが最善だと思う。

**デフォルトの名無しさん** · 2015/02/11(水) 15:47:13.88

C も C++ もなぜか多重ループからの脱出を実装しないね

**デフォルトの名無しさん** · 2015/02/11(水) 15:52:10.12

多重ループを関数やメソッドにつっこみreturnで脱出する方法がある

**デフォルトの名無しさん** · 2015/02/11(水) 16:10:41.43

>>727
やるだけなら goto でいいだろ

> 多重ループを関数やメソッドにつっこみreturnで脱出する方法がある

脱出するためだけにそんなことしたら分かりにくくなるだけ

**デフォルトの名無しさん** · 2015/02/17(火) 06:43:49.08

>>728
C++でもいずれ、foreachとlambdaで
ループを書くのが自然になるかも。
そうなればループ中断はreturnに
なるな。

**デフォルトの名無しさん** · 2015/02/17(火) 20:01:12.88

なんの脈絡もないおれおれ予想乙

**デフォルトの名無しさん** · 2015/02/17(火) 21:10:27.34

gotoで抜けたらスコープどうなんの？
メモリ確保されまままじゃね？
gotoにしろreturnにしろ、スコープからすっこ抜けるルーチンは好きじゃない

**デフォルトの名無しさん** · 2015/02/24(火) 12:09:36.40

スコープがわかってるなら…
Cの場合、
スコープからでたとき（関数からでたとき）、スタックにあるやつは、なくなる（なくなったも同然になる）

「開発者はほとんどの場合gotoの使用を適切に制限しており、
Dijkstra氏が懸念したような無制限な使用は行われていないことが判明した。
これらのことから、実際にはgotoは有害でない」

C言語の開発者によるgoto文の使い方を対象とした実証研究の結果、「goto文は無害だと考えられる」
| スラッシュドット・ジャパンデベロッパー
http://developers.slashdot.jp/story/15/02/14/2017207/
2015年02月15日 13時04分

**デフォルトの名無しさん** · 2016/03/29(火) 10:12:42.49

サッカーブッシュ日本代表日程ぷあたん（しゅっちょうまいくろ教育長交代）春文執行40代売上差額シュガーチョコ
https://www.youtube.com/watch?v=NDq1QoJY0nY宇ドナルドアナリストパワーストーンコーチングとしまえん
サッカーブッシュ日本代表日程古本屋よしたけしゅっちょうちょこしゅがー
ディーラー税務署天才開発者死亡詰みヨミドクターマイクロサービス不足
サッカーブッシュ日本代表日程ぷあたんシフト光金さかい強制バイト人権侵害問題
春分資源執行ニューヨーク低原価ぼったステーキソルトレイク福岡横浜新橋奴隷課金パチシフト強制バイト問題新潟米センター生残
コスメ２４チャリティー隠れ40代生活保護プレイボーイバイトレードいたりあん接待問題
マスコミKARDローンケーオーサービス不足婚活パーティー寄付金執行原発ビジネス
FBIチャイニーズタイホテル売上事務所ガチャ決算ガチャキャンペーン（販売報道陣過激派組織向携帯最新情報提供終了
校長発言細心注意ノートン産廃エラー(著作権クレーム中国反応融資高額教育費)(中国捕鯨団体40代社員サッカーコメント
高額入学金ヤフウ新橋大学ヤフウ新橋理事長FX経費おじや50代資産ガリバズフィード40代エリート