Previous Page | Next Page

炎上プロジェクトとクソコード

――「クソコードを書け」と私が思うようになったもう一つの理由――


「クソコードを書け」という主張はいささか偏っていますが、思えば私は他人のコードを「くそ」と思ったこともなければ、「直せ」と思ったこともほとんどありません。
では、なぜそう思うようになったのか。


キャリアのごく初期――35年程前になります――には、「コードの美学」というものが確かにありました。しかしそれは1、2年で消えました。
さらに数年後、決定的な出来事が起き、その美学は吹っ飛びました。




30年近く前、1996年前後のこと。私が担当していたプロジェクトが炎上しました。
どのくらい炎上したかというと、残業が170時間、それが3か月ほど続いたのです。
残業代で数十万円入ったものの、翌年の社会保障費が爆上がりし、給与の支給額が「14万円」とかになった。
総務に「なんでやねん!」と詰め寄ったら、「あなた去年170時間残業したから…」と言われたのを今でも覚えています。
ちなみに、そのときの数十万円がどこに消えたのかは、いまだに謎です。


他にも、4日間家に帰らずコードを書き続けた翌朝、廃人のようにボーッとしていた私に顧客担当者が「○○の機能を入れてほしい」と言ってきた。
相手の言うことが理解できず、とりあえず「無理です」と返して、担当者がすんなり帰っていったこともあります。
int型で定義した変数を、なぜかshort型でexternして、朝の4時にバグを仕込んだり、
風邪で寝ていたら上司に呼び出され、現場で顧客から「ボーっとするな!」と説教を受けたり。
炎上プロジェクトの「あるある」は、わりと経験しました。




その後、私は数年間体調を崩しました。
体調を直すのに苦労しましたが、それ以上に、「何が悪かったのか?」をずっと考え続けました。


そこで学んだことは、


  • 顧客からの無理な要求は受け入れないこと(プロジェクトマネジメントの重要性)
  • 一日の労働時間には限界があること(体調管理の重要性)
  • 信頼できるメンバーを見極めること
  • テストコードの必要性
  • そして何よりプログラミングテクニックだけでは炎上プロジェクトは救えない

ということです。




もちろん、開発プロジェクトに必要なプログラミング技術は存在します。
しかし、一度炎上した現場では、もっと広い範囲で問題を見抜く力が必要になります。
たとえば、最近のマイナ保険証の問題。あれは「プログラミングが悪かった」という話ではありません。
私たちが学ぶべきことは、コードの書き方だけではなく、プロジェクト全体を見渡す力です。
ただし、だからといってプログラミングの勉強をやめてよいという話でもありません。




幸い、その反省が生かされたのか、私が担当して炎上したプロジェクトは一度きりでした。
その後は、いくつかの炎上現場に「助っ人」として呼ばれる側になり、記憶にある限り3回ほど大炎上の火消しに入りました。




では、火消し屋にとって最も重要なプログラミングスキルとは何でしょうか?
あまり知られていないかもしれませんが、コードを読む力(リーディング能力)です。




コードリーディング能力を高めるには、まず自分がCPUになったつもりでコードを読むこと。
つまり、「このコードはどのように実行されるのか?」という観点で、主観や美学を捨てて追うのです。
次に問うべきは、「このコードの動作は正しいか?」ということ。
ここで確認するのは仕様との整合性であって、コードが綺麗かどうかではありません。
論理的な間違いを追い、問題箇所を見極めることが目的です。




バグを探しているときにコードを見て「クソコードだ!」と思った瞬間、冷静な論理的判断はほとんど不可能になります。
私の経験では、感情的な印象を保ちながら正確に問題箇所を指摘できる人はほぼいません。
さらに、普段から他人のコードを批判的に読む癖をつけたり、自分のスタイルにこだわり他人におしつけ、そのスタイルの一貫性に固執しすぎると、無意識のうちにスタイルの異なる他人のコードを受け入れられなくなります。
つまり、コードの「あるべき姿」を追い求めすぎたがために、多様な書き方を認められなくなってしまうのです。




確かに、コードの出来が悪くてバグが潜むこともあります。
しかし、それを炎上中に嘆いたところで、現場は何も救われません。




