[Feature]: Rethinking package classes design pattern.

[wiseaidev]

Description

Hey @nidhaloff, I have a question related to the design pattern followed in this class. For me it looks redundant. Take a look at its usage in the tests:

b-rabbit/tests/test_b_rabbit.py

Lines 10 to 11 in f71ed6e

	def rabbit():
	return BRabbit()

In order to create an EventPublisher, you have to call the inner class and pass the outer reference, BRabbit, as a parameter to the __init__ function:

b-rabbit/tests/test_b_rabbit.py

Line 23 in f71ed6e

publisher = rabbit.EventPublisher(b_rabbit=rabbit,

It kind of looks unnecessary to do so since you already have the outer reference, the BRabbit instance. It looks like a self-reference problem or something of that nature. Also, I think this way it will allocate unnecessary space in memory.

Solution

All Inner classes should be moved outside the BRabbit class for better design.

[nidhaloff]

@wiseaidev Good idea. Unfortunately, I can't remember the reason for this design. It has been years since I worked on this. Would you be interested to contribute your ideas? I can add you as a direct contributor to this repo in order to give you more freedom.

[wiseaidev]

Yup!

[nidhaloff]

@wiseaidev Done!