golang のswitchで複数当てはまる場合の複数実行の方法を教えてください

Question

#環境
golang1.8.3
gogland

#質問

package main
import (
	"fmt"
	"regexp"
)

func main() {
    messageType := []string{"check", "on", "off"} //スライス作成

    re1 := regexp.MustCompile(messageType[0]) //checkをコンパイル
    re2 := regexp.MustCompile(messageType[1]) //onをコンパイル
    re3 := regexp.MustCompile(messageType[2]) //offをコンパイル

    message := "on and off" //on, offの2つが入っている文字列

    switch {
    case re1.MatchString(message): //messageにcheckが含まれていた場合
        fmt.Print("check")

    case re2.MatchString(message): //messageにonが含まれていた場合
        fmt.Print("on")

    case re3.MatchString(message): //messageにoffが含まれていた場合
        fmt.Print("off")

    default: //その他の時
        fmt.Print("I cant understand what you mean")
    }
}

このコードは、スライスの中身が変数messageに入っていたらcaseの中を処理するコードなのですが、messageの中に、2つ入っている場合、おそらくcaseで上に来ているonの中が処理された後にswitch構文から抜けると思います。

上記のように、caseの複数が該当する時に、二つの処理をさせるためにはどうすればいいですか？

#####関連した別件
上記のコードで、

re1 := regexp.MustCompile(messageType[0]) //checkをコンパイル
re2 := regexp.MustCompile(messageType[1]) //onをコンパイル
re3 := regexp.MustCompile(messageType[2]) //offをコンパイル

となっている部分なんですが、forを使って簡潔に書く方法があれば教えていただきたいです。

---

例えば以下のようにすると変数reが上書きされてしまうと思います。。
変数名にiを関連付ければなんとかなる気もしますが、スマートな方法があればと思います。。

i := 0
for _, i range messageType{
    re := regexp.MustCompile(messageType[i]) 
    if i == 3{ break }
    i++
}

Answer 1

要件を整理すると作らないといけない物が分かると思います。

• パターンAにマッチした場合はメッセージ a を表示する
• パターンBにマッチした場合はメッセージ b を表示する
• パターンCにマッチした場合はメッセージ c を表示する
• いずれにもマッチしなかった場合にはメッセージ d を表示する

この要件には少なくとも条件マッチが 4 ついるのが分かるかと思います。switch で書くと A または B または C またはそれ以外という記述になるため、switch は向いていない、というのが理解できるかと思います。それぞれ独立した条件なので「いずれか」を実現するのであればループで回すしかなく、かつどれにもマッチしなかった場合も処理を書く事になります。

以下、元のソースと大きく変わってしまいますが、僕であればこう書くというのを載せておきます。

package main

import (
	"fmt"
	"regexp"
)

var matchTypes = []struct {
	pattern string
	message string
	re      *regexp.Regexp
}{
	{
		pattern: "check",
		message: "check にマッチしました",
	},
	{
		pattern: "on",
		message: "on にマッチしました",
	},
	{
		pattern: "off",
		message: "off にマッチしました",
	},
}

func init() {
	for i := 0; i < len(matchTypes); i++ {
		matchTypes[i].re = regexp.MustCompile(matchTypes[i].pattern) //コンパイル
	}
}

func main() {
	source := "on and off" //on, offの2つが入っている文字列

	found := false
	for _, m := range matchTypes {
		if m.re.MatchString(source) {
			fmt.Println(m.message)
			found = true
		}
	}
	if !found {
		fmt.Print("I cant understand what you mean")
	}
}

この形の方が後からパターンを追加する場合等にも見通しが良く、またテストを作る際にも役立ちます。switch 文だとテストケース作るの大変ですからね。

Comments

[退会済みユーザー]

構造体を使うことでパターンと処理を紐づけてるんですね！
処理の部分を関数などにすると応用することができますね！
とても綺麗です。ありがとうございました！

[退会済みユーザー]

これ、messageの部分を即時関数などにしたい場合はどうすればいいですか？
message stringの部分の型を関数の場合どう表記すればいいのかわかりません

[mattn]

https://play.golang.org/p/N7G3DLHVNv

こうすればできます。ただ、テストとしては使いづらくなります。

Answer 2

caseの複数が該当する時に、二つの処理をさせるためにはどうすればいいですか？

こちらに関しては、switchで実現しないほうが簡単かもしれません。
switch(切り替える)という意味からも分かる通り、どれか一つの処理を実行させたい場合に向いています。

また、部分一致があるかないかを判定するときにMatchStringを使うのは最適ではないでしょう。
今回の場合はstrings.Contains()でいいかと思われます。

https://play.golang.org/p/EVmmV1f_Wl

func main() {
	messageType := []string{"check", "on", "off"} //スライス作成
	message := "on and off"                       //on, offの2つが入っている文字列

	contain := false
	for _, v := range messageType {
		if strings.Contains(message, v) {
			// メッセージに特定の文字が含まれていたら特定の文字を出力する
			fmt.Print(v, " ")
			contain = true
		}
	}
	if !contain {
		// 何も含まなければ見つからなかったことを表示して伝える
		fmt.Print("I cant understand what you mean")
	}
}

一応、正規表現版も
https://play.golang.org/p/2kjrZB_lqK

Comments

[退会済みユーザー]

ベストアンサーを決めた後に回答をくださったため気がつくことができませんでした。
初心者の僕でもわかりやすい回答だと感じました。
strings.Contains()という便利な関数があることも知りませんでした。
他にも他人のコードを読んでいると、メッセージごとの処理をファイルで分けてHandlerにセットして云々というようなものもありました。
わざわざありがとうございました。

Answer 3

message := "on and off"
msgFields := strings.Fields(message)

for _, v := range msgFields {
    switch {
    case re1.MatchString(v): 
        // 処理
    }
}

for文とswitch文を合わせて使えば、できるかと思います。

Comments

[mattn]

switch の意味がないですね。

[退会済みユーザー]

この場合if文を使っても同じ結果が出ると思いますが、私はswitchの方が読みやすいのでswitchを使いました。
if-else-if-elseの代わりに使うとコードが読みやすくなると思います。
https://golang.org/doc/effective_go.html#switch

[mattn]

switch が if の代わりに書けるのは知っています。(もう7～8年もgoを書いていますからね)
今回の要件は「パターンAにマッチしていればメッセージ a を表示する」という物なので、条件を書いたｔしても if-else-if-else にはならなく if-else しかないのです。switch で書くと case 1つと default になります。なお条件にマッチしてループを止めたい場合、switch の場合は labeled goto を使わないと抜けられなくなります。

このコードにはいずれの条件にもマッチしなかった場合、という物がありません。

[退会済みユーザー]

私が条件をちゃんと理解してなかったかもしれませんが、質問の内容は「caseの複数が該当する時に、二つの処理をさせるためにはどうすればいいですか？」と書いてあって、for文と条件文を合わせたコードを書きました。

Answer 4

fallthroughを使うとswitchから抜けません。

case re1.MatchString(message): //messageにcheckが含まれていた場合
        fmt.Print("check")
        fallthrough

Comments

[退会済みユーザー]

fallthroughを使うと、caseで条件分岐するのではなく、したのcase内部の処理を問答無用で行ってしまいます。(自分の環境ではそうなりました。)
全部実行したいのではなく、caseの条件がtrueの時だけ実行したいです。
switchをやめて大人しくifで条件分岐するしかないのでしょうか...

[kokardy]

各caseでtrueの時だけなら、大人しくというかifで書くのが正当だと思います。
十分スマートです。
条件が多かったらre1,re2,re3をスライスにして、
for i := 0 ; i < len(re) ; i++{
re[i] = MatchString(message)
}
としてもいいと思います。