コードの美しさよりも、まず動作の理解。
「クソコード」と切り捨てる前に、CPUのように冷静に読み、仕様との整合性を確認すること。
それこそが、火消し屋としての第一歩であり、炎上を防ぐための最良の訓練でもあるのです。




この文章は、原稿を元にChatGPTが校正・編集しています

2025-10-13 | コメント:0件

クソコードを書け

 ちょっと気づくのが遅い面がありますが、ネット界隈ではクソコードというワードが広がっているようです。要は品質の悪いコードを指して”くそコード”と称しているというこです。
ひと昔前に流行った「オブジェクト指向」と同様に今度は「クソコード」がある種のバズワードになっている感があります。
オブジェクト指向信者との闘いはここを見ていただければと思いますが、おかげ様でバズワードとしてのオブジェクト指向について冷静な議論をすることに一定の抑止効果があったと自負(?)しています。
一方で、今度はクソコードというバズワードが流行っていると認識しています。


実は、私自身ですが、他人のコードを見て「くそ」と思ったことはほとんどありません。「あーそう書くのか」と思うことがあります。そこに自分の常識にとらわれない新しい発見があるからです。


ちょっと古い25年前の例になるのですが、新人が書いたVBScriptの


name = "与太"
output = "ようこそnameさん"


というコードで、新人からの質問が「なぜoutputが "ようこそ与太さん" ではなく "ようこそnameさん"と表示されるのですか?」と質問を受けて逆に感銘を受けました。
ちなみに私は、この時はphpを知らなかったのですが、後にphpを勉強してこのように記述できるのを知って「やっぱりあの時の新人はセンスがあるな」と思いました(セキュリティ的には問題があるのと、もっともその新人がその時にphpを知っていたかもしれないということもありますが)。
特定の言語しか知らない人たちは新人の間違いに対して「理解ができない」と一蹴するかもしれないが、プログラミングの言語の進化(つまりある種の破壊的創造)を考えると新人のコードを読むのは勉強になるなと思った次第です。


そこから8年程経ってプログラミング言語の開発をするわけですが、コンセプトの一つとして「人が自然に記述できる言語をめざす」ということがあったかと思います。
要はエンジニアが書きがちなコードは一見クソに見えるかもしれないが、そこには「言語として表現されたもの」としてなにか意味があるのではないか?と思う次第です。


例えば、クソコードの一つに挙げられる「コピぺ」のコードですが、「なぜプログラマはコピペをするのか?」という風にコピペを行うプログラマの考え方やコピペが必要とされる場面や良さを考えるようになります。
オープンソースとかであるフォークもある種のコピペだと思います。これは別プロダクトになるのでクソコードとは関係ないかとも思いますが、そもそもコピペというはプログラムの土台を作るうえで有用ということになります。
誤解のないように言いますと、ある種の品質が問われる場面ではコピペは避けています。例を挙げると、私の場合、料金計算を行うコードは「一か所」と決めています。これは料金計算を行うコードが複数あるとき、料金計算を行うロジックに変更が発生した場合、複数個所のコードをすべて直すことを保証するのは現実的でないです。むしろ関数なりクラスなりにして料金計算は1か所で行う方が最終的に品質を確保できます。
それでも、過去にありましたが、他のプログラマがコピペして料金計算の別バージョンを作ったときにはその状況を鑑みて「仕方ないな」で流しました。


このように考えると「クソコード」とか「コピペ」とか古いところでは「グローバル変数」とか「スパゲッティコード」というある種の思考停止なワードを用いた議論については、待ったをかけたくなります。


別の観点でいいますと、プロジェクトの開発・保守の現場では、個々のコードについてはそこまで保守性が重要ではない場面があります。
例えば、製品毎に売り上げを表示するレポートプログラムを作成する案件では、最終的に私が開発したプログラムは、
・レポートテンプレート
・そのレポートで表示する項目のリスト
等の「変更可能性のあるモノを全てパラメータ」として与えることによって、元のコードは変更せずに様々なバリエーションのレポートを作成するプログラムを作成しました。
保守業務において、もはや元のコードは「そのままでよい」というこになり実際にコードを変更することはあまりありませんでした。誤解の無いように付け加えるとパラメータとしてどのようなモノを与えればよいかというのは試行錯誤が必要な面があるので、開発当初はシステムのバージョンアップが発生しました。


これはある種のプログラミング言語を開発し、その言語の上で開発を行っているという状況になります。この場合、私たちの目は元のシステムのコードに行くべきなのか新たに作った言語に行くべきなのか?ということが言えます。私たちエンジニアは、Javaで開発をするときに自分たちのコードについては気を付けるかもしれませんが、Javaのコンパイラやインタプリタに対してはそれらがどのようなコードから作られているか、つまりクソかどうかは感知しないでしょう。


さらに別の観点でいいますと、自動的に生成されたコードを保守しようとしたときに「本当にそのコードを保守するのか?再生成させたほうがよいのではないか?」という話もあります。
比喩的にいうと、Javaでコンパイルしたマシンコードの可読性をいちいち気にしないということと同じです。
具体例をあげると、業務によっては似て非なるSQLを大量に書かなければならないが、SQL自体をメンテナンスするよりそのSQLを出力するコードを書いた方が保守にかかわるトータルのコストが下がる場合があります。この場合、当然ですが、出力後のSQLを確認する意味では読みますがその可読性は問題視しません。
ちなみに私はコンパイルしたC++のアセンブラコードを読むこともあります。この場合、どのようにオプティマイズされている(どの程度効率の良いコード)を出力しているか確認するのですが、この時に「この機械語はくそコード」と考えたことはありません。もっとも、すでに20年以上前からコンパイラは腹が立つぐらいに(私的には)美しいコードを吐いてきます。


この2つの例ですが、要するに「ソフトウェアの品質を上げるには?」ということに関して「くそコードを書かないようにする」以外の回答があるということになります。


さて、AI時代になると、もはや人間がコードを書くことは減るかと思いますが、それでも人間がコードを読む時間は増えるでしょう。
この時に、AIに向かって「くそコードを書くな!」とかおっしゃるのでしょうか?


最後にChatGPTで遊んでみました。


P.S.
「お前は可読性の高いコードについて関心がないのか?」との質問については「当然関心があるが、自身が読みずらい・保守しずらいコードを読んで、クソコードということはない。もちろん個別の問題点については指摘する」ということが言いたいことです。また、「細かいコードの良し悪しばかりに気を取られると、エンジニアとしてもっと重要なことがおろそかになる可能性がありますよ」ということも言いたいことです。

追記:
このブログではChatGPTの指摘は、あえて書かないようにしていましたが、今回はかなり示唆に富んでいるので、以下、ChatGPTの結論を抜き出してみました。正直、そこまでは明確に意識していませんでしたが言われたらそうだと気付きました。


さらにChatGPTで遊んでみました。


ChatGPT1の 結語:「クソコードを書け」


クソコードを恐れるな。
クソコードを書け。

そこにはあなたの思考、試行錯誤、そして創造の痕跡が残る。
真にクソなのは、挑戦をやめて他人の美学に従うことだ。


さらにさらにChatGPTで遊んでみました。


ChatGPT2の結論


この文章は「クソコード=悪」という単純な構図を脱し、
「コードという文化現象」や「言語進化」「AI時代の人間の役割」まで踏み込む、
非常に知的で示唆に富む論考です。


ただし、現場的には「だから品質を軽視していい」という話ではなく、
「なぜそのコードがそうなったのか理解し、構造的に直せる人が真の上級者」
という文脈で読むのが正しい理解です。

2025-10-11 | コメント:0件

我がマシン達のWindows 11 25H2のインストール状況(2025/10/8)

Windows 11 の25H2がリリースとなり、我がマシンたちにインストールし始めました。その状況

2025/10/8現在
25H2にアップデート
Ryzen 9 5950X:25H2をクリーンインストール
Core i7-6850K:23H2 → 25H2
24H2のまま
Ryzen 9 3950X
Core i9-10980XE
Core i7-7820X
Core i7-4960X
Core i7-990X
23H2のまま
Core i7-6950X
Core i7-5960X
になっています。


メインマシンである、Ryzen 9 5950Xですが25H2にアップデートすると、RamPhantomEXが動かなくなりました。もともとこのマシンですが、22H2を3年前にインストールして23H2、24H2と2回アップデートを行い今回3回目ですが、ここでどうやらOSが壊れたようです。腹をくくって25H2をクリーンインストールすると無事にRamPhantomEXが動作しました。


