From 2d533c804d691ada3712858426f0b559bf60038d Mon Sep 17 00:00:00 2001 From: schopenhauerzhang Date: Tue, 5 Nov 2019 15:47:05 +0800 Subject: [PATCH 1/2] change the shuffle_origin function ,simple is bettter --- pycaptain/client.py | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/pycaptain/client.py b/pycaptain/client.py index acaa17f..e61ece4 100644 --- a/pycaptain/client.py +++ b/pycaptain/client.py @@ -62,17 +62,11 @@ def origin(cls, host, port): def shuffle_origin(self): ''' refresh current origin + random find origin to current origin ''' total_probe = 0 - for origin in self.origins: - total_probe += origin.probe - rand_probe = random.randint(0, total_probe) - acc_probe = 0 - for origin in self.origins: - acc_probe += origin.probe - if acc_probe > rand_probe: - self.current_origin = origin - break + probe = [] + self.current_origin = random.choice(self.origins) @property def url_root(self): From bb0d43db5c2f661a352dc029360b4f713ba58241 Mon Sep 17 00:00:00 2001 From: schopenhauerzhang Date: Tue, 5 Nov 2019 15:51:40 +0800 Subject: [PATCH 2/2] delete redundancy code --- pycaptain/client.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/pycaptain/client.py b/pycaptain/client.py index e61ece4..276312c 100644 --- a/pycaptain/client.py +++ b/pycaptain/client.py @@ -64,8 +64,6 @@ def shuffle_origin(self): refresh current origin random find origin to current origin ''' - total_probe = 0 - probe = [] self.current_origin = random.choice(self.origins) @property