说的有2点
第一点
orderpizza里面有create pizza,这样违背了开放-关闭模式,也就是要加一个新的Pizza或者删除一个pizza时候,orderpizza里面的代码都要修改,所以可以把里面的代码搬到另外一个地方,也就是在创建一个simplefactory类来create pizza
第二点
pizza类底下有好多种类的pizza,如ypizza,xpizza,mpizza。。。这样会导致多个类依赖一个类,违背了依赖倒置原则,可以定义一个抽象类,然后让所有的类去实现这个抽象类的接口,总的来说是不要让类派生自具体类