695. Max Area of Island #17

rinost081 · 2025-05-23T18:05:59Z

No description provided.

potrue · 2025-05-23T18:26:55Z

lc695.md

+- https://github.com/plushn/SWE-Arai60/pull/18#discussion_r2099839166
+
+- https://github.com/plushn/SWE-Arai60/pull/18/files?short_path=2e957ae#diff-2e957aed1bb9dc5099abe16f37afb1a7980cfb2170d04ad76a48b4566e75eb35
+- step1で再帰で実装するのを諦めたが際に, この方のstep1をみて実装のミスに気づいた.- [再帰のコールスタック制限を変更するsetrecursionlimit](https://docs.python.org/ja/3/library/sys.html#sys.setrecursionlimit)を見たが, 結局これを設定するのにはどうすれば良いのかわからない. とりあえず知識として持っておこうと思う


import sys sys.setrecursionlimit(1000000)

みたいな感じじゃないでしょうか。
調べてみたところデフォルトは1000のようです（環境によっては違うかもしれません）

ありがとうございます、そうですね書き方としてはそちらで良いと思いました。ただ、ここでコメントした"分からない"というのは、値をどの判断基準で設定するのかという点です(説明不足でした)。

例えば概算をしてみて、再帰がどれくらいになりそうだからデフォルトを超えそうなのでXと設定しようみたいにして使うんですかね。

potrue · 2025-05-23T18:29:30Z

lc695.md

+class Solution:
+    def maxAreaOfIsland(self, grid: List[List[int]]) -> int:
+        WATER = 0
+        ISLAND = 1


ISLANDではなくLANDのほうがいいかもしれません。ただ結局使われてないので定義しなくてもいいかもです。

potrue · 2025-05-23T18:32:58Z

lc695.md

+        def inside_island(row, column):
+            return 0 <= row < num_rows and 0 <= column < num_columns
+
+        def traverse_and_count_island(row, column):


個人的な感覚かもしれませんが、面積(area)は測ったり計算するものであって数える(count)というイメージではないので、countよりもcalculate_areaなどの名前のほうが適切かもしれません。

calculate は数式を用いて計算するというニュアンスを感じます。今回の場合は get_area() あたりが良いと思います。

potrue · 2025-05-23T18:37:38Z

lc695.md

+
+            while island:
+                r, c = island.popleft()
+                for (i, j) in ((1, 0), (-1, 0), (0, 1), (0, -1)):


iとjという名前についてですが、rとi、およびcとjの間に名前の直接的な関連性がないので、少しわかりにくいかもしれないと思いました。
drやdcなどの変数名にしてる人が多い気がしますが、両方1文字にしておかないと直後で何回も繰り返すので面倒という感じもします。
(r + i, c + j)のtuple自体をcellみたいな別の変数名に保存してしまって後の行で使う選択肢もあるかもしれません。

r + i, c + j がこんだけ繰り返すならば変数に置きませんか。

あと、append_if_necessary みたいな関数を作って
append_if_necessary(r + 1, c) などと四回呼ぶ手もありますよ。

それと、私は i, j は避けるかもしれません。index らしさがあまりないですし、寿命の長い i, j はたまに事故を起こすので。
https://discord.com/channels/1084280443945353267/1367399154200088626/1375219482989166745

そうですね、ここは改善の余地がありそうです。

コメントいただいたように変数として置き直すとかでも良いと思いました

oda · 2025-05-23T20:01:26Z

lc695.md

+
+            while island:
+                r, c = island.popleft()
+                for (i, j) in ((1, 0), (-1, 0), (0, 1), (0, -1)):


r + i, c + j がこんだけ繰り返すならば変数に置きませんか。

あと、append_if_necessary みたいな関数を作って
append_if_necessary(r + 1, c) などと四回呼ぶ手もありますよ。

それと、私は i, j は避けるかもしれません。index らしさがあまりないですし、寿命の長い i, j はたまに事故を起こすので。
https://discord.com/channels/1084280443945353267/1367399154200088626/1375219482989166745

oda · 2025-05-23T20:01:48Z

lc695.md

+                for (i, j) in ((1, 0), (-1, 0), (0, 1), (0, -1)):
+                    if not inside_island(r + i, c + j):
+                        continue
+                    elif grid[r + i][c + j] == WATER:


私はこれは if にしますね。elif にする理由がないので。

tokuhirat · 2025-05-24T00:12:07Z

lc695.md

+                    continue
+                elif (i, j) in visited_island:
+                    continue
+                area = traverse_island(area, i, j)


L45 を見て area = 0 ってなんだろうと思ってここまで読むと、L50 の引数に登場していました。あまり整理できていない段階でコードを書くと不要な部分が含まれることもありますが、書き上げた後、上からコードを読んでシミュレーションすることでコード整理までできれば良さそうと思いました。

解くことに集中してしまって雑になってしまいました、そもそもここで定義する必要がないですね

shintaro1993 · 2025-05-26T07:59:05Z

lc695.md

+        def inside_island(row, column):
+            return 0 <= row < num_rows and 0 <= column < num_columns
+
+        def traverse_and_count_island(row, column):


好みだと思いますが、シンプルに count_island の方が意図が伝わりやすくなるのかなと思いました。

count_island だと、面積ではなく島の数を数えているように感じられました。

quinn-sasha · 2025-06-14T17:35:42Z

lc695.md

+        def inside_island(i, j):
+            return 0 <= i < num_rows and 0 <= j < num_columns
+
+        def traverse_island(area, row, column):


area を引数とせずに、この関数内のローカル変数とする方法もあります。
以下の実装を見ると、この関数を呼び出したときは必ず area = 1 となります。

nodchip · 2025-06-16T10:09:58Z

lc695.md

+# step1
+思考ログ
+- 前回のやつとかなり似ている問題
+- dfsとbfsでどの場面でどう使い分けるのかにピンときていない.


あまり明確な基準ではないのですが、自分は以下のように使い分けています。

dfs

答えが一つ見つかればよい場合

再帰関数でシンプルに書け、スタックオーバーフローしない場合

bfs

最短経路を見つけたい場合

探索済みのノードの隣のノードの集合が必要になる場合

塗りつぶし等、上記のどれにも当てはまらない場合は、どちらでもよいと思います。その場合は気分で決めることが多いです。

nodchip · 2025-06-16T10:12:22Z

lc695.md

+        def inside_island(row, column):
+            return 0 <= row < num_rows and 0 <= column < num_columns
+
+        def traverse_and_count_island(row, column):


calculate は数式を用いて計算するというニュアンスを感じます。今回の場合は get_area() あたりが良いと思います。

nodchip · 2025-06-16T10:13:22Z

lc695.md

+        def inside_island(row, column):
+            return 0 <= row < num_rows and 0 <= column < num_columns
+
+        def traverse_and_count_island(row, column):


count_island だと、面積ではなく島の数を数えているように感じられました。

update

3d33d88

potrue reviewed May 23, 2025

View reviewed changes

oda reviewed May 23, 2025

View reviewed changes

tokuhirat reviewed May 24, 2025

View reviewed changes

shintaro1993 reviewed May 26, 2025

View reviewed changes

quinn-sasha reviewed Jun 14, 2025

View reviewed changes

nodchip reviewed Jun 16, 2025

View reviewed changes

695. Max Area of Island #17

Are you sure you want to change the base?

695. Max Area of Island #17

Uh oh!

Conversation

rinost081 commented May 23, 2025

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!

Uh oh!