chore: use f'string for all and check None instead empty list(because…

… those are the same)

src/check_phat_nguoi/get_data/engines/phat_nguoi.py

@@ -31,7 +31,7 @@ def __init__(self) -> None:
     def get_violations(html: str):
         soup = BeautifulSoup(html, "html.parser")
         violation_htmls: ResultSet[BeautifulSoup] = soup.find_all("tbody")
-        if violation_htmls == []:
+        if violation_htmls is None:
             return
         violation_detail_set: set[ViolationDetail] = set()
 
@@ -72,10 +72,8 @@ def _get_violation(violation_html: BeautifulSoup):
         return tuple(violation_detail_set)
 
     async def _request(self, plate_info: PlateInfo) -> str | None:
-        url = (
-            API_URL_PHATNGUOI
-            + plate_info.plate
-            + f"/{get_vehicle_enum(plate_info.type)}/"
+        url: str = (
+            f"{API_URL_PHATNGUOI}{plate_info.plate}/{get_vehicle_enum(plate_info.type)}"
         )
         try:
             async with self._session.get(url=url) as response: