Python-сообщество

Доброго вечера!
Допустим есть 2 класса:
-класс коллекционер
-класс машина
Коллекционер может иметь много машин, машина имеет только одного коллекционера.
Прошу простить, если “это” больно читать.
Коллекционер:

from Car import *
from Exceptions import *

		
class Collectioner(object):
	def __init__(self, name, car=None):
		self.__dict__['cars'] = [car]
		self.__dict__['name'] = name

	def addCar(self, car):
		if type(car) != Car:
			raise SomeException()
		if car.owner != self:
			raise SomeException()
		self.__dict__['cars'].append(car)

	def removeCar(self, oldCar):
		if type(oldCar) != Car:
			raise SomeException
		if oldCar.owner == self:
			oldCar.changeOwner(None)
			self.__dict__['cars'].remove(oldCar)

from Collectioner import * 
from Exceptions import *
		
class Car(object):
	def __init__(self,collectioner,model):
		if type(collectioner) == Collectioner:
			self.__dict__['owner'] = collectioner
			self.__dict__['model'] = model
	
	@property 
	def owner(self):
		return self.__dict__['owner']
		

	def changeOwner(self, newOwner):
		if newOwner == None:
			self.__del__(self)
			return
		if type(newOwner) != Collectioner:
			raise SomeException()
		self.__dict__['owner'] = newOwner

Отредактировано Tropick (Фев. 4, 2014 15:52:50)

Укажите питон который используете.

self.__dict__['owner'] = collectioner
self.owner = collectioner

from Car import *

if type(newOwner) != Collectioner:

Циклические импорты зобороть можно вынеся общий код в отдельный модуль. Если хотит чтобы была связь двусторонняя то все что относится к Колллеционеру в Авто и в Авто к Коллекционеру выделяте в отдельный класс.

Да и НИКОГДА не используйте wild import огребете с именами.

PanovSergey
Циклические импорты зобороть

нужно загружать так

import Car

Car.Car

doza_and
проверка типа - архитектурный недостаток

doza_and
Просто это некрасиво

Отредактировано sergeek (Фев. 4, 2014 19:06:30)

нужно загружать так

import Car

и обращаться к классу через точку (не надо называть модуль с заглавной буквы)

Car.Car

from car import Car

Отредактировано PanovSergey (Фев. 4, 2014 19:15:02)

PanovSergey

from car import Car

Отредактировано sergeek (Фев. 4, 2014 19:28:10)

sergeek
ну так у ТСа все равно будет программа падать от рекурсивного импорта.

PanovSergey
Выше написано как победить.

PanovSergey
Циклические импорты зобороть можно вынеся общий код в отдельный модуль. Если хотит чтобы была связь двусторонняя то все что относится к Колллеционеру в Авто и в Авто к Коллекционеру выделяте в отдельный класс.

там общий код в методах, по-нормальному их никак не вынести в отдельный модуль