Loop AddLayers - Help

[LorraineBarry]

I am trying to loop (for 630 layers) through the code to add a layer to the current mxd, change the title according to the layer name and export the map. My code is running fine for a single layer but I can't get it to loop using a list.
I attach the working single layer code:

import arcpy, os
mxd=arcpy.mapping.MapDocument("CURRENT")
lyrFile=arcpy.mapping.Layer(r"C:\Temp\LAYERS\Acer_campestre.lyr")
df=arcpy.mapping.ListDataFrames(mxd,"Layers")[0]
arcpy.mapping.AddLayer(df,lyrFile)
arcpy.RefreshTOC()
arcpy.RefreshActiveView()
print mxd.filePath
maplayers=arcpy.mapping.ListLayers(mxd,"*Layer",df)
print maplayers
for lyr in maplayers:
    lyrname=lyr.name
    lyrname2=lyrname.replace("_Layer","")
    lyrname3=lyrname2.replace("_"," ")
    print lyrname3
for elm in arcpy.mapping.ListLayoutElements(mxd, "TEXT_ELEMENT"):
    if elm.name == "Species":
        elm.text = lyrname3
    else:
        print "Element not updated!"
print elm.name       
arcpy.RefreshActiveView()
arcpy.mapping.ExportToTIFF(mxd,r"C:\Temp\TIFF\_"+str(lyrname3)+".tif",resolution=300)
del mxd


Now here is a stripped down version of what's not working. I have stripped it down to the part where its falling over. It seems to be at addLayer I get an error - Object:create object layer invalid data source.

import arcpy, os
from arcpy import env
env.workspace="C:/Temp/LAYERS"
mxd=arcpy.mapping.MapDocument("CURRENT")
slayer=arcpy.ListFiles("*.lyr")
lcount=len(slayer)
print lcount
for filelay in slayer:
    arcpy.env.OverwriteOutput=True
    print filelay
    df=arcpy.mapping.ListDataFrames(mxd,"Layers")[0]
    lyrFile=arcpy.mapping.Layer(filelay)
    arcpy.mapping.AddLayer(df,lyrFile)
    arcpy.RefreshTOC()
    arcpy.RefreshActiveView()
    mxd.SaveACopy(r"C:/Temp/Project.mxd")
    del mxd,lyrFile
   

From looking online this doesn't help either: (I just get an mxd error)

import arcpy, os
mapDoc=arcpy.GetParameterAsText(0)
lyrFiles=arcpy.GetParameterAsText(1)
layers=lyrFiles.split(";")
mxd=arcpy.mapping.MapDocument("CURRENT")
mxd=arcpy.mapping.MapDocument(mapDoc)
df=arcpy.mapping.ListDataFrames(mxd)[0]
for lyrFile in layers:
    lyr=arcpy.mapping.Layer(lyrFile)
    arcpy.mapping.AddLayer(df,lyr)
mxd.save()

Can anyone help please? I would really appreciate any advice - relatively new to python

[LorraineBarry]

Sorry my code is indented properly -but got lost through copy and paste

[MathewCoyle]

try using code tags
    for proper indentation

Without the spaces
[ CODE][/CODE ]

[MathewCoyle]

As for your code problem, try this. Notice how disrespectful of workspaces layers are.

import arcpy, os
from arcpy import env
arcpy.env.OverwriteOutput=True
inputWS = r"C:/Temp/LAYERS"
env.workspace=inputWS
mxd=arcpy.mapping.MapDocument("CURRENT")
df=arcpy.mapping.ListDataFrames(mxd,"Layers")[0]
slayer=arcpy.ListFiles("*.lyr")
lcount=len(slayer)
print lcount
for filelay in slayer:
    print filelay
    lyrFile=arcpy.mapping.Layer(inputWS+os.sep+filelay)
    arcpy.mapping.AddLayer(df,lyrFile)
    arcpy.RefreshTOC()
    arcpy.RefreshActiveView()

[LorraineBarry]

Brilliant Many thanks Matthew. This worked a treat. I can now add in the rest of the tasks for each layer.
You've really helped. I appreciate it
Lorraine

[LorraineBarry]

Hello Matthew

This code really helped. The code runs through the layers and outputs as Tiff etc. to output folder.
Upon verification however I've noticed that there are 524 input layers but only 480 output images. The output maps and the text on the maps are out of sync. Some of the data layers do not match up to the text title (TEXT_ELEMENT=Species) (they are quite often one layer out while others are correct.
I have attached the code.
I am confused. I have considered a few possible causes - slow refresh (but I've added in a couple of extra view refreshes), longer layer names (none are beyond 30 characters) or the fact that the second layer added in is given the addition of _Layer1 instead of _Layer. Or it could be something else - I'm not sure. Do you have any thoughts on this?