Fixed PEP-8 voilations.

* add python headers. * Remove extra whitespaces.

Author: theskumar

README.md

@@ -1,35 +1,35 @@
 python-patterns
 ===============
 
-A collection of design patterns implemented (by other people) in python
+A collection of design patterns implemented (by other people) in python.
 
 Current Patterns:
 
-* 3-tier
-* composite
+* 3-tier
+* composite
 * mvc
-* decorator
+* decorator
 * null
-* facade
+* facade
 * observer
-* abstract_factory
-* factory_method
+* abstract_factory
+* factory_method
 * pool
-* adapter
-* flyweight
+* adapter
+* flyweight
 * prototype
-* borg
+* borg
 * proxy
-* bridge
-* graph_search
+* bridge
+* graph_search
 * state
-* builder
-* iterator
+* builder
+* iterator
 * strategy
-* chain
-* mediator
+* chain
+* mediator
 * template
-* command
-* memento
+* command
+* memento
 * visitor
-* publish_subscribe
+* publish_subscribe

abstract_factory.py

@@ -1,3 +1,6 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
 # http://ginstrom.com/scribbles/2007/10/08/design-patterns-python-style/
 
 """Implementation of the abstract factory pattern"""

adapter.py

@@ -1,4 +1,7 @@
-# http://ginstrom.com/scribbles/2008/11/06/generic-adapter-class-in-python/
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+"""http://ginstrom.com/scribbles/2008/11/06/generic-adapter-class-in-python/"""
 
 importos
 

borg.py

@@ -1,3 +1,6 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
 classBorg:
 __shared_state={}
 

bridge.py

@@ -1,4 +1,7 @@
-# http://en.wikibooks.org/wiki/Computer_Science_Design_Patterns/Bridge_Pattern#Python
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+"""http://en.wikibooks.org/wiki/Computer_Science_Design_Patterns/Bridge_Pattern#Python"""
 
 
 # ConcreteImplementor 1/2
@@ -20,11 +23,11 @@ def __init__(self, x, y, radius, drawing_api):
 self._y=y
 self._radius=radius
 self._drawing_api=drawing_api
-
+
 # low-level i.e. Implementation specific
 defdraw(self):
 self._drawing_api.draw_circle(self._x, self._y, self._radius)
-
+
 # high-level i.e. Abstraction specific
 defscale(self, pct):
 self._radius*=pct
@@ -35,11 +38,11 @@ def main():
 CircleShape(1, 2, 3, DrawingAPI1()),
 CircleShape(5, 7, 11, DrawingAPI2())
  )
-
+
 forshapeinshapes:
 shape.scale(2.5)
 shape.draw()
-
+
 
 if__name__=='__main__':
 main()

catalog.py

@@ -1,3 +1,6 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
 """
 A class that uses different static function depending of a parameter passed in init
 Note the use of a single dictionnary instead of multiple conditions

chain.py

@@ -1,4 +1,7 @@
-# http://www.testingperspective.com/wiki/doku.php/collaboration/chetan/designpatternsinpython/chain-of-responsibilitypattern
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+"""http://www.testingperspective.com/wiki/doku.php/collaboration/chetan/designpatternsinpython/chain-of-responsibilitypattern"""
 
 
 classHandler:

command.py

@@ -1,3 +1,6 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
 importos
 
 
@@ -34,4 +37,4 @@ def main():
 cmd.undo()
 
 if__name__=="__main__":
-main()
+main()

composite.py

@@ -1,9 +1,12 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
 """
 A class which defines a composite object which can store
 hieararchical dictionaries with names.
 
 This class is same as a hiearchical dictionary, but it
-provides methods to add/access/modify children by name,
+provides methods to add/access/modify children by name,
 like a Composite.
 
 Created Anand B Pillai <[email protected]>
@@ -17,7 +20,7 @@
 defnormalize(val):
 """ Normalize a string so that it can be used as an attribute
  to a Python object """
-
+
 ifval.find('-') !=-1:
 val=val.replace('-', '_')
 
@@ -65,7 +68,7 @@ def __setattr__(self, name, value):
 else:
 # New attribute
 self[name] =value
-
+
 
 classCompositeDict(SpecialDict):
 """ A class which works like a hierarchical dictionary.
@@ -106,15 +109,15 @@ def __getattr__(self, name):
 attr=getattr(self[self._name], name)
 ifattr:
 returnattr
-
+
 raiseAttributeError('no attribute named %s'%name)
 
 defisRoot(self):
 """ Return whether I am a root component or not """
 
 # If I don't have a parent, I am root
 returnnotself._father
-
+
 defisLeaf(self):
 """ Return whether I am a leaf component or not """
 
