31. Next Permutation #67

hayashi-ay · 2024-03-26T16:06:55Z

nodchip · 2024-03-29T07:20:05Z

31. Next Permutation.md

+```python
+class Solution:
+    def nextPermutation(self, nums: List[int]) -> None:
+        def first_decreasing_index_from_right():


関数名は動詞で始めるとよいと思います。 find_first_decreasing_index_from_right または find_last_of_descreasing_num_index はいかがでしょうか？

nodchip · 2024-03-29T07:25:54Z

31. Next Permutation.md

+                    return i
+            return -1
+
+        def next_number_index(begin, lower):


next_number_index ですと、次の数字インデックスを返しているように感じます。 find_minimum_number_index_larger_than はいかがでしょうか？

nodchip · 2024-03-29T07:33:11Z

31. Next Permutation.md

+                index += 1
+            return len(nums) - 1
+
+        def swap_in_range(left, right):


swap_in_range ですと、何をスワップして、最終的にどのような状態にしようとしているのかが分かりにくく感じました。 reverse_in_range のままでよいと思います。

hayashi-ay added 3 commits March 27, 2024 01:06

Create 31. Next Permutation.md

a50e99e

Update 31. Next Permutation.md

fc0fba9

Update 31. Next Permutation.md

e6e3793

nodchip reviewed Mar 29, 2024

View reviewed changes

hayashi-ay added 2 commits April 28, 2024 15:19

Update 31. Next Permutation.md

fbdfad6

Update 31. Next Permutation.md

04ccdbb

rihib mentioned this pull request Aug 8, 2024

Next Permutation rihib/leetcode#12

Open

Provide feedback