From 2228cb510f3475c0085ca59e124a503c7c2a986b Mon Sep 17 00:00:00 2001 From: haniwachan Date: Wed, 13 Nov 2024 06:18:58 -0800 Subject: [PATCH 1/5] 200 Number of islands --- 05_Graph,BFS,DFS/200_Number_of_Islands.md | 252 ++++++++++++++++++++++ 1 file changed, 252 insertions(+) create mode 100644 05_Graph,BFS,DFS/200_Number_of_Islands.md diff --git a/05_Graph,BFS,DFS/200_Number_of_Islands.md b/05_Graph,BFS,DFS/200_Number_of_Islands.md new file mode 100644 index 0000000..146286e --- /dev/null +++ b/05_Graph,BFS,DFS/200_Number_of_Islands.md @@ -0,0 +1,252 @@ +＜問題＞ +https://leetcode.com/problems/two-sum/ + +# step1 + +5分程度答えを見ずに考えて、手が止まるまでやってみる。 +何も思いつかなければ、答えを見て解く。 +ただし、コードを書くときは答えを見ないこと。 +正解したら一旦OK。 +思考過程もメモしてみる。 + +```c++ +class Solution { +private: + int row_size,column_size; + int num_island=0; + vector> recognized_island; //既に島として認識されている点を1とする。 + +//start_pointと同じ島である点を調べ上げる。 + void check_same_island (pair start_point, vector>& grid) { + queue> seen; + vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + + seen.push(start_point); + while (!seen.empty()) { + int x_connected,y_connected; + tie(x_connected, y_connected) = seen.front(); + recognized_island[x_connected][y_connected] = '1'; + seen.pop(); + for (int i=0; i= row_size || y >= column_size){ + continue; + } + if (grid[x][y] == '0' || recognized_island[x][y] == '1'){ + continue; + } + seen.push({x,y}); + } + } + } +public: + int numIslands(vector>& grid) { + row_size = size(grid); + column_size = size(grid[0]); + recognized_island.assign(row_size, vector(column_size, 0)); + + for (int i = 0; i < row_size; i++) { + for (int j = 0; j < column_size; j++){ + if (grid[i][j] == '0' || recognized_island[i][j] == '1') { + continue; + } + check_same_island ({i,j}, grid); + ++num_island; + } + } + return num_island; + } +}; +``` + + +【考えたこと】 +- (i,j)からスタートして、幅優先探索で縦横繋がっている点を追っていく。 +- (i,j)をfor文で回す。 +- 通った点をわかるようにするテーブルを用意した方がよさそう。 + これにより一度通った点かどうかがわかり、(i,j)スタートで幅優先探索する必要があるかわかる。 + →最初にどういう変数を作ると良いかや、必要な処理を言語化してからコーディングを始めるとよさそうな気がする。 + + +- アルゴリズムは頭なのかでどうやって実施したらいいかわかっていても、コーディングに落とすのが難しく感じている。 +- vectorを動的に作成する必要あり.配列の初期値を作る方法を調べるのに時間がかかること多い。 +https://qiita.com/alchemist/items/6cd2a86db7377ad8d236 + +# step2 +他の方が描いたコードを見て、参考にしてコードを書き直してみる。 +参考にしたコードのリンクは貼っておく。 +読みやすいことを意識する。 +他の解法も考えみる。 + +Nがノード数(ここではgridの数)、Mがエッジ数(端を除いて4N) +計算量：O(N) + + +```c++ +class Solution { +private: + int row_size,column_size; + int num_island=0; + vector> recognized_island; //既に島として認識されている点を1とする。 + +//start_pointと同じ島である点を調べ上げる。 + void check_same_island (pair start_point, vector>& grid) { + queue> seen; + vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + + seen.push(start_point); + while (!seen.empty()) { + int x_connected,y_connected; + tie(x_connected, y_connected) = seen.front(); + recognized_island[x_connected][y_connected] = '1'; + seen.pop(); + for (int i=0; i= row_size || y >= column_size){ + continue; + } + if (grid[x][y] == '0' || recognized_island[x][y] == '1'){ + continue; + } + seen.push({x,y}); + } + } + } +public: + int numIslands(vector>& grid) { + row_size = size(grid); + column_size = size(grid[0]); + recognized_island.assign(row_size, vector(column_size, 0)); + + for (int i = 0; i < row_size; i++) { + for (int j = 0; j < column_size; j++){ + if (grid[i][j] == '0' || recognized_island[i][j] == '1') { + continue; + } + check_same_island ({i,j}, grid); + ++num_island; + } + } + return num_island; + } +}; +``` + +dscordやネットなどを調べてみましたが、step1から変更点はありません。 + + +【以前に解かれた方のコメントから学んだこと。】 + +参照:https://github.com/hroc135/leetcode/pull/17#pullrequestreview-2305316186 + +- 参照透過性：関数を呼び出す位置によって、挙動が変わらないこと。 + https://web.sfc.keio.ac.jp/~hattori/prog-theory/ja/functional.html +- 関数名は動詞にするべき。 +- 変数名の英語は省略しない。 + +参照:https://github.com/seal-azarashi/leetcode/pull/17#pullrequestreview-2276239814 + +> 不等号の式の左右の方向は、比較されるほうを左に持ってくる派と、数直線上に一直線上に並べる派がいるように思います。実際の現場においては、チームのやり方に合わせることをお勧めします。 + +>また、一般に、変数には肯定的な意味合いを持たせ、式の中で ! を使って否定したほうが読みやすくなると思います。今回の場合とは違うのですが、否定的な意味合いの変数を ! を使って否定すると、二重否定による肯定になり、認知負荷が上がり、読みにくく感じます。 + +>dfs という名前は、内部実装の分類で名前をつけているわけですが、関数の呼び出し元を読んだ人は、その内部実装を知りたいと思うことは考えにくいのです。 +電源コンセントがあったら、100V 15A なのか 200V 20A なのかがまず知りたくて、石炭なのか水力なのかだけ書いてあってもそこじゃないでしょう。 +だいたいの場合、関数名は中身を見なくてもだいたい何をしているか分かるようにして、変数名は何に使うかが分かればいいのです。上から読んでいって明らかで、そして、すぐに忘れて良い場合には一文字でもよいです。たとえば、0からある配列の長さまで回していたら添字だろうと推測できますね。 + +- ヒープ領域 +聞いたことはあるが、あまり詳しくないので調べてみた。 +プログラムの中で、動的にメモリを確保したり、削除したりできる。 +ポインタを使ってアドレスを管理するときにはこの領域を使う。 +https://ja.wikipedia.org/wiki/%E3%83%92%E3%83%BC%E3%83%97%E9%A0%98%E5%9F%9F + +- スタック領域 +ヒープ領域を調べているとこちらも出てきた。 +関数を実行したり、ローカル変数を定義するときに使う。 +https://uquest.tktk.co.jp/embedded/learning/lecture07-1.html + + + +参照:https://github.com/thonda28/leetcode/pull/15 + +- 再帰でも実装できる。 + + +参照:https://github.com/TORUS0818/leetcode/pull/19 + +- 深さ優先探索でも実装しておきたい。 +- union find木について名前だけ聞いたことがあるレベルなので、調べてみました。 + - Discord + https://discord.com/channels/1084280443945353267/1183683738635346001/1197738650998415500 + - https://note.nkmk.me/python-union-find/ + - Union:iとjの大親分を同じにする。find:iの一番の大親分を見つける。 + - この問題なら、(i,j)に対して上下左右が陸なら、上下左右の親を(i,j)にするという操作を全i,jに対して行い、最後(i,j)の大親分を全て探索して、大親分の数を数える。 + +参考: +https://github.com/fhiyo/leetcode/pull/20/files + + + + +# step3 + +今度は、時間を測りながら、もう一回、書きましょう。書いてアクセプトされたら文字消してもう一回書く。これを10分以内に一回もエラーを出さずに書ける状態になるまで続ける。3回続けてそれができたらその問題はOK。 + +実施しました。 + + +```c++ +class Solution { +private: + int row_size,column_size; + int num_island=0; + vector> recognized_island; //既に島として認識されている点を1とする。 + +//start_pointと同じ島である点を調べ上げる。 + void check_same_island (pair start_point, vector>& grid) { + queue> seen; + vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + + seen.push(start_point); + while (!seen.empty()) { + int x_connected,y_connected; + tie(x_connected, y_connected) = seen.front(); + recognized_island[x_connected][y_connected] = '1'; + seen.pop(); + for (int i=0; i= row_size || y >= column_size){ + continue; + } + if (grid[x][y] == '0' || recognized_island[x][y] == '1'){ + continue; + } + seen.push({x,y}); + } + } + } +public: + int numIslands(vector>& grid) { + row_size = size(grid); + column_size = size(grid[0]); + recognized_island.assign(row_size, vector(column_size, 0)); + + for (int i = 0; i < row_size; i++) { + for (int j = 0; j < column_size; j++){ + if (grid[i][j] == '0' || recognized_island[i][j] == '1') { + continue; + } + check_same_island ({i,j}, grid); + ++num_island; + } + } + return num_island; + } +}; +``` From bb0d8a870976720e3bdba4e4e3916b7b98134640 Mon Sep 17 00:00:00 2001 From: haniwachan Date: Wed, 13 Nov 2024 06:21:36 -0800 Subject: [PATCH 2/5] =?UTF-8?q?=E5=95=8F=E9=A1=8C=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- 05_Graph,BFS,DFS/200_Number_of_Islands.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/05_Graph,BFS,DFS/200_Number_of_Islands.md b/05_Graph,BFS,DFS/200_Number_of_Islands.md index 146286e..b1f60d2 100644 --- a/05_Graph,BFS,DFS/200_Number_of_Islands.md +++ b/05_Graph,BFS,DFS/200_Number_of_Islands.md @@ -1,5 +1,5 @@ ＜問題＞ -https://leetcode.com/problems/two-sum/ +https://leetcode.com/problems/number-of-islands/description/ # step1 From 3f2c0ab5bb061ac00fc326194e1272712566b05b Mon Sep 17 00:00:00 2001 From: haniwachan Date: Wed, 13 Nov 2024 06:25:02 -0800 Subject: [PATCH 3/5] =?UTF-8?q?=E3=82=B9=E3=83=9A=E3=83=BC=E3=82=B9?= =?UTF-8?q?=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- 05_Graph,BFS,DFS/200_Number_of_Islands.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/05_Graph,BFS,DFS/200_Number_of_Islands.md b/05_Graph,BFS,DFS/200_Number_of_Islands.md index b1f60d2..5450cb1 100644 --- a/05_Graph,BFS,DFS/200_Number_of_Islands.md +++ b/05_Graph,BFS,DFS/200_Number_of_Islands.md @@ -19,7 +19,7 @@ private: //start_pointと同じ島である点を調べ上げる。 void check_same_island (pair start_point, vector>& grid) { queue> seen; - vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + vector> connected_point = {{1,0},　{-1,0},　{0,1},　{0,1}}; seen.push(start_point); while (!seen.empty()) { @@ -27,7 +27,7 @@ private: tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); - for (int i=0; i start_point, vector>& grid) { queue> seen; - vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + vector> connected_point = {{1,0},　{-1,0},　{0,1},　{0,1}}; seen.push(start_point); while (!seen.empty()) { @@ -102,7 +102,7 @@ private: tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); - for (int i=0; i start_point, vector>& grid) { queue> seen; - vector> connected_point = {{1,0},{-1,0},{0,1},{0,1}}; + vector> connected_point = {{1,0},　{-1,0},　{0,1},　{0,1}}; seen.push(start_point); while (!seen.empty()) { @@ -217,7 +217,7 @@ private: tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); - for (int i=0; i Date: Wed, 13 Nov 2024 06:27:16 -0800 Subject: [PATCH 4/5] =?UTF-8?q?=E3=82=B9=E3=83=9A=E3=83=BC=E3=82=B9?= =?UTF-8?q?=E4=BF=AE=E6=AD=A3#2?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- 05_Graph,BFS,DFS/200_Number_of_Islands.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/05_Graph,BFS,DFS/200_Number_of_Islands.md b/05_Graph,BFS,DFS/200_Number_of_Islands.md index 5450cb1..13797b7 100644 --- a/05_Graph,BFS,DFS/200_Number_of_Islands.md +++ b/05_Graph,BFS,DFS/200_Number_of_Islands.md @@ -23,7 +23,7 @@ private: seen.push(start_point); while (!seen.empty()) { - int x_connected,y_connected; + int x_connected, y_connected; tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); @@ -98,7 +98,7 @@ private: seen.push(start_point); while (!seen.empty()) { - int x_connected,y_connected; + int x_connected, y_connected; tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); @@ -213,7 +213,7 @@ private: seen.push(start_point); while (!seen.empty()) { - int x_connected,y_connected; + int x_connected, y_connected; tie(x_connected, y_connected) = seen.front(); recognized_island[x_connected][y_connected] = '1'; seen.pop(); From a799a8e7598e51d24022fd17ca1eeee8ad11406e Mon Sep 17 00:00:00 2001 From: haniwachan Date: Thu, 14 Nov 2024 22:27:32 -0800 Subject: [PATCH 5/5] =?UTF-8?q?#step4=E3=82=92=E8=BF=BD=E5=8A=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- 05_Graph,BFS,DFS/200_Number_of_Islands.md | 68 +++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/05_Graph,BFS,DFS/200_Number_of_Islands.md b/05_Graph,BFS,DFS/200_Number_of_Islands.md index 13797b7..c1f7d19 100644 --- a/05_Graph,BFS,DFS/200_Number_of_Islands.md +++ b/05_Graph,BFS,DFS/200_Number_of_Islands.md @@ -250,3 +250,71 @@ public: } }; ``` + +# step4 +レビューを受けてコードの修正。 + +```c++ +class Solution { +private: + int _row_size, _column_size; + int _num_island = 0; + +// start_pointと同じ島である点を調べ上げる。 + void check_same_island (pair start_point, const vector>& grid, + vector> & visited_island ) { + queue> cells_to_visit; + const vector> deltas = {{1,0}, {-1,0}, {0,1}, {0,-1}}; + cells_to_visit.push(start_point); + visited_island[start_point.first][start_point.second] = 1; + while (!cells_to_visit.empty()) { + auto [row_current, col_current] = cells_to_visit.front(); + cells_to_visit.pop(); + for (auto [delta_row, delta_col] : deltas) { + int row, col; + row = row_current + delta_row; + col = col_current + delta_col; + if( ! (row >= 0 && row < _row_size && col < _column_size && col >= 0 ) ){ + continue; + } + if (grid[row][col] == '0' || visited_island[row][col] == 1) { + continue; + } + cells_to_visit.push({row, col}); + visited_island[row][col] = 1; + } + } + } + +public: + int numIslands(vector>& grid) { + vector> visited_island; // 既に島として認識されている点を1とする。 + _row_size = grid.size(); + _column_size = grid[0].size(); + visited_island.assign(_row_size, vector(_column_size, 0)); + + for (int i = 0; i < _row_size; ++i) { + for (int j = 0; j < _column_size; ++j){ + if (grid[i][j] == '0' || visited_island[i][j] == 1) { + continue; + } + check_same_island ({i, j}, grid, visited_island); + ++_num_island; + } + } + return _num_island; + } +}; +``` + +- 定数の宣言に関して +https://ttsuki.github.io/styleguide/cppguide.ja.html#Use_of_constexpr + +https://learn.microsoft.com/ja-jp/cpp/cpp/constexpr-cpp?view=msvc-170 + +https://rinatz.github.io/cpp-book/ch07-05-constructors/ + +https://stackoverflow.com/questions/27065617/const-vector-implies-const-elements + +- スレッドセーフについて +https://www.divx.co.jp/media/techblog-220627 \ No newline at end of file