@@ -128,15 +131,15 @@ def getName(self):
 
 defgetIndex(self, child):
 """ Return the index of the child ConfigInfo object 'child' """
-
+
 ifchildinself._children:
 returnself._children.index(child)
 else:
 return-1
 
 defgetDict(self):
 """ Return the contained dictionary """
-
+
 returnself[self._name]
 
 defgetProperty(self, child, key):
@@ -156,25 +159,25 @@ def setProperty(self, child, key, value):
 childDict=self.getInfoDict(child)
 ifchildDict:
 childDict[key] =value
-
+
 defgetChildren(self):
 """ Return the list of immediate children of this object """
-
+
 returnself._children
 
 defgetAllChildren(self):
 """ Return the list of all children of this object """
-
+
 l= []
 forchildinself._children:
 l.append(child)
 l.extend(child.getAllChildren())
-
+
 returnl
 
 defgetChild(self, name):
 """ Return the immediate child object with the given name """
-
+
 forchildinself._children:
 ifchild.getName() ==name:
 returnchild
@@ -185,7 +188,7 @@ def findChild(self, name):
 # Note - this returns the first child of the given name
 # any other children with similar names down the tree
 # is not considered.
-
+
 forchildinself.getAllChildren():
 ifchild.getName() ==name:
 returnchild
@@ -195,33 +198,33 @@ def findChildren(self, name):
 
 # Note: this returns a list of all the children of a given
 # name, irrespective of the depth of look-up.
-
+
 children= []
-
+
 forchildinself.getAllChildren():
 ifchild.getName() ==name:
 children.append(child)
 
 returnchildren
-
+
 defgetPropertyDict(self):
 """ Return the property dictionary """
-
+
 d=self.getChild('__properties')
 ifd:
 returnd.getDict()
 else:
 return{}
-
+
 defgetParent(self):
 """ Return the person who created me """
 
 returnself._father
-
+
 def__setChildDict(self, child):
 """ Private method to set the dictionary of the child
  object 'child' in the internal dictionary """
-
+
 d=self[self._name]
 d[child.getName()] =child.getDict()
 
@@ -236,25 +239,25 @@ def setParent(self, father):
 # child is orphaned - see addChild and addChild2
 # methods !
 self._father=father
-
+
 defsetName(self, name):
-""" Set the name of this ConfigInfo object to 'name' """
+""" Set the name of this ConfigInfo object to 'name' """
 
 self._name=name
 
 defsetDict(self, d):
 """ Set the contained dictionary """
-
+
 self[self._name] =d.copy()
-
+
 defsetAttribute(self, name, value):
 """ Set a name value pair in the contained dictionary """
-
+
 self[self._name][name] =value
 
 defgetAttribute(self, name):
 """ Return value of an attribute from the contained dictionary """
-
+
 returnself[self._name][name]
 
 defaddChild(self, name, force=False):
@@ -264,10 +267,10 @@ def addChild(self, name, force=False):
 
  This function returns the child object, whether
  new or existing """
-
+
 iftype(name) !=str:
 raiseValueError('Argument should be a string!')
-
+
 child=self.getChild(name)
 ifchild:
 # print 'Child %s present!' % name
@@ -278,22 +281,22 @@ def addChild(self, name, force=False):
 child=self.__class__(name)
 self._children[index] =child
 child.setParent(self)
-
+
 self.__setChildDict(child)
 returnchild
 else:
 child=self.__class__(name)
 child.setParent(self)
-
+
 self._children.append(child)
 self.__setChildDict(child)
 
 returnchild
-
+
 defaddChild2(self, child):
 """ Add the child object 'child'. If it is already present,
  it is overwritten by default """
-
+
 currChild=self.getChild(child.getName())
 ifcurrChild:
 index=self.getIndex(currChild)
@@ -303,10 +306,10 @@ def addChild2(self, child):
 # Unset the existing child's parent
 currChild.setParent(None)
 delcurrChild
-
+
 self.__setChildDict(child)
 else:
-child.setParent(self)
+child.setParent(self)
 self._children.append(child)
 self.__setChildDict(child)
 
@@ -316,7 +319,7 @@ def addChild2(self, child):
 frame=window.addChild('Frame')
 tfield=frame.addChild('Text Field')
 tfield.setAttribute('size', '20')
-
+
 btn=frame.addChild('Button1')
 btn.setAttribute('label', 'Submit')
 

decorator.py

@@ -1,4 +1,7 @@
-# http://stackoverflow.com/questions/3118929/implementing-the-decorator-pattern-in-python
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+"""http://stackoverflow.com/questions/3118929/implementing-the-decorator-pattern-in-python"""
 
 
 classfoo_decorator(object):