意図しない繰り返しをなくしたい

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

下記のプログラムを実行すると、SolveModelの部分を三度繰り返してしまいます。
クラス化する前は問題なく動いていたのですが…

該当のソースコード

N = 2.0 #確保すべき非重複経路の本数
start_node = "hoge"
goal_node = "hogehoge"

class MinPath():
    def __init__(self):
        self.list_base = []
        self.list_ward = []
        self.list_node = []
        self.list_link = []
        self.my_vars   = []
        self.my_obj    = []
        self.my_types  = []
        self.my_senses = []
        self.my_rhs    = []
        self.my_lhs    = []
        self.my_cons   = []
        self.list_solution  = []

#### 道路と頂点のデータを読み込む
    def ReadCsv(self):
        with open("hogehogehoge.csv", encoding = "utf-8-sig") as f:
            reader = csv.reader(f)
    
            for row in reader:
                self.list_node.append(row[1])
                if "base" in row[0]:
                    self.list_base.append(row[1])
                elif "ward" in row[0]:
                    self.list_ward.append(row[1])
                else:
                    pass


        with open("hogehogehoge.csv", encoding = "utf-8-sig") as p:
            reader = csv.reader(p)
            header = next(reader)

            for row in reader:
                self.list_link.append([row[0], row[1], row[2], row[3], row[4]])  
                                     #[変数  , 始点  , 終点  , 距離  , 所属]



#### 線形計画問題の作成
    def CreateModel(self):
        list_start_in = []
        list_start_in_coeff = []
        list_start_out = []
        list_start_out_coeff = []
        list_goal_in = []
        list_goal_in_coeff = []
        list_goal_out = []
        list_goal_out_coeff = []

## 目的関数の係数、変数名とタイプを追加
        for link in self.list_link:
            self.my_vars.append(link[0])
            self.my_types.append("B")
            self.my_obj.append(float(link[3]))


## 制約式の係数と変数、等号、右辺の値を追加
        i = 1
        for node in self.list_node:
## nodeがstart_nodeと同じ場合
            if node == start_node:
                for link in self.list_link:
                    if node == link[1]:
                        list_start_out.append(link[0])
                        list_start_out_coeff.append(1.0)
                    elif node == link[2]:
                        list_start_in.append(link[0])
                        list_start_in_coeff.append(1.0)
                    else:
                        pass

                self.my_lhs.append([list_start_in, list_start_in_coeff])
                self.my_rhs.append(0.0)
                self.my_senses.append("E")
                self.my_cons.append("c{}".format(i))
                i += 1

                self.my_lhs.append([list_start_out, list_start_out_coeff])
                self.my_rhs.append(N)
                self.my_senses.append("E")
                self.my_cons.append("c{}".format(i))
                i += 1


## nodeがgoal_nodeと同じ場合
            elif node == goal_node:
                for link in self.list_link:
                    if node == link[1]:
                        list_goal_out.append(link[0])
                        list_goal_out_coeff.append(1.0)
                    elif node == link[2]:
                        list_goal_in.append(link[0])
                        list_goal_in_coeff.append(1.0)
                    else:
                        pass

                self.my_lhs.append([list_goal_in, list_goal_in_coeff])
                self.my_rhs.append(N)
                self.my_senses.append("E")
                self.my_cons.append("c{}".format(i))
                i += 1

                self.my_lhs.append([list_goal_out, list_goal_out_coeff])
                self.my_rhs.append(0.0)
                self.my_senses.append("E")
                self.my_cons.append("c{}".format(i))
                i += 1


## nodeがstart_nodeでもgoal_nodeでもない場合
            else:
                list_vars = []
                list_coeff = []
                for link in self.list_link:
                    if node == link[1]:
                        list_vars.append(link[0])
                        list_coeff.append(-1.0)
                    elif node == link[2]:
                        list_vars.append(link[0])
                        list_coeff.append(1.0)
                    else:
                        pass

                self.my_lhs.append([list_vars, list_coeff])
                self.my_rhs.append(0.0)
                self.my_senses.append("E")
                self.my_cons.append("c{}".format(i))
                i += 1



### 線形計画問題を解く
    def SolveModel(self):
        prob = cplex.Cplex()
        prob.set_results_stream(None)
        prob.objective.set_sense(prob.objective.sense.minimize)
        prob.variables.add(obj = self.my_obj, types = self.my_types, names = self.my_vars)
        prob.linear_constraints.add(lin_expr = self.my_lhs, senses = self.my_senses, rhs = self.my_rhs, names = self.my_cons)
        prob.solve()

        numrows = prob.linear_constraints.get_num()
        numcols = prob.variables.get_num()
    
        if prob.solution.status[prob.solution.get_status()] == "MIP_infeasible":
            print("\n！！！！！実行可能解がないか、エラーがあります！！！！！")
            print("！！！！！非重複経路数の条件を緩和してください！！！！！")
            sys.exit()
        else:
            vname = prob.variables.get_names()
            x = prob.solution.get_values()
            for j in range(numcols):
                if x[j] == 0:
                    pass
                else:
                    for link in self.list_link:
                        if vname[j] == link[0]:
                            self.list_solution.append(link)
                        else:
                            pass

        return prob.solution.status[prob.solution.get_status()], prob.solution.get_objective_value()


### 解を表示する
    def print_ans(self):
        if self.SolveModel()[0] == "MIP_infeasible":
            pass
        else:
            print("始点：{}、終点：{}、非重複経路数{}本".format(start_node, goal_node, N))
            print("総延長は", self.SolveModel()[1], "kmです。\n")

            for link in self.list_solution:
                print(link[4], ":", link[1], "から", link[2], "({}km)".format(link[3]))



a = MinPath()
a.ReadCsv()
a.CreateModel()
a.SolveModel()
a.print_ans()

試したこと

プログラム全体を確認していくとどうやらSolveModelの部分に問題がありそうということが分かりました。

補足情報（FW/ツールのバージョンなど）

Answer 1

print_ans関数の中でself.SolveModel()[0]のようにSolveModel関数を呼び出しているところが２か所ありa.CreateModel()と合わせて３回呼ばれています。
a.SolveModel()の戻り値を変数に代入して、それをprint_ansの引数として渡すように処理を見直してはいかがでしょうか？
概念修正例

# 略
ret = a.SolveModel()
a.print_ans(ret)

Comments

[syc98]

おかげさまでうまく動作するようになりました！
ありがとうございました！