CODEWITHSUNDEEP
pythonclassmayapublicmembers
Mahmoud Youssef
Mahmoud Youssef

Reputation: 23

Python Class And public Members

Hello i wont to create a class with multiple function each function i need to create its own public Members so i did this but it gives me an error 

import maya.cmds as cmds

class creatingShadingNode():

    def _FileTexture( self, name = 'new' , path = '' , place2dT = None ):

        # craeting file texture

        mapping = [

              ['coverage', 'coverage'],
              ['translateFrame', 'translateFrame'],
              ['rotateFrame', 'rotateFrame'],
              ['mirrorU', 'mirrorU'],
              ['mirrorV', 'mirrorV']

              ]

        file = cmds.shadingNode ( 'file' , asTexture = 1 , isColorManaged = 1 , n = name + '_file' )

        if not place2dT:
            place2dT = cmds.shadingNode ( 'place2dTexture' , asUtility = 1 , n = name + '_p2d' )

        for con in mapping:

            cmds.connectAttr( place2dT + '.' + con[0] , file + '.' + con[1] , f = 1 )

        if path:
            cmds.setAttr( file + '.fileTextureName' , path, type = 'string' )

        self.File = file
        self.P2d = place2dT

test  = creatingShadingNode()._FileTexture(name = 'test' , path = 'test\test' )
print test.File

i get line 1: 'NoneType' object has no attribute 'File'

Upvotes: 0

Views: 131

Answers (1)

theodox
theodox

Reputation: 12218

Two things:

First, you're not returning anything from _FileTexture() -- you're creating an instance and calling its method with no return. If the idea is to set instance members you want

instance = creatingShadingNode()
instance._FileTexture(name = 'test' , path = 'test\test' )
print instance.File

Second, you're not creating the class in the common Python manner. Most people would do it like this:

class ShadingNodeCreator(object):
      def __init__(self):
          self.file = None
          self.p2d = None

      def create_file(name, path, p2d):
          # your code here

Most of the difference is cosmetic, but you'll have an easier time if you use the Python conventions. Imheriting from object gives you a bunch of useful abilities, and it's a good idea to declare your instance variables in __init__ -- if nothing else it makes it obvious what the class may contain.

Upvotes: 2

Related Questions