Open kerbalwzy opened 5 years ago
在utils模块下添加了addr_check模块, addr_check模块包含以下几个函数:
def _host_addresses() -> set; 获取本机的所有有效IPV4地址 def ip_validate(ip: str) -> bool; 检查IP的有效性 def _port_is_free(port: int) -> bool; 检查端口是否被占用 def port_validate(port: int) -> bool; 检查端口的有效性
并且简单写了个单元测试 ⚠️:运行单元测试前要去掉 “函数注解”, 否则会引发语法错误 😁我只新增了这个模块, 但是没有改动你的其他任何代码 如果你采纳了本次提交, 关于项目中的 # TODO 验证addr有效性等位置的代码请有您自己修改 如果你在CodeReview中发现问题, 请回复告知我.
Coverage decreased (-1.05%) to 94.667% when pulling 4e618d091544903e705989a0ca4b27e8ba8cc117 on kerbalwzy:master into b3dec6792b2d701597b45bb37fb2f3f73e11293e on linw1995:master.
添加这四个函数有什么用呢?
😂不是你在源码里面留了检查IP和端口的TODO么
太久忘了😂。能不能在这个 pr 加上去呢?
在utils模块下添加了addr_check模块, addr_check模块包含以下几个函数:
def _host_addresses() -> set; 获取本机的所有有效IPV4地址 def ip_validate(ip: str) -> bool; 检查IP的有效性 def _port_is_free(port: int) -> bool; 检查端口是否被占用 def port_validate(port: int) -> bool; 检查端口的有效性
并且简单写了个单元测试 ⚠️:运行单元测试前要去掉 “函数注解”, 否则会引发语法错误 😁我只新增了这个模块, 但是没有改动你的其他任何代码 如果你采纳了本次提交, 关于项目中的 # TODO 验证addr有效性等位置的代码请有您自己修改 如果你在CodeReview中发现问题, 请回复告知我.