Question

我只是在工作时写这种东西，想知道是否有任何pythonic的方法来减少与Item.display（）中的列表，迭代器和集合处理有关的无关紧要的东西。

在Item.display（）中还有更多的Python方式来处理过滤和转换吗？

是否可以通过某种方式使用其他类型来简化打字？

import unittest
from dataclasses import dataclass, field
from typing import Union, Tuple, List, Iterator

@dataclass
class Category:
    id: str
    name: str = field(compare=False)
    canonical: Union['Category', None] = field(compare=False)

    def __hash__(self)->int:
        return self.id.__hash__()

    def unalias(self)->'Category':
        #There could be more tests here
        if self.canonical:
            return self.canonical
        else:
            return self

    def display_str(self)->str:
        if self.id and self.name:
            return f'{self.id} [{self.name}]'
        else:
            return self.id


@dataclass
class Item:
    id: str
    categories: List[Category]


    def display(self,primary:Category)-> List[str]:
        """Unalias, dedup and sort for display."""
        if not self.categories:
            return []

        def unalias(secs:Iterator[Category])->Iterator[Category]:
            return map(lambda c: c.unalias(), secs) 

        def de_prim(secs:Iterator[Category])->Iterator[Category]:
            return filter(lambda c: c.id != primary.id, secs)

        def to_display(secs:List[Category])->List[str] :  
            return list(map(lambda c: c.display_str(), secs) )

        de_primaried = set(de_prim(unalias(iter(self.categories))))
        if not de_primaried:
            return []    
        return sorted(to_display(de_primaried))


class DoiTest(unittest.TestCase):
    def test(self):
        primary = Category('shirt', 'Shirts', None)

        cats = [Category('snow','Snowflakes',
                         Category('frozen_water','Frozen Water',None) ),
                Category('t-shirt','Tee Shirts', primary),
                Category('cheese','Yummy Cheese',None)]
        item = Item( 'ItemX', cats)
        self.assertEqual( item.display(primary) , ['cheese [Yummy Cheese]', 'frozen_water [Frozen Water]'])

Answer 1

剥离出mypy位，看来这实际上是您在做什么？

def display(self, primary):
    return sorted(
        map(
            lambda c: c.display_str(), 
            set(
                filter(
                    lambda c: c.id != primary.id, 
                    map(
                        lambda c: c.unalias(), 
                        self.categories
                    )
                )
            )
        )
    )

除非您出于某些原因想要categories拥有值None，否则我将其初始化为一个空列表。由于de_primaried始终是map的有效参数，即使它为空，也无需检查它并手动返回一个空列表。

您可能要考虑在几个嵌套循环中构建返回值以提高可读性（我认为，其中的一次性功能绝对是行不通的，特别是考虑到额外的类型混乱）。另外，我很有可能遗漏了一些东西，但是其中似乎有一些不必要的list()和iter()调用。

在处理列表和使用mypy时如何减少过时性？

1 个答案: