Qiita 7 - キータぞ、来たぞ、キータだぞー

2025/09/13(土) 12:24:07.83ID:mucntwOq
Hello hackers !
Qiitaは、エンジニアリングに関する知識を記録・共有するためのサービスです。
コードを書いていて気づいたことや、自分がハマったあの仕様について、
他のエンジニアと知見を共有しましょう ;)

https://qiita.com/

Qiita(キータ)は、Incrementsが運営するプログラミング情報のナレッジコミュニティ。
2016年現在で日本最大のプログラマーコミュニティとされている[1]。

https://internet.watch.impress.co.jp/docs/news/1025972.html

前スレ Qiita
https://mevius.5ch.net/test/read.cgi/tech/1542357242/
Qiita 2 - キータぞ、来たぞ、キータだぞー
https://mevius.5ch.net/test/read.cgi/tech/1658762410/
Qiita 3 - キータぞ、来たぞ、キータだぞー
https://mevius.5ch.net/test/read.cgi/tech/1685235361/
Qiita 4 - キータぞ、来たぞ、キータだぞー
https://mevius.5ch.net/test/read.cgi/tech/1705486836/
Qiita 5 - キータぞ、来たぞ、キータだぞー
https://mevius.5ch.net/test/read.cgi/tech/1717651046/
Qiita 6 - キータぞ、来たぞ、キータだぞー
https://mevius.5ch.net/test/read.cgi/tech/1739527246/
2025/09/14(日) 01:56:40.69ID:CZ0V8fQ4
『【Go】配列を再帰的に逆順にするサンプルコードを書いてみた』
> 再帰を使うことで、ループを使わずにエレガントに実装できる。
 
根本的なところで誤解してる人な模様。
2025/09/14(日) 02:53:44.41ID:bGojT+ur
>>2
一般論として
再帰で表現した方が抽象度が高く理解しやすく定義そのまま表現できることが多い
特に末尾再帰をコンパイラがループへ変換してくれて実行効率が同等ならば全ての点で再帰による表記が勝る
2025/09/14(日) 12:10:16.28ID:RhzWmJy7
入力配列と出力配列渡してループで処理するわ
2025/09/14(日) 12:59:52.05ID:ZqIkDajJ
記事の再帰版のコードは

> func ReverseArray(arr []int, start, end int) []int {
>   // ベースケース: 配列の中央に到達したら終了
>   if start >= end {
>     return arr
>   }
>
>   // 要素を交換
>   arr[start], arr[end] = arr[end], arr[start]
>
>   // 再帰呼び出し: 範囲を狭めて継続
>   return ReverseArray(arr, start+1, end-1)
> }

引数で渡した配列の内容書き換えるんでreturn要らないんだよなあ。

再帰で書くよか繰り返しで書いた方が余程シンプルとも思う。

func ReverseArrayWithFor(arr []int, start, end int) {
  for start < end {
    arr[start], arr[end] = arr[end], arr[start]
    start += 1
    end  -= 1
  }
}
2025/09/14(日) 13:06:57.84ID:ZqIkDajJ
記事のスライス操作版のコードは

> // ReverseArrayWithSlice はスライス操作で配列を逆順にする(再帰)
> func ReverseArrayWithSlice(arr []int) []int {
>   // ベースケース: 要素が1つ以下なら逆順にする必要なし
>   if len(arr) <= 1 {
>     return arr
>   }
>
>   // 最初の要素を取り出し、残りを再帰的に逆順にしてから結合
>   return append(ReverseArrayWithSlice(arr[1:]), arr[0])
> }

引数の配列内容を書き換えないし再帰を説明する上では面白い例だと思うが、パフォーマンスは最低だ罠。
2025/09/14(日) 13:28:33.58ID:ZqIkDajJ
そもそもの話として、ひとつの記事の中で複数の関数が引数だとか配列の内容を書き換えるのかどうかとか仕様が合ってないから比較になってないのよね。
8デフォルトの名無しさん
垢版 |
2025/09/14(日) 13:42:25.65ID:yOrWt/NI
>>7
問題はそこだね
ループでも書けるよとかの筋違いな批判は要らんて
2025/09/14(日) 14:21:09.67ID:ZqIkDajJ
・引数は配列を渡すのみ
・内容を逆にした配列を返し、引数で渡した配列の内容は書き換えない
という条件にしたとして、繰り返しで書くより再帰のほうが良い書き方ができるだろうか?

func ReverseArrayWithFor(arr []int) []int {
  n := len(arr)
  result := make([]int, n)

  for i := 0; i < n; i++ {
    result[i] = arr[n - i - 1]
  }

  return result
}

「全ての点で再帰による表記が勝る」という人には再帰の素晴らしさをぜひ証明してほしいところだが口だけ番長だろうなあ。
2025/09/14(日) 15:20:41.57ID:wFXoVVHv
>>9
大昔からの古典すら知らない無知
LISPも知らないんだろうな
2025/09/14(日) 15:24:57.73ID:ZqIkDajJ
GoとLISPの区別もつかない人とはなあw
2025/09/14(日) 15:30:37.23ID:ZqIkDajJ
記事のスライス操作版のコードは

> // ReverseArrayWithSlice はスライス操作で配列を逆順にする(再帰)
> func ReverseArrayWithSlice(arr []int) []int {
>   // ベースケース: 要素が1つ以下なら逆順にする必要なし
>   if len(arr) <= 1 {
>     return arr
>   }
>
>   // 最初の要素を取り出し、残りを再帰的に逆順にしてから結合
>   return append(ReverseArrayWithSlice(arr[1:]), arr[0])
> }

要素数1の配列を渡した場合は引数の配列をそのまま返してしまうので

  // ベースケース: 要素が0以下なら逆順にする必要なし
  if len(arr) <= 0 {

とした方が良いな。
2025/09/14(日) 15:35:06.93ID:ZqIkDajJ
試しに
・引数は配列を渡すのみ
・内容を逆にした配列を返し、引数で渡した配列の内容は書き換えない
再帰で書いてみたが

func ReverseArray(arr []int) []int {
  n := len(arr)
  result := make([]int, n)
  copy(result, arr)
  var reverseArray func(int, int)
  reverseArray = func(start int, end int) {
    if start < end {
      result[start], result[end] = result[end], result[start]
      reverseArray(start + 1, end - 1)
    }
  }
  reverseArray(0, n - 1)
  return result
}

エレガントとは程遠いな。
ぜひLISP仕込みのエレガントな例をGoで書いて披露してほしいものだ。
レスを投稿する

5ちゃんねるの広告が気に入らない場合は、こちらをクリックしてください。

ニューススポーツなんでも実況