在python中附加到列表会引发错误

Question

我对 Python 还很陌生，我想做的是以下几点：

1. 读取文件
2. 将元素添加到字典中的每一行
3. 如果键已经存在，则将它们的键值放入列表中，并将新值附加到列表中。
4. 用新的值列表更新字典

这是我的代码：

jobChangeMap=dict() # Maps the changed job name. Format will be      jobset,originaljob,jobNumber:jobset,changedjob,jobnumber
jobValues = []

if(not jobsetName in jobsetJobMap.keys()):
    logger.debug("{0} not found as a key. Adding key and value pair".format(jobsetName))
    jobsetJobMap.update({jobsetName:jobName+","+jobNumber})
    logger.debug(jobsetJobMap)
else:
    jobValues = []
    jobValues[:]=[]
    logger.debug("{0} already found as a key. Getting list of values for key".format(jobsetName))
    jobValues=jobsetJobMap[jobsetName]
    logger.debug("Number of values for {0} is {1}".format(jobsetName,jobValues.count))
    logger.debug(jobsetJobMap)
    if(not jobName in jobValues):
        myJobName=jobName+","+jobNumber
        logger.debug("Size before adding: {0}".format(len(jobValues)))
        jobValues.append(myJobName)

这是引发以下错误的行：

File "D:\Git-Python\MountSophie-Conversion\jmoConversion\JMOExtract.py", line 163, in createJobMap
    jobValues.append(myJobName)
AttributeError: 'str' object has no attribute 'append'

为什么 Python 抱怨 jobValues 是一个 str 对象，即使我将它明确定义为类型列表？

Answer 1

根据这一行，jobsetJobMap 的值似乎是字符串：

jobsetJobMap.update({jobsetName:jobName+","+jobNumber})

这将使用新字符串 jobName+","+jobNumber 覆盖条目 jobsetJobMap[jobsetName]，您稍后会在此行中检索并设置为 jobValues 的值：

jobValues = jobsetJobMap[jobsetName]

这会用字符串覆盖之前的list 值jobValues，并且字符串没有.append 方法——因此会出现错误消息。

从你的变量名推断你的意图，我认为你想做这样的事情：

# (earlier initialization)
import collections
jobsetJobMap = collections.defaultdict(list)
# ...
# (later update):
    jobsetJobMap[jobsetName].append( jobName + "," + jobNumber )

这会将作业描述符附加到与 jobsetJobMap 字典中的键 jobsetName 关联的列表（如果需要，隐式创建列表）。

Answer 2

关于您遇到的错误。 @jez 回答完美地解释了它。

您显然是 python 新手，而且您在 Java/C# 世界中生活的时间太长了。 JK。

我为什么这么说？

首先，您使用的变量命名约定 (CamelCase) 是 [Java/C#] 风格。你应该更喜欢 python 上的snack_case。更多信息请参考Style Guide for Python Code。

您对使用 python 进行此类程序是正确的。因为当涉及到这类问题时，python 是如此强大。但是，您并没有充分利用 python 的力量。这不应该占用这么多代码行。

保持您的数据结构同质化。在jobsetJobMap 中，您存储的是字符串或数组。即使它们只包含一个元素，您也可能希望将它们全部保存为数组。异构数据导致更多代码，更多代码等于更多错误。

我会建议更多类似的东西。

#!/usr/bin/env python

lines = ['name1,1', 'name2,2', 'name1,3', 'name2,1', 'name3,1', 'name3,1']

jobs_map = {}

for line in lines:
    [name, number] = line.split(',', 1)
    if name in jobs_map:
        jobs_map[name].append("%s,%s" % (name, number))
    else:
        jobs_map[name] = ["%s,%s" % (name, number)]

Answer 3

我想我找到了解决这个问题的方法。它可能不是最聪明的代码，但它是我对 Python 解决方案的尝试

if(not jobsetName in jobsetJobMap.keys()):
                    jobValues = []
                    myNewJobName=jobName+","+jobNumber
                    jobValues.append(myNewJobName)
                    logger.debug("{0} not found as a key. Adding key and value pair".format(jobsetName))
                    jobsetJobMap.update({jobsetName:jobValues})
                    logger.debug(jobsetJobMap)
                else:
                    jobValues = []

                    logger.debug("{0} already found as a key. Getting list of values for key".format(jobsetName))
                    jobValues=jobsetJobMap.get(jobsetName) # Get list of values associated with the key.


                    logger.debug(jobsetJobMap)
                    if(not jobName in jobValues):
                        myJobName=jobName+","+jobNumber
                        logger.debug("Size before adding: {0}".format(len(jobValues)))
                        jobValues.append(myJobName)
                        jobsetJobMap.update({jobsetName:jobValues})

                    else:
                        jobCount=jobValues.count(jobName)
                        logger.debug("Job {0} already exists in the list. Adding a unique name")
                        newJobName=jobName+".M"+jobCount+","+jobNumber
                        logger.debug("{0} changed to {1}".format(jobName,newJobName))
                        jobChangeMap.update({jobsetName+","+jobName+","+jobNumber:jobsetName+","+newJobName+","+jobNumber})
                        jobValues.append(newJobName)
                        jobsetJobMap.update({jobsetName: jobValues})