【gRPC】gRPCサーバーを利用してAPIのうまい構成を実現したい

Question

いつもお世話になっております。
毎度、新しいことを初めては壁にぶち当たっております。
お知恵をお借りできればと思いますのでよろしくお願いします。

前提・実現したいこと

go言語でgRPCサーバーを使ってAPIを作成しているのですが、サンプルを試してみて動いたのですが複数作った場合の実装で本当にこれでいいのか？となっているのでご教授いただければと

発生している問題・試したこと

サンプルにあったprotoファイルを複製、改変し2つめのAPIを作成
1.importにて読み込み
2.関数を追加
3.サービスを起動

以下サンプル改変後のソースです

package main

import (
	"context"
	"log"
	"net"

	"google.golang.org/grpc"
	pb "google.golang.org/grpc/examples/helloworld/helloworld"
	pb2 "google.golang.org/grpc/examples/helloworld/helloworldTest"
	"google.golang.org/grpc/reflection"
)

const (
	port = ":50051"
)

// server is used to implement helloworld.GreeterServer.
type server struct{}

// SayHello implements helloworld.GreeterServer
func (s *server) SayHello(ctx context.Context, in *pb.HelloRequest) (*pb.HelloReply, error) {
	return &pb.HelloReply{Message: "Hello " + in.Name}, nil
}

// SayHello implements helloworld.GreeterServer
func (s *server) SayHelloTest(ctx context.Context, in *pb2.HelloRequest) (*pb2.HelloReply, error) {
	return &pb2.HelloReply{Message: "Sum is ", Sum: in.V + in.V1}, nil
}

func main() {
	lis, err := net.Listen("tcp", port)
	if err != nil {
		log.Fatalf("failed to listen: %v", err)
	}
	s := grpc.NewServer()
	pb.RegisterGreeterServer(s, &server{})

	pb2.RegisterGreeterServer(s, &server{})
	// Register reflection service on gRPC server.
	reflection.Register(s)
	if err := s.Serve(lis); err != nil {
		log.Fatalf("failed to serve: %v", err)
	}
}

protoファイルを作成、関数を追加するのは当然として1,2をAPIごとに行うのはどうかと思っています。
毎回、それぞれ別の名前でimportをしてサービスを立てる必要があるのでしょうか？
gRPC、Goともに初心者でして良い方法があるのかこうするしかないのか判断できていない状況です。
また、必要だったとしてももう少し人為的ミスの少なくなる記述の仕方があればご教授いただければと

何卒、よろしくお願いします。

Answer 1

冒頭に宣言しているのはサーバーではなくサービスの振る舞いです。サーバーと呼べるものはひとつしかなくて、複数のサービスがぶら下がっているわけなのでなにも無駄な事はありません。
もちろんimportしておくだけでサービスをデフォルトサーバーにぶら下げるという事はできなくはありませんが、サーバーインスタンスを隠蔽してやるべき事を別のところに書くだけです。
僕は今の書き方の方が良いという考えです。
なぜならサーバーは運用のどこかでカスタマイズがしたくなるからです。タイムアウト値やロギングなどなど。
メインパッケージがいろんな依存を取りまとめる方がより明示的で追いやすくて良いと思います。

Comments

[tanamochi]

回答ありがとうございます！
確かにサービスですね、修正しましたm(_ _)m
せっかくお詳しい方が回答してくださったのに質問のしかたが悪かったです、すみません。
import、サービス追加をapiごとに追記すると人為ミスが起きそうだと思ったのでimportをまとめる、サービス起動をで外部で配列追加してまとめてforeachで行うなどができるといいなと思った次第です。
protoファイルにある程度まとめてapiを記述できるみたいなので自己解決の形にいたしました。

Answer 2

結果的に記述量はやや増えましたが機能ごとある程度ファイルを分割することができました！
APIごとにimportするimport pb123 "../../proto/***"が増えるのかと思っていましたが
protoファイルの中に複数のAPIを作れるというのがわかったので
importが数百とか書かなくていいということがわかりこの形になりました
もっとこうしたらいい、ここが間違っているなどあればツッコミお願いしますm(_ _)m

フォルダ構成

|--cmd
|  |--main.go
|--conf
|  |--config.toml
|--proto
|  |--hoge
|  |  |--hoge.pb.go
|  |  |--hoge.proto
|  |--foo
|  |  |--foo.pb.go
|  |  |--foo.proto
|--service
|  |--hoge.go
|  |--foo.go
|--sys
|  |--gRPC
|  |  |--gRPC.go

ソース
service/hoge.go

package service

import (
	"context"

	pb "../proto/hoge"
)

type HogeServer struct{}

func (s *HogeServer) Hoge(ctx context.Context, in *pb.HogeRequest) (*pb.HogeReply, error) {
	return &pb.HogeReply{Message: "hoge"}, nil
}

service/foo.go

package service

import (
	"context"

	pb "../proto/foo"
)

type FooServer struct{}

func (s *FooServer) Foo(ctx context.Context, in *pb.FooRequest) (*pb.FooReply, error) {
	return &pb.FooReply{Message: "foo"}, nil
}

gRPC.go

package grpcservice

import (
	"log"
	"net"

	// proto直下にまとめてimportをひとつにしたかったが
	// 変換の際に内部で同名関数を生成しているため一階層掘ってわけた
	pb1 "../../proto/hoge"
	pb2 "../../proto/foo"
	service "../../service"

	"google.golang.org/grpc"
	"google.golang.org/grpc/reflection"
)

func StartService(port string) {
	lis, err := net.Listen("tcp", port)
	if err != nil {
		log.Fatalf("failed to listen: %v", err)
	}
	s := grpc.NewServer()
	pb1.RegisterHogeServer(s, &service.HogeServer{})
	pb2.RegisterFooServer(s, &service.FooServer{})

	// Register reflection service on gRPC server.
	reflection.Register(s)
	if err := s.Serve(lis); err != nil {
		log.Fatalf("failed to serve: %v", err)
	}
}

main.go

package main

import grpcservice "../sys/gRPC"

const (
	port = ":50051"
)

func main() {
	grpcservice.StartService(port)
}