虹裏img歴史資料館

1 17/10/09(月)13:31:39 No.458285951

Oなのか0なのかわからん

2 17/10/09(月)13:34:59 No.458286624

入れ子しすぎて何やってるのかよくわからない

3 17/10/09(月)13:36:23 No.458286900

二重ループくらいはいいけど三重まで行ってたら 関数化するか処理見直せよと思う

4 17/10/09(月)13:36:27 No.458286916

こういう初期化きらい！

5 17/10/09(月)13:39:19 No.458287456

init();

6 17/10/09(月)13:40:32 No.458287690

>こういう初期化きらい！ VC++の言語拡張を無効にするオプションいいよね…

7 17/10/09(月)13:42:07 No.458287964

++i

8 17/10/09(月)13:48:15 No.458289053

if条件内で代入もしてるときに条件追加して括弧足すの忘れてバグるのはよくやる

9 17/10/09(月)13:49:58 No.458289416

if i = 0 then とかやっちゃうのまだあるのかな

10 17/10/09(月)13:51:45 No.458289766

>二重ループくらいはいいけど三重まで行ってたら >関数化するか処理見直せよと思う 動いている部分を弄っちゃ駄目という世界はあるのだ… 動いてる部分ってそんな仰々しいもんじゃないだろって部分も変えちゃダメって

11 17/10/09(月)13:54:50 No.458290376

画像のコードが本来どうしたかったのかすらわからん…

12 17/10/09(月)13:55:37 No.458290534

>動いている部分を弄っちゃ駄目という世界はあるのだ… いざリファクタリングしたらコード量が1/10000になったとかザラだよね…

13 17/10/09(月)13:57:33 No.458290926

>動いている部分を弄っちゃ駄目という世界はあるのだ… そういう企業文化だと思ってたところにいたけど 隣の部に移ったら保守性悪いから代えましょうというチームで たんに自分で責任取りたくないチームだったという

14 17/10/09(月)13:57:37 No.458290937

単純に関数にして置き換えるだけというのは簡単なのにそれでも危ない

15 17/10/09(月)13:57:43 No.458290955

ギンガ？

16 17/10/09(月)14:01:00 No.458291571

書き込みをした人によって削除されました

17 17/10/09(月)14:04:18 No.458292186

>たんに自分で責任取りたくないチームだったという 大体はそれ あと本当に能力低くてヘタにいじるとバグ出すからという場合もある

18 17/10/09(月)14:04:26 No.458292210

コーディングには専用フォントを使うに限る ゼロの中に点が打ってあるから見分けるのかんたん！とか

19 17/10/09(月)14:06:03 No.458292533

>あと本当に能力低くてヘタにいじるとバグ出すからという場合もある 全く手を入れない箇所ならいいんだ 機能変更や追加で手を入れるんだから整理させてください 元のスパゲティの方がルートテスト面倒だよ…

20 17/10/09(月)14:08:00 No.458292887

for i in range(7):

21 17/10/09(月)14:08:02 No.458292892

老舗の鰻屋かってぐらい歴史のあるソースを使っていると みんな既存部分を理解しているのか怪しくなってくるのだ… 「」君変数名にnewを使うのは予約語だからダメだよってソースコードレビューで怒られた後 すみませんそれ(指摘した)先輩が作った部分ですって反論するのいいよね…

22 17/10/09(月)14:09:48 No.458293227

>コーディングには専用フォントを使うに限る >ゼロの中に点が打ってあるから見分けるのかんたん！とか BMP外の文字に対応してるコード向きのフォントを教えて

23 17/10/09(月)14:11:17 No.458293482

友人から聞いた話だけどnameを間違えてnamaeって打ってたのが面白かった

24 17/10/09(月)14:12:54 No.458293745

読み込みサイズがread_lenだから 読み込み済みサイズはreaded_lenだな

25 17/10/09(月)14:13:08 No.458293783

英語わからんから変数名は変換サイト使う …Webフィルタリングされた

26 17/10/09(月)14:13:24 No.458293827

スレ画は >Oなのか0なのかわからん ってところが何やってんだろうミスなんだと思ったけど違うかな…

27 17/10/09(月)14:13:48 No.458293890

>BMP外の文字に対応してるコード向きのフォントを教えて コーディング用フォントは基本英数字記号の範囲までしかないよ だからエディタ側で複数のフォントをミックスして指定するよ 英数字はこのフォントでそれ以外はこのフォントでって

28 17/10/09(月)14:14:20 No.458293969

的外れなコードレビューされると対応に困る

29 17/10/09(月)14:14:46 No.458294039

i=0; i<x; j++;

30 17/10/09(月)14:15:16 No.458294120

Cとかの変数名は_が使えるからどこまでも読みづらくできる

31 17/10/09(月)14:15:31 No.458294166

>友人から聞いた話だけどnameを間違えてnamaeって打ってたのが面白かった それはコンパイラ先生がエラー吐いてハネてくれるじゃろ？ それを当てにした雑なコーディングしててよく事故が怒らなかったなって今なら思う

32 17/10/09(月)14:16:41 No.458294373

>i=0; >i<x; >j++; こういうことがあるとカウンタも真面目な名前付けた方がいいのかな…ってなる

33 17/10/09(月)14:16:47 No.458294388

最近のコンパイラもうお前が書けよってくらい頭良いよね…

34 17/10/09(月)14:16:47 No.458294389

>それはコンパイラ先生がエラー吐いてハネてくれるじゃろ？ 全部統一して間違ってればOK！

35 17/10/09(月)14:16:58 No.458294409

mojiretu1 mojiretu2 mojiretu3 mojiretu4…

36 17/10/09(月)14:17:28 No.458294492

iとかjってPGには常識かと思ってたけどそうでもないんだよな…

37 17/10/09(月)14:17:34 No.458294510

どんなときでもInconsolataだぞ

38 17/10/09(月)14:18:22 No.458294657

>こういうことがあるとカウンタも真面目な名前付けた方がいいのかな…ってなる 配列名の先頭一文字+ijklでイナフ リーダブルコードを信じろ

39 17/10/09(月)14:18:23 No.458294659

どんな理由があろうが一文字変数はやめよう

40 17/10/09(月)14:18:35 No.458294697

kまでは許す それ以上は流石に許さんが俺にそんな権限はない

41 17/10/09(月)14:19:03 No.458294765

関数化しすぎてあちこちに飛んで何やってるのかよく分からない事もある

42 17/10/09(月)14:19:04 No.458294766

>こういうことがあるとカウンタも真面目な名前付けた方がいいのかな…ってなる 長くてもcountにするとかネスト具合のわかる名前にするとかかな… 最近はそもそも生のforをあまり書かないようにしてる

43 17/10/09(月)14:19:15 No.458294804

意味のある変数名を付けろという現場でも iとjとcとpは許されると思う tはtime_tなんて使うなという話でダメ

44 17/10/09(月)14:20:01 No.458294939

条件によって再帰的に呼ぶ処理は条件しくじるとえらいことになるよね

45 17/10/09(月)14:20:09 No.458294962

数字を使ってる変数は基本的にクソしかない

46 17/10/09(月)14:20:14 No.458294974

i++ j++ k++ l++

47 17/10/09(月)14:20:15 No.458294982

>こういうことがあるとカウンタも真面目な名前付けた方がいいのかな…ってなる 複数のループを連続して使うとやらかしやすい ので複数ループさせる場合は関数なりで別スコープに追い出す 別スコープならjの初期化してねーよってコンパイラ先輩が怒ってくれる

48 17/10/09(月)14:20:18 No.458294986

>どんな理由があろうが一文字変数はやめよう tmp

49 17/10/09(月)14:20:22 No.458295003

foreachだと今何周目だっけ…というのをログに出すときにiを作るから ならforでいいじゃんってなることが多い

50 17/10/09(月)14:20:24 No.458295007

jは絶対に許さないよ

51 17/10/09(月)14:20:51 No.458295083

最近は++が使えない言語が結構あるらしいな

52 17/10/09(月)14:21:07 No.458295121

変数宣言に上司の承認が必要だった現場は地獄のようなコードがバンバン出てきたな

53 17/10/09(月)14:21:15 No.458295156

>tmp たまにチンポって読みそうになる

54 17/10/09(月)14:21:24 No.458295176

>>どんな理由があろうが一文字変数はやめよう >tmp tmp2

55 17/10/09(月)14:21:28 No.458295186

>foreachだと今何周目だっけ…というのをログに出すときにiを作るから >ならforでいいじゃんってなることが多い 言語によっては両方出せるんだけどねぇ…

56 17/10/09(月)14:21:45 No.458295232

>>>どんな理由があろうが一文字変数はやめよう >>tmp >tmp2 _tmp

57 17/10/09(月)14:22:51 No.458295438

100行近くあるループでiがループカウンタだったからとても検索出来なくて辛かった シンタックスハイライトがある環境なら全然違うんだろうけど

58 17/10/09(月)14:22:55 No.458295460

>条件によって再帰的に呼ぶ処理は条件しくじるとえらいことになるよね スレッドが沢山出来るような所だと関数再帰自体がダメよされて ループに書き換えさせられる

59 17/10/09(月)14:23:00 No.458295471

>変数宣言に上司の承認が必要だった現場は地獄のようなコードがバンバン出てきたな 変数名の解説を紙のドキュメントに残す必要があった職場にいました プロジェクトが火を吹いてから親会社に怒られてなくなりました

60 17/10/09(月)14:23:24 No.458295543

>どんな理由があろうが一文字変数はやめよう ii

61 17/10/09(月)14:23:26 No.458295554

>最近は++が使えない言語が結構あるらしいな swiftなんか以前は使えたのにアップデートで使えなくなった 気軽に破壊的変更やめてほしい…

62 17/10/09(月)14:23:33 No.458295573

>コーディング用フォントは基本英数字記号の範囲までしかないよ >だからエディタ側で複数のフォントをミックスして指定するよ >英数字はこのフォントでそれ以外はこのフォントでって VSCodeで文字コード範囲ごとに指定できたかなって見てみたら 範囲ごとの指定は無さそうだったけど複数フォントの指定はできるみたいだった ASCIIのみ定義されてるフォントを先にUnicodeのフォントを後に指定すれば似たようなことはできそう

63 17/10/09(月)14:23:50 No.458295628

なんで古いコードをコメントアウトして残すの… すぐ戻せるようにってぜってーもどさねえじゃん あとバージョン管理見ればいいじゃん…

64 17/10/09(月)14:24:40 No.458295783

>あとバージョン管理見ればいいじゃん… うｎうｎブランチいっぱい切ろうねぇ…

65 17/10/09(月)14:25:05 No.458295853

大体foreach系のループかmap-reduce系の集約でやるようになったからループカウンタ使う機会がめっきり減った

66 17/10/09(月)14:25:17 No.458295886

>_tmp Cの識別子名では先頭アンダースコアは予約扱いだから なるべく使うなってのはローカルループなんだろうか…

67 17/10/09(月)14:25:21 No.458295904

わたしforeach嫌い！ for好き！

68 17/10/09(月)14:26:09 No.458296031

>なんで古いコードをコメントアウトして残すの… >すぐ戻せるようにってぜってーもどさねえじゃん >あとバージョン管理見ればいいじゃん… バージョン管理あって残してるの？…なんで？って思ったけど よく考えたら俺の昔のとこもそうだった

69 17/10/09(月)14:27:00 No.458296161

>バージョン管理あって残してるの？…なんで？って思ったけど >よく考えたら俺の昔のとこもそうだった なんでかと言うとバージョン管理入れても使いこなせてる人がいないからだ

70 17/10/09(月)14:27:09 No.458296185

hoge foo bar

71 17/10/09(月)14:27:19 No.458296232

>なんで古いコードをコメントアウトして残すの… >すぐ戻せるようにってぜってーもどさねえじゃん >あとバージョン管理見ればいいじゃん… 今の案件がこんな感じでほんとに勘弁してほしい ひどいのは関数の中身全部コメントアウトされてて何も動いてなかった

72 17/10/09(月)14:27:39 No.458296285

>なんでかと言うとバージョン管理入れても使いこなせてる人がいないからだ 怖すぎる管理だ…