githubで最もやべー関数を発掘する - Qiita

はじめに 先日、職場で「自分が 改修したor 書いちゃった いちばんやべー関数」ネタで盛り上がりました。 みんないろいろ話してくれましたが、やっぱり僕の書いた「コマンドパターンのメインループ関数(1500行)」の圧勝でした。 なんであんなコード書いたんだろ。 そこで、今日は僕の傷ついたプライド癒すべくgithubから「世界でいちばんやべー関数」を発掘します。 つまり、「俺が書いた関数よりやべー関数に会いに行く」 結論 マジでやべー関数は次の2つ 「opencvリポジトリのcv::agast_cornerScore<AgastFeatureDetector::AGAST_7_12s>関数」(複雑度1868) 「SuiteCRMリポジトリのOpenTag関数」(複雑度1509) 言語毎の傾向に着目すると... javascriptにはやべー関数が多い python/java/swift/rub

[prograti]

OpenCVのベースとなっているコードは機械的に作られたコードのようです。おそらくそれを流用したのではないでしょうか。 http://www.i6.in.tum.de/Main/ResearchAgast　構文解析とか機械的に生成するケースが多いので。

[e_denker]

&&が2000個並んでるだけの関数もなかなか迫力があった。

[wwolf]

opencvのやつ、スクロールしてるとDNAに見えてきて神秘性を感じる

[megamouth]

見た瞬間に気絶するやつ

[kp1]

前者は自動生成だとしてももう少し構造化出来るのでは？という感想/速度のために可読性を犠牲にした実装にも見えないし。

[pongeponge]

agast_cornerScoreをチラッと見たけど変な笑い出たわ

[z1h4784]

本当にやべーやつで笑った。でも画像処理ってこういう力技になるのはしょうがないよなあhttps://github.com/opencv/opencv/blob/master/modules/features2d/src/agast_score.cpp#L3383

[eru01]

jsにやべーのが多いって書いてるくせに特出ししてるのはC++とPHPなんだが。それに画処理のコードって大体あんな感じになるやろ。4方位ないし8方位みるんだから。なんなんお前

[jhoshina]

コード開いてﾌﾌｯってなった

[wordi]

agast_cornerScoreは実装もヤバいがまだ氷山の一角(テンプレートの特殊化)に過ぎないのもヤバい

[Nyoho]

「循環的複雑度が高い関数」から探す記事

[igrep]

おもしろい

[ene0kcal]

すげぇ。でも"プログラム"って「編む」という意味でもあるから、こんなif文条件列挙を編むのは少しだけわかる。わかるが私ならジャンプテーブル（条件データとジャンプ先の表）データ作りますな。

[jsstudy]

IDEが強力すぎるととんでもなく巨大なクラスや関数を作れる。たまにはテキストエディターで初心に戻ろう。

[tmatsuu]

やばい。どうしてこうなった

[kkobayashi]

5位のコードチラッと見たら酔った

[nmcli]

素直にやべー

[cl-gaku]

phpのはちょっとスクロールした後スクロールバー見て笑った

[mas-higa]

備考「こいつはやばい」

[n314]

PHPのやつ、綺麗に整理したらそれはそれで遅くなる気がするな。3秒でPDF作れたところを綺麗に書いたら10秒かかるようになった、とかありそう。

[furu_ichi]

実際にソースを見たら本当にやばかった（語彙力）。ただ、これを見てもヤバさがわからなそうなプログラマは受託開発の現場だと普通に存在する。

[escape_artist]

mPDFか。もう少しエクステンションに逃がせないのかな。。

[kazoo_oo]

3300行のほとんどすべてが分岐とジャンプに使われている。

[ina-amagami]

OpenCVのやつもOpenTagのやつも、見た瞬間に考えることをやめるやつだ...。

[nakag0711]

switchは複雑度急上昇するんだよな

[spam_lover]

初期のテラリアのソースコードもこんな感じのifの鬼ネスト羅列だった

[topiyama]

該当のgithubページ、Android版Chromeで見たらいけないやつだった（固まった）

[tohokuaiki]

SuiteCRMてかmPDFか。PDFの吐き出しはヤバくて当然な気も。昔、TCPDF/FPDF見たけど「絶対に書きたく無い。作ってくれて感謝」としか思えなかった。俺もXML変換のプログラム書いた時「ヤベーの作ってる」と自覚してた。XSLTは神

[Shisama]

GitHub内の循環的複雑度が高い関数の抽出について。nodejs/node内の関数もTop10にあったので見たら全部v8のコードだった。nodeはv8のコードそのまま取り込んでいるだけなのに、v8のリポジトリが引っかからないのはなんでだろう

[chimerast]

opencvのは、コード生成でしょ。後者のは、手で書かれて、アドホックに機能が追加されていった雰囲気がある。。。こういう時のgit blame。

[hayashikunsan]

思ってたより凄くて面白い

[tx198]

大量のIF文吐きそう。。

[homarara]

俺これをメンテナンスしろと言われたら取りあえず転職活動始めるわ。

[sds-page]

これ思い出したhttp://tamakobkrunion.hatenablog.com/entry/2018/02/26/220717

[hajimepg]

ジェットコースターみたいなインデント

[gabill]

openCVのやつはifの入れ子を&で置き換えれば大分すっきりしそう。『if文が全部trueならA、それ以外はB』みたいな処理ばかりだよね？

[threethree]

下手とかそういう次元じゃなくて、「こうせざるを得なかった」感があるのがなお悲しいな。/誤植が恥ずかしいから修正

[gomer-pyle]

ヤバいの開いたらpixel3のはてブアプリがハングアップした

[UhoNiceGuy]

「if文を2000個並べているだけ」←！！

[mole-studio]

自動生成されたものがかなり混じってそう