2025-10-09 | コメント:0件

35歳からのSESについて考える

 以前にYoutubeでアップした動画(デジタル人材育成のための「実践の場」開拓モデル事業に与太話的に物申す)にコメントをもらいました。



 コメントされた方は、『この事業に参加してSES企業に入り2か月の研修で経歴詐称して客先に派遣されそうになった』ということで、大変憤慨されておられるようです。実は日本の中小のIT会社の多くはSES企業(つまり人売り企業)になります。見分け方の一つになりますが、入社したい会社の会社概要のページに「労働者派遣事業 許可番号 」というものがあればSESもやっている企業になります。もっとも全ての会社がきちんと労働者派遣事業許可番号をとっているか怪しいところもあるのでこの番号がないからといってSESをやっていないとは限らないので注意が必要です。


 SESとは「システムエンジニアリングサービス」の略で、要は派遣なのですが、派遣と言えば場合によっては違法になるので、客先常駐といったりSESといったりしています。私の知る限り少なくとも35年以上前からあり、30年前にはSESという言葉ができていたかと記憶しています。15年程前にはデジタル土方と言われるようになったかと記憶しています。経歴詐称も昔からあり、業務経歴書を見ながら面談をして『これは嘘だな』と思ったこともありました。ちなみに、建前上は面談(面接)はご法度ですが、実際にはコメント主のように『未経験者が偽って入ってくる』ということもあるので面談してある程度(実際の実力)を見てフィルターをかけないとお金をドブに捨てることになります。


このSESですが、悪しき習慣といわれていますが、一向になくならないだけでなく、最近では裁判沙汰になったりもしています。求職者の方々はこういうヤバイ会社には引っかからないようにしたいところですが、SESにもメリットというのは存在します。


SESのメリット


  • 顧客:雇用の調整弁になっている。正規や非正規でエンジニアを雇うよりも簡単に首が切れる。
  • SES企業:請負契約でシステムを構築するよりリスクがない。収支が読める。
  • 労働者:未経験者でも就業ができる(経験ができる)。短期かつ残業代がでるのなら正規雇用より儲かる。嫌なら後腐れなく辞めれる。向いている人がいる。

 顧客企業にとっては、非正規社員以上に合法的に労働者の首を切れることになります。あまり具体的なことは言えませんが、長い年月を経て多くの人が職場を去っていった後に、私自身も去ったことがあります(もっともこれは自らになりますので首を切られたというのはちょっと違います)。


本来ソフトウェア開発というのは請負契約で行うものなのですが、これは受注企業にとってはリスクがあります。特に顧客企業にソフトウェア開発の知見が無い場合は案件が赤字で終了する可能性があります。これを避ける為に準委任契約(SES契約)を行い、要は定額料金ではなく労働者が稼働したらその分課金するということを行います。
自ら作ったサービスを売るということもあります。こちらは王道と言えますが、当然にサービスが売れないというリスクがあり、赤字になれば、SES契約でエンジニアを売り、日銭を稼ぐという手段に出ます。
このようにSES契約が全て悪ということもないのですが、一度SES契約の味をしめると企業自体が努力をしなくなります。つまり請負契約で失敗しないように経験を積むとか顧客が求めているサービスをひねり出すという努力をしなくなります。


労働者にとってのメリットは『SESは未経験者の登竜門』ということも言えます。『経歴詐称はどうなるんだ?』と思われるかと思いますが、多くの場合、雇う側も経歴詐称であることを気が付いています。また、経歴書に詐称がなくても実際にプロジェクトに貢献していたかどうかというのもあり、実務的な観点からみると経歴詐称が一概に悪いとも断言できないところもあります。『じゃなぜ経歴詐称をするんだ』と思われるかと思いますが、これは受け入れ側の企業が書類選考をちゃんとしているという安心感を得る為にあります。もちろんですがプロジェクトによっては、経験者が求められていることがあったりするのでその場合に経歴詐称されると労働者があとで困ることになります。


SESのデメリット


