Question

我正在尝试编写我的第一个面向对象程序。我提出的代码是：

    class Lattice:
        'Commomn base for all sublattice'
        latc = 0

        def __init__(self, conc, name, pot):
            self.atcon = atcon
            self.name =name
            self.pot=[]
            Lattice.latc += 1
    atcon=[]
    with open(inpf, "r") as f:
        for line in f:
            match = re.search(reall, line)
            if match:
                if (match.group(1).strip() == "atcon"):
                    atcon.append(match.group(2).split())

print("CON    =>"+str(atcon))
print("CON[0] =>"+str(atcon[0]))
lat0=Lattice(atcon[0],pot[0],name[0])
print("lat0   =>"+str(lat0.atcon))

我原以为lat0.atcon将是atcon[0] 但是最后3个打印语句的结果是：

CON    =>[['0.5d0', '0.50d0'], ['0.98d0', '0.02d0'], ['0.98d0', '0.02d0'], ['0.98d0', '0.02d0']]
CON[0] =>['0.5d0', '0.50d0']
lat0   =>[['0.5d0', '0.50d0'], ['0.98d0', '0.02d0'], ['0.98d0', '0.02d0'], ['0.98d0', '0.02d0']]

我不明白为什么。我是一个绝对的初学者，没有正式的python培训（使用net和SO学习），所以请耐心等待。

更新在接受回复后，这是我目前正在处理的代码。问题是，我正在阅读list中的所有内容，然后将这些列表插入lat0，即

    #if match found
    atcon.append(match.group(2).split())
# after getting all match, in global scope
lat0=Lattice(atcon[0],name[0],pot[0])

所以，我想我要么浪费list要么浪费对象lat0。当找到匹配时，我是否有可能直接填充lat0？ e.g。

    #if match found for <name>
    lat0=Lattice(name)

mini.py：

#!/usr/bin/env python3
import sys
import re


class Lattice:
    'Commomn base for all sublattice'
    latc = 0

    def __init__(self, conc, names, pots):
        self.atcon = conc
        self.name =names
        self.pot=pots
        Lattice.latc += 1

reall='(.*)\s*=\s*(.*)'
inpf = sys.argv[1]
print(inpf)

with open(inpf, "r") as f:
    pot=[]
    name=[]
    atcon=[]
    for line in f:
        match = re.search(reall, line)
        if match:
            if (match.group(1).strip() == "atcon"):
                atcon.append(match.group(2).split())

            if (match.group(1).strip() == "pot"):
                pot.append(match.group(2).split())

            if (match.group(1).strip() == "name"):
                name.append(match.group(2).split())

lat0=Lattice(atcon[0],name[0],pot[0])
print("POT =>"+str(pot))
print("NAME =>"+str(name))
print("CON    =>"+str(atcon))
print("CON[0] =>"+str(atcon[0]))
print("lat0   =>"+str(lat0.pot))

典型输入

   pot=Rh1.pot Rh2.pot Fe1a.pot Fe2a.pot 
   name=Rh-up Fe-up
   atcon=0.5d0 0.50d0 

   pot=Rh3.pot  Rh4.pot Fe3a.pot Fe4a.pot
   name=Rh-up Fe-up
   atcon=0.98d0 0.02d0

Answer 1

我打赌您在IDLE中编写或测试过此类。在这一点上，我确信它真的令人困惑，但错误非常简单。在您实例化课程时，通常建议您使用发送到__init__的值，而不是引用其他值。

class Lattice:
    'Commomn base for all sublattice'
    latc = 0

    def __init__(self, conc, name, pot):
        self.atcon = conc
        self.name =name
        self.pot=pot
        Lattice.latc += 1

发生了什么atcon，pon和name是在全局范围内定义的，您在下面的示例中引用了它们：

atcon=[1, 2, 3, 4, 5, 6]
pot = [7,8,9]
name = ["foo", "bar"]

class globs:
    def __init__(self):
        self.atcon = atcon
        self.pot = pot
        self.name = name

其中给出了以下输出：

>>> g = globs()
>>> g.atcon
[1, 2, 3, 4, 5, 6]
>>> g.pot
[7, 8, 9]
>>> g.name
['foo', 'bar']

编辑回答扩展编辑原始问题。

我想我明白了。有两件事让我感到困惑：

如果我遵循代码，看起来你只想把文件中的第一个命中为lat0。但是你还没有解释你是否只想要文件中的第一个命中，或者是所有命中的对象列表。
你做了一个分割，但是根据你的样本输入仍然会返回一个列表，即["Rh1.pot", "Rh2.pot", "Fe1a.pot", "Fe2a.pot"]，我可能会冒昧但我在分割后添加[0]只能检索第一击。如果我错过了这一点，请删除它。

这是在找到第一个匹配后停止循环的代码。我将atcon，pot和name声明为列表，因为.split()会返回一个列表，但我不会将结果追加到浪费内存中。我也return Lattice反对退出函数并避免浪费时间解析其余的行。

此外，最终的if atcon and pot and name是为了避免在有一段文字匹配的情况下返回，但不包含所有重要信息。在python if中，空列表将为False。您可以保留其余代码（打印语句除外）。

inpf = sys.argv[1]
print(inpf)

def parse(inpf):
    atcon, pot, name = [], [], []
    reall='(.*)\s*=\s*(.*)'
    with open(inpf, "r") as f:
        for line in f:
            print(line)
            if match:            
                if (match.group(1).strip() == "atcon"):
                    atcon = match.group(2).split()[0]

                if (match.group(1).strip() == "pot"):
                    pot = match.group(2).split()[0]

                if (match.group(1).strip() == "name"):
                    name = match.group(2).split()[0]

                if atcon and pot and name:
                    return Lattice(atcon, name, pot)

lat0 = parse("test.txt")
print("lat0   =>"+str(lat0.pot)+" "+str(lat0.name)+" "+str(lat0.atcon))

经过测试

atcon=0.5d0 0.50d0
atcon=0.5d0 0.50d0 
atcon=0.5d0 0.50d0 

pot=Rh1.pot Rh2.pot Fe1a.pot Fe2a.pot 
name=Rh-up Fe-up
atcon=0.5d0 0.50d0 

pot=Rh3.pot  Rh4.pot Fe3a.pot Fe4a.pot
name=Rh-up Fe-up
atcon=0.98d0 0.02d0

输出：

lat0   =>Rh1.pot Rh-up 0.5d0

python对象采取完整列表

1 个答案: