コードレビュー

こんにちは。AS 部の兼安です。 お盆で実家に帰ったら、学生の頃の C 言語の教科書が出てきました。C 言語のポインタとアドレスの概念は、結局全部アドレス持っているのだから関数だって変数に代入できるよね！などという形で一応今でも役に立っています。 ちなみに、当時私が書いたコードも見つけました。コードレビューしたら確実に弾かれそうです。というわけで（？）今回はコードレビューのお話です！ この記事の内容について The Standard of Code Review(コードレビューの基準) What to look for in a code review(コードレビューの観点) Speed of…

スマレジの テックファーム（SES 部門） でWebエンジニアとして働いている やまて（@r_yamate） と申します。 実務では、2023 年 3 月末で SES の派遣先で、テーブルオーダーシステムの機能改修業務の設計などを担当していた業務を終えたところです。 4月からは、スマレジの関連アプリの開発業務を担当しています。触ったことのなかった Flutter での開発で、日々奮闘中です。 はじめに 一週間ほど前からレビュアーとして、コードレビューデビューしました。今回の記事では、コードレビューはどうやったら上手くできるか、という課題とじっくり向き合うことにします。 現状の課題 コードレビュ…

大きなPull Requestのレビューがつらい 修正ファイル数が多いこと自体が問題なのではない 1つの内容に集中する 小さなPull Requestの作り方 リファクタリングの修正は気になっても別で出す Web API 1つに着目して実装を切り分ける 小さなPull Requestで作ったときのリリースの仕方 featureブランチを作って、そこから更にブランチを作っていく フィーチャートグルを使う 小さいPull Requestで小さくフィードバックをもらおう 大きなPull Requestのレビューがつらい 転職ドラフトでWebアプリケーションエンジニアをしている @iwtn です。 こ…

こんにちは、2022年4月にフォースタートアップスにジョインしたエンジニアの八巻（@hachimaki37）です。主にタレントエージェンシー支援システム（SFA/CRM）のシステム開発を担当しております。 コードレビュー自動化サービス Siderがサービス終了となった背景から、移行先となるサービスを調査し導入をしました。今回のテックブログでは、調査過程で出てきた疑問、そして調査結果、導入方法などを合わせて執筆していきたいと思います。 背景 タレントエージェンシー支援システム（SFA/CRM）では、コードレビュー自動化サービスとしてSiderを利用し​​ておりました。そんなSiderですが、si…

今回は前回の記事の続きになります。 ryubloblog.hatenablog.com この記事では下記の項目についてまとめていきたいと思います。 ネストが深すぎないこと 不要なコメントがないこと/コメントが適切に残されていること 処理の共通化が行われていること ネストが深すぎないこと ここではif文のネストについてまとめていきたいと思います。 if文による条件分岐を書くことは多々あると思うのですが、このネストが深くなればなるほど 処理を追うことがとても大変になります。 実務上ではあまりない例にはなりますが、下記ページとボタンの種類によって処理を分ける例になります。 type PageType…

業務でコードレビューを受けたり、コードレビューをしたりする機会が増えてきたので、 個人的に読みやすく綺麗なコードを書くために気をつけていることをまとめたいと思います。 また、最近はTypeScript/React.jsでフロントエンド開発がメインのため、 サンプルコードはそちらを使用したいと思います。 読みやすいコード/綺麗なコードとは 読みやすいコード、綺麗なコードの解釈には個人差があると思いますが、 ざっくり下記項目ができていたらオッケーとします。 変数名、関数名が適切な名前になっていること プロジェクトの命名規則に統一されていること 改行が適切に行われていること ネストが深すぎないこと …

フロントエンドエンジニアをしている瀧です。 フロントエンドとは、ユーザーが直接目にする画面や動きの部分のことで、ユーザーインタフェースや、ユーザー体験を開発しています。 この記事では私のチームで行っているコードレビューについて紹介します。 瀧 章倫（タキ アキノリ） シナジーマーケティング2005年入社の古参のメンバーです。 現在はSynergy!のフロントエンドの開発に携わっています。 はじめに コードレビューとは開発工程の１つで、自分が書いたコードを見てもらったり他人が書いたコードを見たりする仕事のことです。 レビューと聞くと印象的には評価・評論・批評といった感じですよね。粗探しをされてい…

I went to my office. I disscussed with the intern to improve his code. He's smart. Could I do smething for him...? I had one-on-one with my boss. I'm asked be interested in being a manager. But I want to go to Denmark next year. How do I behave? My girlfriend and I will have discussion about it tomo…

はじめに こんにちは、食べログシステム本部 アプリ開発部でAndroidアプリエンジニアをやっているhokutonikukyuです！ 柴犬が大好きです🐕 入社から食べログを担当し、ちょうど丸２年経ちました。 普段は口コミ周りの開発を担当しています！ 去年も参加したのですが、今年はTabelog Tech BlogにレポートとしてDay1~3まで連載することになりました！ 本記事はDroidKaigi2023のDay2のレポートです。 Day1の和田さんのレポートは以下から、こちらもぜひご覧下さい！ 各社のブースも大盛況でした！ブースでスタンプを集めてノベルティーを貰えたり、両日共にコーヒーやお…

現代のテクノロジーの基盤となるプログラミングは、私たちの日常生活において非常に重要なスキルとなっています。コンピュータやスマートフォン、ウェブサイトなど、私たちが日常的に利用している多くのものがプログラミングによって実現されています。しかし、プログラミングの基礎知識を理解することは初心者にとって容易ではありません。この記事では、プログラミングの基礎知識について詳しく解説し、初心者の方々がプログラミングの世界に足を踏み入れる際の手助けとなることを目指します。 プログラミング言語の選択 プログラミングの基礎を学ぶには、まずプログラミング言語の選択が非常に重要です。プログラミング言語は、コンピュータ…

初めに みなさんは人生において挫折経験はありますか？ 現在25歳の僕は色々な物事に手を出しては挫折を繰り返してきました。 直近では動画編集ブームに乗り独学でスキル習得を試みるもののすぐに投げ出す。 趣味でピアノを始めようと電子ピアノを買ったが全く弾かずに売り払う。 原因は全て自分の怠惰によるものです。 やる気が出ない。仕事で疲れて勉強ができない。好きな事を辞められない。 自分に言い訳ばかりして生きてきました。 そんな自他共に認める自堕落な僕ですが、HapinessChain（以下HC）というプラグミングスクールに入会して3週間が経ちました。 今回は実際に入会して感じたことをテーマに記事を書いて…

1.はじめに 2.データ活用の民主化に向けた現状と課題 3.データアーキテクチャーデザイン 3-1.避けては通れぬ茨の道 3-2.イシューから始める 3-3.採用したデータパイプライン 4.開発デザイン 4-1.データマート設計 4-2.ルールメイキング 5.おわりに 5-1.利用状況 5-2.カイゼン計画 1.はじめに NTTドコモ データプラットフォーム部（以下DP部）外山です。 NTTドコモでは、携帯回線事業の他、dポイント・dカード・d払いを中心にお客様の新たなライフスタイルを創出する各種d系サービスを提供し、提供後のプロダクト改善やマーケティング施策の実施にデータを活用しています。 …

エンジニアという職種は、絶えず進化する技術の最前線で活躍する者たちの集まりです。しかし、その華やかな世界の入り口は、どこにあるのでしょうか？この記事では、エンジニアとしての道を歩み始めるための具体的なステップと、その背景にある考え方を紹介します。 興味を持つこと エンジニアリングの世界への第一歩は、興味を持つことから始まります。興味は、学びの原動力となり、未知の領域への探求心を駆り立てるエネルギー源となります。 なぜ興味が大切なのか 興味は、新しい知識やスキルを獲得する際の最も強力なモチベーションの一つです。興味を持っていることを学ぶ時、それは楽しみであり、情熱をもって取り組むことができます。…

フルスタックエンジニアとしてのキャリアパスは多様で、技術的スキル、ビジネススキル、コミュニケーション能力などが組み合わさる形で進行します。以下に一般的なキャリアパスとその各ステージで考慮すべきポイントを示します。 初級フルスタックエンジニア 主な仕事: シンプルな機能の開発、バグの修正、ドキュメント作成 スキルセット: 基本的なプログラミングスキル、バージョン管理（Gitなど）、データベース操作 成長のために: メンターを見つけてフィードバックを求める、プロジェクトでの責任範囲を広げる 中級フルスタックエンジニア 主な仕事: 複雑な機能の設計と実装、コードレビュー、テスティング スキルセット:…

Agile at Tesla and SpaceX. Elements of agile development seen in the first-generation Prius and its application in the current environment / 初代プリウスにみるアジャイル開発の要素と現代の環境での進め方について 名古屋にある電力小売企業のスクラムチームが自己組織化に向けて奮闘している話 アジャイルな組織を作るために開発チーム作成ガイドを書いた話 ソフトウェア開発に限らない、アジャイルチームとなるための要素 〜VCの支援専門チームにおける事例からの考察〜 ア…

今年の1月から新規立ち上げの開発部署でチームリーダーをやっていて、色々と「俺の考えた最強の開発手法」的なものを試しているのだけれど、その一環で、日本語でなく英語でコミットメッセージやソースコードのコメントを書くという開発ルールを導入してみた。 その結果、メリットよりもデメリットが大きいことが判明して、チームとして英語で書くのをやめにしたので、その振り返りを書きたい。 英語でコメントを書くメリット Web開発の世界でグローバルな共通語として機能している 日本語入力に切り替える必要がないので、タイピング効率が良い 文法的に日本語よりもロジカルに書きやすい GitHub CopilotやChatGP…

今年もDroidKaigi 2023にオフライン参加しました。 そのまとめブログです。 Day1 Modifier.Nodeを使いましょう よく見るあのUIをJetpack Composeで実装する方法〇選 iOSとAndroidで定期購入の意図しない解約を防ぐ YouTubeへのライブ配信機能をリリースするまで Master of NestedScroll Day2 MVIに基づくStateMachineアーキテクチャ：KMMとJetpack ComposeとSwiftUIを組み合わせる Androidの『音』を制御する Androidアプリの良いユニットテストを考える Jetpack Co…

Day1 はこちらにまとめています chigichan24.hatenablog.com 11:00~ Kotlinハイパフォーマンスプログラミング speakerdeck.com Kotlin(JVM)は遅いみたいなのは、ART時代の産物ぽいよねと確かに改めて思いました。 キャッシュヒットまで考えたカリカリチューニングはしたことがないので、勉強になりました。 syntax sugar がインスタンスを使い捨てているのはなるほどになりました。 11:00~ ビジネス向けアプリを開発するときに知っておくべきAndroid Enterpriseの世界 speakerdeck.com Android…

はじめに 私はプログラミングの学習を6月に始めました。 HappinessChainへの入校は8月27日からなので、入校から約3週間が経過したことになります。 HappinessChainに入るまで 他のスクールで受講していました。 そのスクールに不満があったから乗り換えたわけですが、あまり悪口は書きたくないのでこの記事で批判的なことは言わないように控えます。 HappinessChainに入る決断をした理由は、他のスクールで受講していた時に残念だなあと思った部分が改善されそうと感じたからです。 以下にその理由をご紹介します。 主要な理由２つ それぞれの課題に対してプロのコードレビューが提供さ…