高階関数は積極的に使うべきか？

Question

mapやreduce等の高階関数についての質問です。

個人でプログラミングを学んでおりますが、まだ組織に属してコードを書いた経験はありません。
ゆくゆくは会社に所属し、コードを書きたいと思い学んでいます。

「良いコード」と一口に言っても様々な価値観があると思うのですが、
明示的に何をしているのかを説明した方がよいのか？
用意されているメソッドは、開発者全員、既知であることを前提に最小限のコードを書けばよいのか？

以下はシンプルな例ですが、どちらが組織で働く際に好まれるのでしょうか？
ご意見頂けると幸いです。よろしくお願いします。

class Person {
    var name: String?
    var age: Int?
}

var john = Person()
john.name = "john"
john.age = 30

var alex = Person()
alex.name = "Alex"
alex.age = 20

let Persons = [john,alex]

//Pattern A
var namesA:[String] = []
for i in Persons {
    namesA.append(i.name!)
}

//Pattern B
var namesB:[String] = []
map(Persons,{namesB.append($0.name!)})

Answer 1

その組織次第です。

プラットフォームの制約でSwiftを使わざるを得ないが、開発に携わるのはCプログラマばかりというような情況では、Swiftの機能を制限せざるを得ないでしょう。
Rubyプログラマばかりであれば、高階関数のような機能を積極的に使っていくのでしょう。
Swiftの例で書きましたが、「XXX言語で開発せざるを得ないがYYY言語プログラマが多い」というのは珍しいことではありません。

そういう人の条件を外して考えると、定型的な処理は専用の関数を使った方がその部分でバグが出なくて済むのでそちらの方が良いです。

Comments

[umineko]

回答ありがとうございます。
「XXX言語で開発せざるを得ないがYYY言語プログラマが多い」という視点は、とても納得感のある重要な視点だなと感じました。

Answer 2

PatternAとPatternBのどちらが好まれるのか？ということなら、間違いなくPatternAの方です。
とりあえず、map関数の使い方を間違えています。
PatternBのような書き方は誰も好まないでしょう。
正しくは、以下のように書きます。

var namesB = map(Persons, {$0.name!})

おそらく、その組織としてSwift言語を選択した時点で、Swift言語の流儀にしたがってコードを書くことが了承されているのだと思います。
もし、C言語の流儀でコードを書きたいならObjective-Cを使えばいいのですから。

Comments

[umineko]

回答ありがとうございます。
言語の流儀というのはあまり意識した事はありませんでした。いろいろな言語ごとの流儀を調べて勉強してみます。間違いを指摘して頂けるととても助かります。