コーディングスタイルを統一する意味ってあるの？

**デフォルトの名無しさん** · 2018/04/17(火) 10:57:52.15

さすがに1つの関数内でごちゃまぜになってると気持ち悪いけど
別に関数違えばバラバラでも良くない？
統一することでなにかメリットあった？

良い書き方と悪い書き方があって、良い書き方に統一するという話ではなくて
どちらでも良い書き方のうち、どちらかに統一するのは
意味があるのか？って話ね

**デフォルトの名無しさん** · 2018/04/17(火) 12:36:46.88

func_20180417_103423とかいう文化圏で育った奴

**デフォルトの名無しさん** · 2018/04/17(火) 13:26:47.13

たしかに重箱の隅をつつくような規約は効率が良くなるかどうか疑問だな
多少の表記のゆれよりきちんとモジュール化、カプセル化されてる方が大事

**デフォルトの名無しさん** · 2018/04/17(火) 14:29:08.12

整形機能のあるIDEで１発終了だから気にしない。

**デフォルトの名無しさん** · 2018/04/17(火) 14:52:38.93

WBAWBC統一王者

**デフォルトの名無しさん** · 2018/04/17(火) 14:54:58.06

>>1
1人でやるならそれでいいんじゃね？

**デフォルトの名無しさん** · 2018/04/17(火) 16:03:42.08

>>6
スタイルがバラバラで、なにか困るんですか？

**デフォルトの名無しさん** · 2018/04/17(火) 20:11:43.68

>>1
>良い書き方と悪い書き方があって、良い書き方に統一するという話ではなくて

それは、悪い書き方を制限するってとらえた方がいい

>どちらでも良い書き方のうち、どちらかに統一するのは
>意味があるのか？って話ね

どちらでも良い(構わない)なら、統一することが良い(望ましい)書き方だ
統一は制限の一種といえるし、そうすることで誰にとっても理解しやすくなる

**デフォルトの名無しさん** · 2018/04/17(火) 20:18:18.96

そう言われてみればコーディングスタイル決めてないけど
こまったことないわ

**デフォルトの名無しさん** · 2018/04/17(火) 20:23:05.26

いや一関数の行数制限だけはマジでかけてほしい

**デフォルトの名無しさん** · 2018/04/17(火) 20:41:30.11

>>10
なんでや？

**デフォルトの名無しさん** · 2018/04/17(火) 20:56:08.59

>>11
だってほっとくと1000行超えてるのに関数に分けようとしねーし

**デフォルトの名無しさん** · 2018/04/17(火) 21:02:50.90

>>12
なんで関数分けるんや？

**デフォルトの名無しさん** · 2018/04/17(火) 21:04:24.62

ステップ数で成果を測るのが嫌いなのに
ステップ数で関数を評価しようとするみなさん

**デフォルトの名無しさん** · 2018/04/17(火) 21:15:26.81

>>12
そんな分け方で良いはず無いだろw
機能単位で最少単位に分割しても1000行超えちまうなら諦めろ

**デフォルトの名無しさん** · 2018/04/17(火) 21:35:41.97

プログラマの給料は行数ベースの歩合制にすべき

そうすればこんな手抜きコードじゃなくて
001　for (i = 0; i < 10; i++) {
002　　x[i] = func(i);
003　}

誰にでも分かりやすいこういうコードになるだろう
001　x_0 = func_0();
002　x_1 = func_1();
003　x_2 = func_2();
004　x_3 = func_3();
005　x_4 = func_4();
006　x_5 = func_5();
007　x_6 = func_6();
008　x_7 = func_7();
009　x_8 = func_8();
010　x_9 = func_9();

**デフォルトの名無しさん** · 2018/04/17(火) 22:04:23.99

>>8
> どちらでも良い(構わない)なら、統一することが良い(望ましい)書き方だ
なんで？

> 統一は制限の一種といえるし、そうすることで誰にとっても理解しやすくなる
どちらでも良いってことは、人によっては自分のスタイルじゃないものを
強制されるってことですよ？誰にとってもっていうのは違いますよね？

**デフォルトの名無しさん** · 2018/04/17(火) 22:34:52.85

>>17
せやな、日本人と韓国人と中国人が同じチームで作業したら
日本人は変数名や関数名に一番わかりやすい日本語名を付けるし
韓国人も中国人も自国語の名前を付けるのが一番効率いいわな
お互いのコードを見てもチンプンカンプンだけど開発効率を考えたらしょうがないよね

C言語のプロジェクトだけど俺はRubyが好きだから関数名は全部#defineでRubyっぽく書くよ
隣の人はCOBOLっぽいコードを書いてて全然わからんけどそれがその人にだけは一番わかりやすいんだからしょうがないね

**デフォルトの名無しさん** · 2018/04/17(火) 22:52:29.01

>>18
変数名に日本語とかヤベエ奴じゃん

**デフォルトの名無しさん** · 2018/04/17(火) 22:53:26.12

>>18
それは皮肉かなんかですか？
自然言語の違いは、コーディングスタイルじゃないでしょう？
別の話を混ぜないように

> C言語のプロジェクトだけど俺はRubyが好きだから関数名は全部#defineでRubyっぽく書くよ
これもコーディングスタイルじゃないですね

**デフォルトの名無しさん** · 2018/04/18(水) 00:30:32.66

コーディングスタイルを決めておかないと馬鹿が好き勝手に変なことやりだして、
説明したり、質問したりのコミュニケーションコストがかかるじゃん
そういう糞みたいな無駄な時間をすごしたくないじゃん
一人で開発してるなら好き勝手やればいい

**デフォルトの名無しさん** · 2018/04/18(水) 00:49:52.00

>>21
馬鹿と開発しなきゃいいのに

**デフォルトの名無しさん** · 2018/04/18(水) 00:50:18.83

なんで馬鹿と開発するんだろ

**デフォルトの名無しさん** · 2018/04/18(水) 01:16:59.78

働いたことないのかな

**デフォルトの名無しさん** · 2018/04/18(水) 03:30:06.22

一行のif分でも{}をつけろとか、こういうのはたしかにいらんよな

**デフォルトの名無しさん** · 2018/04/18(水) 07:47:15.76

同じスタイルになっていれば、似たような処理の中の違いを拾いやすい。
関数分割・統合など既存コードを切り貼りする際にスタイルをどうするか考える必要が無い。

**デフォルトの名無しさん** · 2018/04/18(水) 12:48:57.42

>>25
なぜ？

**デフォルトの名無しさん** · 2018/04/18(水) 18:19:59.90

>>21
これ。
1人じゃなくても、お互いに信用できる程度の小規模チームならぶっちゃけいらない。
ただいつスポットで愚か者が手を出すことになるか分からんから、ある程度は決めておいた方が後々楽になるっていう経験則を多くのチームが持っているから規約を定めるんだと思う。

**デフォルトの名無しさん** · 2018/04/18(水) 19:32:04.33

>>24
馬鹿とは働いたことない

**デフォルトの名無しさん** · 2018/04/18(水) 19:33:45.48

>>25
Appleがそれで盛大にミスってたんだよなあ
コーディングスタイルとかそういうの関係なくif文に{はつけるべき

**デフォルトの名無しさん** · 2018/04/18(水) 21:32:56.63

>>25
必要だと思うよ
絶対付ける
他の人に迷惑かけないために

lintで強制してほしい

**デフォルトの名無しさん** · 2018/04/18(水) 22:28:21.42

一行のifなら

if(・・・) return 0;

みたいにすればOK

**デフォルトの名無しさん** · 2018/04/18(水) 22:55:19.15

>>32
そういうの見ると死ねって思う

**デフォルトの名無しさん** · 2018/04/18(水) 22:58:50.31

あー、ごめんごめん

if (・・・) return 0;

ifの後にスペースが必要だったねｗ

**デフォルトの名無しさん** · 2018/04/18(水) 23:14:02.68

これが馬鹿か

**デフォルトの名無しさん** · 2018/04/18(水) 23:14:07.92

>>33
なんで？（殺意）

**デフォルトの名無しさん** · 2018/04/18(水) 23:27:00.47

>>36
バグの温床だから

**デフォルトの名無しさん** · 2018/04/19(木) 00:11:07.89

#define ERROR_CHECK(cond) if (cond) return 0

ERROR_CHECK(ret);

こうすべきだよね！

**デフォルトの名無しさん** · 2018/04/19(木) 00:45:59.80

コーディングスタイルとは関係ない話ばっかり

**デフォルトの名無しさん** · 2018/04/19(木) 07:25:36.99

俺はコーディングする時は左足を上にして足を組むスタイルだけど、昔の同僚には椅子の上にあぐらかく奴もいたわ
ここらへんのスタイルも統一した方が良いのか

**デフォルトの名無しさん** · 2018/04/19(木) 12:06:43.81

>>37
なんで？（殺意）

**デフォルトの名無しさん** · 2018/04/19(木) 12:16:44.81

椅子の上に正座するスタイルは？

**デフォルトの名無しさん** · 2018/04/19(木) 16:18:59.20

ワイシャツにネクタイが基本スタイル

**デフォルトの名無しさん** · 2018/04/19(木) 17:47:07.24

とりあえず靴脱ぐのはいいがその足をこっちに向けるな

**デフォルトの名無しさん** · 2018/04/19(木) 19:19:03.44

>>41
アップルが実際にやらかしたバグ

　　if ((err = ReadyHash(&SSLHashSHA1, &hashCtx)) != 0)
　　　　goto fail;
　　if ((err = SSLHashSHA1.update(&hashCtx, &clientRandom)) != 0)
　　　　goto fail;
　　if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
　　　　goto fail;
　　if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0)
　　　　goto fail;
　　　　goto fail;
　　if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0)
　　　　goto fail;

**デフォルトの名無しさん** · 2018/04/19(木) 20:06:51.62

>>45
いまのツールならオートインデントとか到達しないコードとかですぐ捕まりそうだけど
結構昔なのかな？