見て絶望したソースコード９選 【SNSで4.5万人に聞いた】
HTML-код
- Опубликовано: 12 апр 2023
- ▼応募のあったあるある全編note（いいねしてね！）
note.com/ara_sun_/n/nb0d19155...
▽アラサンのキャリア
１社目独立系SIer→２社目事業会社の社内SE→３社目事業会社の社内SE
▽あるある登録フォームはこちら
ara-sun.com/neta/
▽Twitterはこちら
/ ara_sun_
▽自己紹介ネタの応募はこちら
ara-sun.com/prof-qa-boshu/
// TODO これを見たあなた 実装してください
というのを一度見た
もはやスタンド攻撃だろ
If 1==1 意味わかん無さすぎてほんと好き
ショート動画の時はエンジニアへの就職が怖くなる雰囲気の動画なのに
普通の動画だと優しく見える
これがブラック企業の飴と鞭か
さっきたまたまショート動画で回ってきたんですけど、話し方とか笑い方とか好きすぎる… 社内SE1年目なので勉強になります！
"MVVMで作ってますが、MVVMになってません"の件、自分が同じようなこと書いた説明書を作ったことがあるので驚きました。
私の場合は引き継いだソースコードがMVVMで作ろうしているがそれとはかけ離れたものになってました。引継ぎ資料を作るに当たり皆が騙されたこの部分を強調するために書いた記憶があります。
恐らく私と動画の件は無関係だと思いますがあまりにも類似していたので驚きました。
テストコードのケースは日本語で書くのめっちゃわかりやすいからおすすめです！
変数を表管理は斬新でおもろい
ガチで絶望するのはソースコードではなくDB設計が糞なパターン
めっっっちゃわかる
変なこだわり持った人間がDBとかディレクトリの設計やると作業効率が終わる
これ系は人によって意見分かれそうな予感
DDDとかは前提知識ない人からすると学習コスト高いのか、衝突をよく目にする笑
外部キー・主キー制約が存在しない基幹システムを見たときは、戦慄しましたわ
@@sawara_saikyoyaki キーなしって重複どうするんや…同一データ作れるとか事故やん
まじでこれ
// ここを消すと動かなくなる なぜかわからない
ってコメント箇所を発見して絶望したことあります
Gitで管理してるのに1行目〜10行目、12行目から30行目、32行目から〜みたい延々コメントアウトしてる上に、何千行のソース見た時は逆に天才が書いたのかと錯覚した
修正していいなら絶望しない、修正しないでくれって言われると絶望する。
明治時代だとステップ数という名目でコードの行数が評価基準になってたので、それをかさ増しするために意味の無い処理が無限に書かれていることがあると、最近の歴史の教科書には書いてあるらしいです。
素のアラサンめっちゃ優しさ溢れてる、徹夜の作業指示されてもしたがっちゃいそう///
自分がスパゲッティー作ってる自信があるから職場辞めるときどうしようかな～って今から悩んでる
if(1)なら確実に条件満たすからデバッグ用かな？と思うんですけど、1=1でしかも大量ネストってどういう意図なんですかね？
プログラミング初学者だから分からなくて
ショートでよく見掛けて、今回はじめて動画を観ました。
そもそもプログラミングなんも知らんはずなのに、ポジティブな雰囲気、ネガティブな雰囲気が不思議と伝わって観ちゃいます。
苦労は万人に伝わっちゃうものなんだなぁと。
おもろー
近くのエンジニア大事にしてあげて🥺笑
cssファイルがサーバー中に散り散りになっていて、外しても外しても違うスタイルが流れてくるとか、
何行目以降は全容を把握できる人がいないので絶対に触ってはいけない領域…みたいな指示があったり。
つ「何も無い箇所に改行入れたら何故かバグが治った時、かつ、その理由を神経質なお客さんに説明しなければ行けない時😭」
"if(1 = 1){ ... }"は"1 = 0"に変えるとスキップできるからテスト用に使ってる人いたけど……
絶対Bool変数使った方がいいと思ってた
何故かこの文を消すと動かないので消さないでください
//テスト用の仮置きなので後で消す
って部分が普通に本番で稼働してたことはあった
それを何事もなくスルー出来る人と出来ない人が意見ぶつけ合うのもあるある。
「役割の分からない似た名前の変数」→「すべてのクラス名、含まれる変数名などが紙でファイリングされているプロジェクト」（工業系、行政系システムにまれによくある開発方式）の名残
「if(1=1) が大量に」「コメントアウトソースが大量に」→昔あった開発規模指標に「ソースコード行数」というものがあった（今もあるかは知らない）名残。意味がなかろうが何だろうが行数は正義
「コメントが国際的」→できもしないのに安いからって言ってオフショアに手を出して統制できなかった失敗の名残
基本的に「プログラムわからない人間がプログラムを管理しようとして開発効率out of 眼中カマしちゃった」ってこともあるので、あまり「それを書いたやつ」を責めすぎないであげて欲しいｗ
パッケージ、クラス名がローマ字で、フォルダ・ファイル命名の上限を超える😂
調べてみたら結構な言語で日本語の変数名が使えるみたいですね。
考えてみれば表音文字より表意文字を使ったほうが可読性は上がるのでは？と思います。
ｳｯｯ!!他人の振り見て我が振り直せ…肝に銘じます！
ちょうど今日15行×3重ネストの三項演算子見かけて絶望してたとこ😑
判定処理の中身もメソッド戻り値がぎっしり
単体テストのメソッド名を日本語で書くのはある 0:52
kbn(区分)とかbtn(ボタン)とか
変数名がnum1とかnum2はローカル変数だと何とか追えるから未だ良いんだけど、1つの超大作ファイル内にあるサブルーチン名を全部Sub01～SubXXの通し名ってときはギリギリした記憶がある
あとはコメントを書きすぎてるコードは、歴史を重ねると実コードと整合性が取れてない箇所が一杯出てきて、結局実コードを先に見てコメントを修正したりする余計な作業が増える
プログラムの実コードに日本語はプログラム言語仕様としてOKの言語が結構有るから、その場合は下手な英語名をつけるより分かりやすいコードになる
全部今職場にある
キレながらコード規約改修要望出しまくってる
テストコードでカバレッジ満たすためだけに1=1みたときはある
シェルだけど、コメントないかつ設計書も古いのしかなくて苦労した記憶…
おだやかな声で地獄の話してる
最近はツールが検知してくれるようになったけど、TODO ね。
そのTODOは異次元にあって一生来ない。
コメントミルフィーユもあるあるだわw
「ここの処理が正しいか後で確認する」とコメントがある、リリース済みのプログラムを見たことがあります。
絶望はしなかったけど、目が点になりました。
個人的に絶望したのはコメントの内容と実際の処理の内容が全く違うコード
あと関数名と処理が違うコードも見たことある
VBAはわかりやすくなるようにあえて日本語で変数名つけてる
私もです
if(1=1)って何の意味があるんです？？
ググったんですがよくわからんくて…
// 使用禁止
って書いてあるメソッドが使われてたことある(一応障害とかは出てなかった)
エンジニアじゃないから何のことかわからんけどおもろい
わけわからんのだからコメント入れろ！ってのはある
削除予定ガチ笑ったwww
今日見たもんwww
サブシステムごとに担当してる会社が違うから
全部書き方違ってて発狂してます。
次の職場の規約が心配になったぜ！
3:03 ベトナム語で書かれなくてよかった…
組み込み開発でCしか使わないものなのですが、コンパイラとかにもよるかもですがif(1＝1)って通るんでしたっけ、if(1＝＝1)は通る記憶がありますが…
Javaしか使わない者ですがエラーになる気がします
C#しか使わないものですが、エラーになると思います。
やっぱそんなかんじですよね。if(a＝const)はaが代入できる変数なら代入後常時trueで通りそう
If 文の1=1は、確か意味が有ったはず。
ドライバー周りのエンジニアが使って居た記憶が有る。
ほえ〜
区分を「_kbn」ってしてるのうちの職場でめちゃくちゃあって笑ったw
コードレビューでメソッドを全然分けてなくて、メインのメソッドの中身がとんでもなく長いみたいなソース見たときはシンプルに絶望したな。
そのあたり作り直させるにも時間がないし。
さらにfor文の最後でcontinue打ってたりbreakの場面でcontinue打ってたりしてて意味分かってるのか疑問。
書いた人、中途で経験者として来たのに…。
中途経験者「for文作れました！」(絶対に最後まで繰り返す意地の文末continue付き)
関わりたくないけど聞く分はにはおもろい
breakの場面でcontinueは草。
念の為、確認のため最後まで回しちゃおう★！的な感じなのかなww
学校とかだと、その学級のことをclassではなく、klassと書いているところがあった。
パラメータとかプロパティの名前ですら、Tr(取引) とかある
使われてない処理が残ってるのは自分も経験ある
あと関数名が数字の羅列なのもやめて欲しい、マジで何やってるか分からないから中身全部見るしかない
2:32 1=1ではなく1とかtrueで良い。
if消したいけどインデント上げたくなければ使うね。インデント上げてでも消したいという人は消すでしょう。
1:07 変数がそうである以前にテーブルのカラム名がそうであると絶望する
0:05 自分ならスルーだね。書く意味が無いなら書かないけど、消す意味が無いなら消さない。
バージョン管理されてるのに、コードレビューでコメントに変更履歴書けって指摘貰って、頑張って修正したことあるなぁ。。。。
これ意味あるんですか？って聞いたら「決まりだから」って返ってきて、えぇ。。。。ってなった思い出
尚、顧客に意味ある？って言われて速攻で廃止するという顧客絶対主義
むしろそんないい顧客がいるのが羨ましいです
まあ意味ないから...って理由で勝手に辞めるのはかなりリスキーだからやめといた方がいいよねー
1=1がピント来なかった…
kbnあったなぁ！
日本語変数名はもっと積極的に使われていいと思うけどなあ
パッと見でわかる変数名はありがたいですよね…。たまにここだけ日本…てなります。
税金等の法的なワードが大量に英語化されていると訳わからんとかあるからね…
こういうコードは使うかもしれんからコメントアウトしといてーはありましたw
スクラッチ開発する金がないから、残念な感じになっていくよなー
While(1)があった時
全部ワシで草
コメントで改修履歴管理は普通にありますよ。
特に2000年代初め頃までは。今でもバージョン管理しないでソース管理してるところもありますし。
リモートリポジトリ見たら、ブランチごとに全く違う内容(ほとんど同じファイルがないレベル)が入ってて、各作業工程で使うツールがそれぞれのブランチに入ってるとのこと(???????)
全ブランチから使えそうなコードをサルベージして一つにまとめる作業はまじでだるかった記憶。
そういうのってリードエンジニアが共有ライブラリやクラス設計のルールを管理してチームに周知を徹底すべきよね
・エラー文出力するだけでネスト8個
・1つのテンプレファイルで全画面作ってた。その人辞めて負の遺産
・変数名aaa、bbbとかあって名前変えたらまさかのグローバル変数でエラー大量。元に戻した。
辛いねーが人事で飲んでた白ワイン吹いた。
アンリーダブルコード
有識者に丸投げして、ローカルで動かしたらまともに動かなくて、SELECT * FROM A, B, C, D
日本語変数名は別にそれで管理できるなら問題はない気がする
numでbooleanはやめて欲しい笑
でも、こういうクソコード系の話って、なんでダメでどうしたら解決するかまでわかってるエンジニアはごく僅かという、、、
みんな自分を棚に上げてるのさ
変数名がそうである以前にカラム名がそうだとクソカラム、テーブル名がそうだとクソテーブル。
visual studioだと日本語変数名使えるから結構助かるんだけどなぁー
_kbnって区分って事だったのかｗ長年疑問だったわｗ
num1,num2,num3
よりも
num,num2,num3
のほうがつらい
衝撃的だったのはdefaultしかケースのないSwitch文だなぁ
switch ( value )
{
default:
if ( 条件式 )
{
処理;
}
break;
}
void (*signal(int sig, void (*func)(int)))(int);
こういうやつ
バージョン管理?
全部コピペで保存だろ!!!
そろそろソースをAIに食わせれば説明か、書き換えてくれる時代がきたんで、クソコードとおさらばですね。
ごめんなさい削除予定はやってました……
絶望とまではいかないけど、そこそこ開発経験あるって聞いてた人がif([boolean型変数]==true)みたいに書いてた時は「マジか…」ってなったな…
700行の動的クエリ
結合テーブルが30以上www
ます、テーブルのカラムの意味から確認しないとダメwwwプログラミングスキルでは無理で、コメントの業務的な理解か必要w
何というか、違う能力が必要w
和名変数でパッと思いつくのはC#かなぁ
私はVBAを真っ先に思い出しました😅
バージョン管理使ってても、改修履歴をファイルヘッダーに書きませんか？？？
僕が絶望したのは、フラグで最初は０が犬、１が猫の意味で使っていたのに、途中で１が犬、２が猫みたいに意味合いが変わってた奴。。。
逆に多少経験積んだ時に多いのが過剰にコードスタイルを気にしすぎて、無駄なくらい他人のコードに不満を言うやつ。
コーディング規約が最初から厳密に決まってるなら規約違反を指摘するのは良いけど、完全に個人の好みの問題でグダグダいうのを聞くと、
まあもっと経験積めばその程度のスタイルのブレは許容してかないとキリがないと悟るだろうと生暖かい目で見てる。
10秒前！？
変数命名が意味不明だったり、コメントが無価値だったりするのはまだいい
一番やべーのは障害が発生したのに何のログ解析等もできないやつ
そういう奴は大抵テストも甘いと思ってかかる
個人的に絶望するのは判定式に処理関数、判定式の結果がboolじゃない、次どこにjumpするか分からんソース