CodeIQ MAGAZINECodeIQ MAGAZINE

【PHPer女子の本音炸裂】こんなコードを書くヒトはモテません!

2015.11.06 Category:インタビュー ,勉強会・イベント Tag: ,

  • 215
  • このエントリーをはてなブックマークに追加

「PHPer女子が語る2015!こんなコードを書くヒトはモテない〜コラボ編〜」のLTで話題をさらった深澤ひかりさん&PHPer女子の座談会。

こんなコードを書く人はモテない、こういうアドバイスをする人がモテる!PHPer女子の本音とアドバイスをレポート!
by 馬場美由紀 (CodeIQ中の人)

PHPer女子のLTで話題になった「非モテコード」

去年、あるLTで「モテないコード」の例を3つ紹介して話題となった深澤ひかりさん。ちなみに、どんなコードが紹介されたかというと…。

まず1つ目は「意味のわからない名前」。ListAとかalistとか、bCountって一体何を指しているの?という状態。自分にしかわからない名前で書くのはやめましょう!

2つ目はforeachの記述が永遠と続く「foreach地獄」……イケてませんね。

3つ目は「goto fail;」が{}でくくってないので、読み込まれてないif文。も、もう生理的に無理…だそうですorz

2015年バージョンのLTは、なんとCodeIQ問題とコラボしてもらうことに。まずは作戦会議ということで、PHPer女子3人に集まっていただきました。

<座談会参加メンバー>

深澤ひかりさん
株式会社スタジオ・アルカナ勤務。最近ではPHPよりJavaScriptをゴリゴリ書いている。PHP勉強会を運営している。去年のLT「PHPer女子が語る!こんなコードを書くヒトはモテない」で話題に。

わかなさん
PHPのAPI、CakePHPなど、サーバサイドからフロントエンドまで操るギークガール。最近、ベトナム人女性向けオンライン日本語学習サービスを提供する、株式会社CAMON.TOKYOを起業。

なっちゃん
DTPデザイナーだったが、WordPressで構築しているうちにPHPに興味を持つ。PHP勉強会に参加してうちに、ひかりさんにスカウトされて1カ月前に株式会社スタジオ・アルカナ入社し、後輩に。

無意味なコメントアウトはどれ?


ひかり:ということで、いくつかネタを持ってきたよ。最初は「無意味なコメントアウト」。自分自身もコメントアウトで失敗しやすいので出してみたんだけど、どうかな。コメントアウトって、たくさん書かないといけないと思ってたけど、これって必要ないよなっていうのあるよね?


わかな:あるある。変数ちゃんと書いておいてくれればいいのに、なかったり。メソッド名が変だったり。todoの横に何も書いてなかったり…。


なっちゃん:todoは残しておいてほしい!


わかな:コメントアウトがなくて、コードレビューする側が探さなきゃいけないこともあるよね。必要以上に手間かけさせないでほしい(笑)。


なっちゃん:昔、ごめんとか書いてあるコメントアウト見たことある。誰に謝ってるんだって笑っちゃった(笑)。


ひかり:不要だと思うコメントアウトはどれかっていうのを、女子でアンケートとって正解にするのも面白いかもね!じゃ、1問目はこれで決定!

設問1. このコメントダサイなぁ。マシなのはどれ?

1. // modified by 2015-09-14 y.yamamoto
1. // TODO: あとでテスト
1. // カンマ区切りでsplitする

──── PHPer女子が選んだ集計結果を見る

長過ぎる関数やif文条件式とか


ひかり:長過ぎる関数書く人っているよね。1画面におさまるぐらいがベストなんだけど、おさまらないくらいの。


わかな:いる!長過ぎるif文条件式書いちゃう人とか。上手に分割してほしい。


ひかり:「regist(*)」とか、間違った言葉を使っちゃってることもよくあるよね。文法的にも正しくない関数名とか、ググるといっぱい出てくる。


わかな:人が見ることを意識していないからだよね。マジックナンバー書く人とかもいるよね~。


ひかり:じゃあ、ヒドイ命名規則とかマジックナンバーとかを挙げて、どれなら許せるかって聞いてみようか。

設問2. 命名規則がダサいのが並ぶよ!どれがマシ!?

1. $p[‘ctg’] = $ctg; // 製品のカテゴリID
1. $strName = “paul”;
1. $ninzuu = 20;

──── PHPer女子が選んだ集計結果を見る

設問3. マジックナンバーが並ぶよ!酷いよ!どれなら許せる?

1. $expired = 86400;
1. $taxincluding = $price * 1.08;
1. error_reporting(0);

──── PHPer女子が選んだ集計結果を見る

PHP界のコード・イケメンはmpywさん!?


わかな:モテないコードのソースレビューが一番ツライよね。レビューすることのことを考えて書いてほしいって思う。


ひかり:ところで、イケメンなコード書く人ってどんな人かな。


わかな:それは、絶対mpywさんだと思う!Qiitaにいつもすごく詳しい解説記事を書いてるんだけど、記事にすごく愛があるの。まだ学生さんだと思うんだけど。

Qiitaだけじゃなくて、なんとヤフー知恵袋でも困っている人に対して優しくアドバイスしていて。もうPHP界の王子的存在っていうか。会ったことないけど、絶対イケメンのはず!


ひかり:うんうん、mpywさんはたしかにスゴイ!「エラーログ見ろ」とか突き放さないよね、絶対(笑)。


わかな:「INDEXの使い方もわからないの?」とかも、ムカつく!逆に困ってることに気付いてるのに、「ggrks」とかいって、自分で調べろとか、放置プレーする人もいるし。