もちろんSESのデメリットもあります。


  • 中間搾取、人月商売の横行が横行する
    こういう商習慣(人もモノとして扱う)が蔓延すると業界のレベルアップにならないです。個人のスキルに依存するのでチームとしてやプロジェクトの成長が期待できない。日本のITが伸びない理由の一つになっているかと思う。
  • エンジニアとして現場で使う以外の技術が身につかない
    余暇を利用して新しい技術を吸収する(自己学習ができる人)が求められます。
  • IT土方として雇われてるのでエンジニアの社会常識が育たない
    本人達は社会常識を持っていると思っているようで厄介ですが、ビジネスの話ができない人が多いです。実際にSESエンジニア上がりのある人に仕事の依頼の話をしたら、なぜかこちらが受注者として話が進んことがあり、こっちは発注者として仕事を頼んでいるのだが、なぜこのような勘違いをするのか相手の社会的な常識を疑ってしまった。ちなみに通訳案内士界隈も癖のある人が多いが、それでも友人と呼べる人はいるが、ITエンジニアに対しては基本的に人間的に好きになれなず、友人はほぼいないです。
  • 顧客からのフィードバックが「契約終了」でエンジニアとしての見通しがたてられない
    契約終了に関してエンジニア自身に具体的な原因がある場合、本来ならそのフィードバックがないとエンジニアが育たないが、そういう学習機会がそがれるので成長ができない。また、突然に契約終了となるとある種の失業状態になるので、エンジニアのライフスタイルが見通しにくい、40代ぐらいでSESが終了した場合割と困る。

労働者にとっての最大のデメリットは『SESは人売り』になるということで、これに耐えられない人が一定数います。このような方はSES企業には近づかない方がよいです。


ちなみに、私自身は総合的にはむしろSESで客先に常駐する方が気が楽な面があります。それでも会社として誰かを客先に出すというのはやりたくないです。


アドバイス


最後になりますが、未経験で35歳からIT業界に転職させる方に対してアドバイスするとなると以下の点を考慮されたうえでどうするか考えたほうがよいかと思います。


  • 未経験者がIT業界に入るとSESに捕まる確率が高い。実力がないうちはSESを避けて通るのは厳しい。SESに関しては向いている人と向いていない人がいるので、SESが嫌なら日本のソフトウェア開発会社は避けたほうがよい。
  • 本来、中途採用となると即戦力が求められ、新卒採用とは異なることを理解する。先輩とか上司に頼ることはできないと考えたほうがよい。
  • そもそも、SES企業の先輩とか上司自体がエンジニアとしてもいわゆるメンターとしてもきちんとしているかどうか怪しい。
  • 職場環境やその会社の業界の位置づけ、今やっている仕事等を考慮すると、キャリアアップするには転職が必要となる場合がある。実力が付いたらそれに相応しい会社に転職することも視野に入れる。
  • その職場に居続けるという選択肢もあるが、SES企業の場合、終身雇用との相性が悪い(辞めていく人間が多い)。その会社の規模や将来性、社歴と年齢構成(例えば創業から40年のSES企業で、50代の社員が少なく若い人しかいない会社というのは歳をとったら辞めていくと考えたほうがよい)、等を考慮する必要がある。例えば、今、新卒で入った会社で本当に将来性がないのか?、定年まで働けないかを今一度自問自動したほうがよい。
  • IT業界は『モノづくり』範疇に入るが、モノづくりの難しさ(完成させなければならない)を理解して、自分自身がモノづくりの適性(プログラムが意図どおりに動くと何とも言えない高揚感がある等)があるかどうか見極める。
  • 仕事に対しての困難さをどこかで楽しめるようでないと厳しい。
  • 自己学習を続ける必要がある。平均、一日に2,3時間は勉強時間を確保する必要がある。もちろん仕事が忙しいときは仕事に集中する必要がある。暇なときに勉強ができるかどうかがカギになる。
  • AIの台頭についてアンテナを張る。悲観的な見方をすると将来は開発の仕事はAIにとって代わられる。それが何時かということで他の職種も並行して勉強しなけれならない。

と厳しいことを書いていますが、実際には、きちんとできていないエンジニアが多いのも事実です。また、仕事が好きになれるのなら割と何とかなったりします。(私の場合、コンピュータが動いている様をみるのが好きで、面倒な顧客対応をしてイライラしてたのですが、そのあとコンピュータをいじっていたらやる気が出たりしてました。)


SES契約についてChatGPTで遊んでみました。


2025-10-07 | コメント:0件
Previous Page | Next Page