refactor: refactor unnecessary else / elif when if block has a …

…`raise` statement (#37)

`raise` causes control flow to be disrupted, as it will exit the block.
It is recommended to check other conditions using another `if` statement, and get rid of `else` statements as they are unnecessary.

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

src/plugins/nonebot_plugin_heweather/weather_data.py

@@ -84,11 +84,10 @@ async def _get_city_id(self, api_type: str = "lookup"):
         logger.debug(res)
         if res["code"] == "404":
             raise CityNotFoundError()
-        elif res["code"] != "200":
+        if res["code"] != "200":
             raise APIError(f'错误! 错误代码: {res["code"]}{self.__reference}')
-        else:
-            self.city_name = res["location"][0]["name"]
-            return res["location"][0]["id"]
+        self.city_name = res["location"][0]["name"]
+        return res["location"][0]["id"]
 
     def _data_validate(self):
         if self.now.code != "200" or self.daily.code != "200":