なっちゃん:私は控えめにぐいぐい来てほしい。放置が一番ツライなあ。


ひかり:技術書をそっと差し出してくれたりね。


わかな:私はやたら上から目線でわかってることをグダグダ一から説明されるのが苦手。わからないところだけ教えてくれるmpywさんみたいな人がいい!


ひかり:じゃあ、これが最終問題ということで決定!結構意見が分かれるから、結果が楽しみかも!でも結局は、いかに女子のプライドを傷つけないようにサポートやアドバイスしてくれる男子がモテるってことかな。

設問4. 後輩の女子が困っている!どうする?

1. 「エラーログ見てごらん」とさりげなくアドバイス
1. 「この本を読んで勉強してみたら?」とそっと参考書を渡す
1. 自分で理解するのが一番!そっとしておく
1. 1から100まで懇切丁寧に長い時間をさいて教える

──── PHPer女子が選んだ集計結果を見る

【結果発表】モテモテエンジニアは18%!ブサイクエンジニアは…?

PHPer女子が選んだ解答の集計結果は以下のようになりました。予想外に女子の意見も割れる結果でした。コードへのこだわりは人それぞれということかもしれませんね。

あ、これはあくまでネタですよ!!リーダブルコードよんでくださいね!!

【正解】1. このコメントダサイなぁ。マシなのはどれ?

正解は、「// カンマ区切りでsplitする」でした。

  1. // modified by 2015-09-14 y.yamamoto 21%
  2. // TODO: あとでテスト 36%
  3. // カンマ区切りでsplitする 43%

【正解】2. 命名規則がダサいのが並ぶよ!どれがマシ!?

正解は、「$strName = “paul”;」でした。

  1. $p[‘ctg’] = $ctg; // 製品のカテゴリID
  2. $strName = “paul”;
  3. $ninzuu = 20;

【正解】3. マジックナンバーが並ぶよ!酷いよ!どれなら許せる?

正解は、「error_reporting(0);」でした。

  1. $expired = 86400;
  2. $taxincluding = $price * 1.08;
  3. error_reporting(0);

【正解】4. 後輩の女子が困っている!どうする?

正解は、「『エラーログ見てごらん』とさりげなくアドバイス」でした。

  1. 「エラーログ見てごらん」とさりげなくアドバイス
  2. 「この本を読んで勉強してみたら?」とそっと参考書を渡す
  3. 自分で理解するのが一番!そっとしておく
  4. 1から100まで懇切丁寧に長い時間をさいて教える

ちなみに女子は「そっとしておく」がゼロ!放置すると絶対モテないのでご注意を!

  • 215
  • このエントリーをはてなブックマークに追加

■関連記事

池澤あやか直撃!─社会人1年生が、エイベックスの音楽配信プラットフォームを作っちゃった!?... アマチュアアーティストやクリエイター向けの音楽配信プラットフォーム 池澤:まずは、スマートソーシャルさんの事業内容について教えてください。 小田倉:現在はWebシステムやスマホアプリの受託開発を中心とするソリューション事業、ヒューマンリソース事業、マーケティング事業──この三本柱で事業を展開...
営業・事務職もプログラミングを学びたい!「TECH::CAMP」イベントに96人の女性が参加... 女性がITの力で活躍する場を拡げるイベント「TECH LADY HOLYDAY」 「TECH LADY HOLYDAY」は、女性がITスキルの習得の魅力を知る機会、およびスキル習得アクションの機会を提供する女性のためのイベントだ。 ITの力で活躍する場を拡げることを目的に、「世の中にサービスを生...
業務ではコードを1行も書いたことがない元人事が、プログラマとして独り立ちするまで──... エンジニア採用が転機になって、自らエンジニアを目指す 「教育で世界をよくする」を企業理念に掲げ、2006年に創業したヒトメディア。教育分野や異文化領域に対するインキュベーション事業を行っており、大学入試の出願システムの開発や、投資先・提携先の新規事業の立ち上げ、人材提供、海外提携などのサポート...
Goodpatchのギークガール☆趣味はプログラミング!だから、仕事外でもアウトプットしていたい... 中学生の時にガラケーサイトを作ったことがきっかけ プログラミングとの出会いは中学生のとき。ガラケー(フィーチャーフォン)向けのサイトを作り、ブログにリンクするのが流行っていたんです。私も作ってみたいと思い、パソコンを持っていなかったため、ガラケーで作っていました。 HTMLのタグは、単語...
「ドローン」を愛する西脇資哲氏と学ぶ!Windows女子部×「目線で決まるプレゼン」... 西脇氏が所有するドローンは12機、自腹を切った金額は150万円! 勉強会では、まず最初に初代部長である椎野磨美さんから、Windows女子部誕生の経緯と活動内容、今回の開催概要について紹介があった。 同コミュニティは設立4年目で、Facebookを中心に活動を発信。毎回旬の技術や著名なエンジニア...
さあ、女の子たちもコード書こうよ!「Code Girls」が拓く、女性エンジニアの未来... 女子中高生限定。企業とコラボし、さまざまな刺激を提供 世の中のWebサービスやITシステムは、当然ながら女性も使う。女性向けに特化したサービスもある。なのに、それを開発しているのはたいていが男性。企画やデザイン畑への女性進出は珍しくないが、コードを書いているのは、相変わらず男性ばかりという印象があ...

今週のPickUPレポート

新着記事

週間ランキング

CodeIQとは

CodeIQ(コードアイキュー)とは、自分の実力を知りたいITエンジニア向けの、実務スキル評価サービスです。

CodeIQご利用にあたって
関連サイト
codeiq

リクルートグループサイトへ