casemap によるリファクタリング
ldap プロトコル (v3) は rfc 2251 で定義されていて属性へのアクセスは case-insensitive (大文字小文字を区別しない) に扱うという仕様になっている。
ldap サーバーへの問い合わせや検索は case-insensitive となるため、ldap エントリーを扱う他システムの処理もすべて case-insensitive にしないと整合性が取れなくて混乱する。これは利用者だけでなく開発者も同様であり、一部が case-insensitive なのに、一部を case-sensitive (大文字小文字を区別する) にすると、比較処理がマッチしたりしなかったりという混乱を生じる。そういう面倒くさい issue が過去にいくつか散見されてきた。この問題は場当たり的な修正ではなく、本質的な対応が求められた。
結論としては、すべてを case-insensitive に扱う必要があって、そのためにキーを case-insensitive として扱う map like なデータ構造を実装した。オリジナルのキー情報も保持しつつ、case-insensitive にアクセスできるよう小文字変換しておいた名前変換テーブルを内部にもつ。最初からこういったデータ構造を定義しておけば開発時に混乱は起きないが、こういうものは後になってわかってくる。うちは1年半経ってからこの問題の本質を理解した。そのため、関連する処理のインターフェースを見直して置き換えることになった。こういうリファクタリングは時間がかかる割に成果も地味なので労力に対して評価されないことが多い。私もやっていてあまり楽しくはないが、こういうところをきっちり作ると品質に寄与するのを経験的に知っている。
type CaseMap struct {
keyValue map[string][]string
nameConv map[string]string
}
func (m *CaseMap) String() string {
return fmt.Sprintf("%v", m.keyValue)
}
func (m *CaseMap) Values(name string) []string {
origName, ok := m.nameConv[strings.ToLower(name)]
if !ok {
return nil
}
values, ok := m.keyValue[origName]
if !ok {
return nil
}
return values
}
func (m *CaseMap) Get(name string) ([]string, bool) {
origName, ok := m.nameConv[strings.ToLower(name)]
if !ok {
return nil, false
}
values, ok := m.keyValue[origName]
return values, ok
}
func (m *CaseMap) Set(name string, values []string) {
m.keyValue[name] = values
m.nameConv[strings.ToLower(name)] = name
}
func (m *CaseMap) Len() int {
return len(m.keyValue)
}
func (m *CaseMap) Iter() iter.Seq2[string, []string] {
return func(yield func(string, []string) bool) {
for k, v := range m.keyValue {
if !yield(k, v) {
return
}
}
}
}
type CaseName struct {
Original string
LowerName string
}
func (m *CaseMap) IterWithLower() iter.Seq2[CaseName, []string] {
return func(yield func(CaseName, []string) bool) {
for lowerName, origName := range m.nameConv {
name := CaseName{
Original: origName,
LowerName: lowerName,
}
if !yield(name, m.keyValue[origName]) {
return
}
}
}
}
func (m *CaseMap) ToMap() map[string][]string {
return m.keyValue
}
func New(length int) *CaseMap {
return &CaseMap{
keyValue: make(map[string][]string, length),
nameConv: make(map[string]string, length),
}
}
func NewFrom(attr map[string][]string) *CaseMap {
m := New(len(attr))
for k, v := range attr {
m.Set(k, v)
}
return m
}
