LeetCode 200. Number of Islands #40

huyfififi · 2026-01-10T07:01:27Z

https://leetcode.com/problems/number-of-islands/description/

mamo3gr · 2026-01-11T07:21:21Z

leetcode/200/step3.cpp

+class Solution {
+public:
+    int numIslands(const std::vector<std::vector<char>>& grid) {
+        constexpr char kWater = '0';


numIslandsの内部ではなく、Solution のメンバとして宣言する、という選択肢もありそうでした（Solution=問題で与えられた定数ですよね）。

Solutionという抽象的なクラス名なので、複数の問題への解答を持つ可能性もあるかな...と考え numIslandsに関連するものは全て中に置きたい気持ちがあったのですが、考えすぎだったかもしれません。ありがとうございます！

Solutionという抽象的なクラス名なので、複数の問題への解答を持つ可能性もあるかな...と考え

この想定は思いつきませんでした。それはそれで一貫性があるので問題ないと思います！

mamo3gr · 2026-01-11T07:31:31Z

leetcode/200/step2.cpp

+                std::queue<std::pair<int, int>> area_to_visit;
+                area_to_visit.push({row, col});
+                while (!area_to_visit.empty()) {
+                    auto [r, c] = area_to_visit.front();


個人的には、スコープも狭いし、直前で row, col をpushしていることから、r, c でも読めました。

フィードバックいただきありがとうございます。自分以外からどう見えるかのサンプルが増えて助かります。

同じく，直前のrow, colのpushにより意図はすぐ読み取れました．

mamo3gr · 2026-01-11T07:40:28Z

leetcode/200/step3.cpp

+                std::queue<std::pair<int, int>> area_to_visit;
+                area_to_visit.push({row, col});
+                while (!area_to_visit.empty()) {
+                    auto [visiting_row, visiting_col] = area_to_visit.front();
+                    area_to_visit.pop();
+                    if (visiting_row < 0 || num_rows <= visiting_row ||
+                        visiting_col < 0 || num_cols <= visiting_col) {
+                        continue;
+                    }
+                    if (grid[visiting_row][visiting_col] == kWater) { continue; }
+                    if (visited[visiting_row][visiting_col]) { continue; }
+
+                    area_to_visit.push({visiting_row + 1, visiting_col});
+                    area_to_visit.push({visiting_row - 1, visiting_col});
+                    area_to_visit.push({visiting_row, visiting_col + 1});
+                    area_to_visit.push({visiting_row, visiting_col - 1});
+                    visited[visiting_row][visiting_col] = true;
+                } 


好みの範囲ですが、自分ならここをサブルーチンに切り出します。理由としては「未訪問のセルをvisitedとしてマークしてるんだよ」という、forループの中の大半を占める処理（かつ境界判定など細かい処理を含む）が、サブルーチンの名前として要約できるからです。デメリットは関数を呼ぶオーバーヘッドや、num_{rows,cols} を取り出し直したり、grid などを引き回したりする手間があることです。
ちなみにC++詳しくないのですが、コンパイル時に展開することでオーバーヘッドを無くせたりしませんでしたっけ。

ありがとうございます。ここのやや複雑な処理を抽象度高くまとめてしまえば全体としてnumIslandsが何をするのか頭に収まりやすくなりそうですね。

また、コンパイル時の最適化により関数呼び出しのオーバーヘッドをなくすことができるのを知りませんでした、ありがとうございます。確かに、コンパイル時に最適化を有効にすれば

If the compiler performs function inlining, it replaces a call of that function with its body, avoiding the overhead of a function call (placing data on stack and retrieving the result)

関数をそのまま呼び出し側に展開して呼び出しのオーバーヘッドを避けることができるようです。

nodchip · 2026-01-11T09:27:12Z

leetcode/200/step1_iterative_bfs.cpp

+class Solution {
+public:
+    int numIslands(std::vector<std::vector<char>>& grid) {
+        const int kWater = '0';


定数は static なメンバ変数か、グローバルに定義することが多いように思います。

nodchip · 2026-01-11T09:28:07Z

leetcode/200/step1_iterative_bfs.cpp

+public:
+    int numIslands(std::vector<std::vector<char>>& grid) {
+        const int kWater = '0';
+        const int num_rows = (int)grid.size();


static_cast() でキャストしたほうが C++ っぽく感じるのですが、やりすぎかもしれません。チームの平均的な書き方に合わせることをお勧めいたします。

参考までにスタイルガイドへのリンクを共有いたします。

https://google.github.io/styleguide/cppguide.html#Casting

Use C++-style casts like static_cast(double_value), or brace initialization for conversion of arithmetic types like int64_t y = int64_t{1} << 42. Do not use cast formats like (int)x unless the cast is to void.

なお、このスタイルガイドは“唯一の正解”というわけではなく、数あるガイドラインの一つに過ぎません。チームによって重視される書き方や慣習も異なります。そのため、ご自身の中に基準を持ちつつも、最終的にはチームの一般的な書き方に合わせることをお勧めします。

リファレンスまでいただきありがとうございます！

nodchip · 2026-01-11T09:29:53Z

leetcode/200/step1_iterative_bfs.cpp

+public:
+    int numIslands(std::vector<std::vector<char>>& grid) {
+        const int kWater = '0';
+        const int num_rows = (int)grid.size();


個人的にはプリミティブ型のローカル変数には const を付けないのですが、趣味の範囲だと思います。所属するチームの平均的な書き方に合わせることをお勧めいたします。

横から失礼します。

@nodchip

個人的にはプリミティブ型のローカル変数には const を付けない

C++に詳しくないのですが、この趣味の背景を知りたいです。多少視覚のノイズにはなるものの、中身を変えるつもりがないなら const を付けておいたほうが安全に思う（し、そうであることを読み手にも伝えられると思う）のですが、どういう取捨選択なのか教えていただけますか。

非常に個人的な体験からなのですが、ほかの方から「const 教」と揶揄されたためです。

基本的に文字数が増える以外つけて損はないんですけども、たとえば3行しかスコープのない変数には const とつけなくても変更がないことが明らかだろと、言いたい人はいるんじゃないですかね。

nodchip · 2026-01-11T09:31:14Z

leetcode/200/step1_iterative_bfs.cpp

+
+                ++num_islands;
+
+                std::queue<std::tuple<int, int>> area_to_explore;


座標を表す構造体を定義し、それを push() していったほうが、読み手にとって分かりやすいかもしれません。ただ、大げさすぎるように思います。

nodchip · 2026-01-11T09:32:29Z

leetcode/200/step1_iterative_bfs.cpp

+                    auto [r, c] = area_to_explore.front();
+                    area_to_explore.pop();
+
+                    if (r < 0 || num_rows <= r || c < 0 || num_cols <= c) { continue; }


r と c が視覚的にまとまるよう、

if (!(0 <= r && r < num_rows && 0 <= c && c < num_cols))

と書いたほうが分かりやすいと思います。

確かにそうですね、ありがとうございます！

5ky7 · 2026-01-16T08:52:19Z

leetcode/200/step1_recursive_dfs.cpp

+        }
+
+        visited[row][col] = true;
+        const std::tuple<int, int> directions[4] = {{1, 0}, {0, 1}, {-1, 0}, {0, -1}};


staticなメンバ変数として定義して良いと思いました．

5ky7 · 2026-01-16T08:53:43Z

leetcode/200/step2.cpp

+                std::queue<std::pair<int, int>> area_to_visit;
+                area_to_visit.push({row, col});
+                while (!area_to_visit.empty()) {
+                    auto [r, c] = area_to_visit.front();


同じく，直前のrow, colのpushにより意図はすぐ読み取れました．

5ky7 · 2026-01-16T08:55:03Z

leetcode/200/step2.cpp

+                    area_to_visit.push({r + 1, c});
+                    area_to_visit.push({r - 1, c});
+                    area_to_visit.push({r, c + 1});
+                    area_to_visit.push({r, c - 1});


好みだとは思いますがforで回す方が読みやすかったです．

huyfififi added 11 commits January 9, 2026 16:47

LeetCode 200: Add step 1 recursive DFS

a68db3d

LeetCode 200: Add step 1 iterative BFS

a62d084

LeetCode 200: Add step 1 memo

1ada782

LeetCode 200: Prune push operations (step 1 iterative BFS)

c35bc56

LeetCode 200: Browsed PRs in Discord

1a9bcc6

LeetCode 200: Checked Google C++ Style Guide for constant names

eaedaa5

LeetCode 200: Compare push-before-checking vs push-after-checking

311e79a

LeetCode 200: Tweak wording in step 2 note

e10e778

LeetCode 200: Add step 2 solution

01a973a

LeetCode 200: Add note on guard against empty grid

efb510c

LeetCode 200: Add step 3 solution

d94d727

huyfififi self-assigned this Jan 10, 2026

huyfififi marked this pull request as ready for review January 10, 2026 07:20

mamo3gr reviewed Jan 11, 2026

View reviewed changes

nodchip reviewed Jan 11, 2026

View reviewed changes

huyfififi added 2 commits January 11, 2026 18:38

LeetCode 200: Researched function inlining

17e3342

LeetCode 200: Note feedback

09b8620

5ky7 reviewed Jan 16, 2026

View reviewed changes


		++num_islands;

		std::queue<std::tuple<int, int>> area_to_explore;

LeetCode 200. Number of Islands #40

Are you sure you want to change the base?

LeetCode 200. Number of Islands #40

Uh oh!

Conversation

huyfififi commented Jan 10, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants