Tengo el siguiente código con múltiples casos:
def _extract_property_value(selector: Selector) -> str: raw_value = selector.xpath("span[2]") default_value = raw_value.xpath("./text()").get().strip() value_with_a = ', '.join([value.strip() for value in raw_value.xpath("./a /text()").getall()]) value_with_div_and_a = ', '.join([value.strip() for value in raw_value.xpath("./div /a /text()").getall()]) if value_with_a: return value_with_a elif value_with_div_and_a: return value_with_div_and_a elif default_value: return default_valueQuiero deshacerme de las declaraciones if y simplificar este código tanto como sea posible. No soy un buen desarrollador de Pyton. Sé que hay una "estrategia" de patrón y, a continuación, estaba tratando de implementar eso:
def _extract_property_value(selector: Selector) -> str: raw_value = selector.xpath("span[2]") values_dict = { 'default value': raw_value.xpath("./text()").get().strip(), 'value with a': ', '.join([value.strip() for value in raw_value.xpath("./a /text()").getall()]), 'value with div and a': ', '.join([value.strip() for value in raw_value.xpath("./div /a /text()").getall()]) } return [item for item in values_dict.values() if item != ''][0]pero... Ahora, cuando pienso en esto, fue una mala idea usar la estrategia allí. No estoy seguro. ¿Puede alguien ayudarme a simplificar ese código? O esas declaraciones if son simplemente necesarias.
Si bien puede intentar usar un patrón de estrategia, hay una forma mucho más sencilla de hacerlo. En lugar de la cadena if/else, simplemente puede hacer:
return value_with_a or value_with_div_and_a or default_valuePodemos reducir el número de sentencias if pero sin la ayuda de un diccionario.
El código en cuestión asigna incondicionalmente valores a 3 variables. Una vez hecho esto, se examinan esas variables para determinar cuál, si es que hay alguna, se devolverá a la persona que llama. Sin embargo, no hay dependencias entre esas variables. Por lo tanto, podrían ser procesados en orden de prioridad y, si se adquiere un valor adecuado, entonces podría ser devuelto inmediatamente, haciendo que el proceso sea considerablemente más eficiente.
def _extract_property_value(selector): def f1(rv): return rv.xpath("./text()").get().strip() def f2(rv): return ', '.join([value.strip() for value in rv.xpath("./a /text()").getall()]) def f3(rv): return ', '.join([value.strip() for value in rv.xpath("./div /a /text()").getall()]) raw_value = selector.xpath("span[2]") for func in [f2, f3, f1]: # note the priority order - default last if (v := func(raw_value)): return vLa función revisada devolverá implícitamente Ninguno si no se encuentran los valores adecuados. En este sentido, no es diferente al código original del OP.