有什么问题吗？ [关闭]

Question

我正在学习 python Selenium，并创建了一个“电子邮件”列表：“密码”。我需要在正确的位置插入这些电子邮件，然后是密码，然后是登录按钮。这是代码：

f = open('users.txt','r')
for line in f.readlines():
    print(line)
    mail = line[0:line.index(':')]
    line = line[line.index(':')+1:]
    password = line
    email_input = driver.find_element_by_xpath('//*[@id="email"]')
    email_input.click()
    email_input.send_keys(mail)
    password_input = driver.find_element_by_xpath('//*[@id="password"]')
    password_input.click()
    password_input.send_keys(password)
    lgn_btn = driver.find_element_by_css_selector('#sign-in')
    lgn_btn.click()
    n_url = driver.current_url
    time.sleep(5)
    driver.refresh() 

我得到的错误是：

mail = line[0:line.index(':')]
ValueError: 未找到子字符串

txt文件是：

g_santeusanio@arcor.de:sportpark13 
mrtslabbert@absamail.co.za:oohethooq12313w 
lauriecd@absamail.co.za:christcorem

Answer 1

你应该使用

line.find(':')

而不是line.index(':') 这是因为，index() 方法在 中搜索子字符串并返回它的索引。

由于要在字符串中查找单个字符，所以应该使用find() 方法！

两个代码都使用的文本文件，演示：

根据@John Gordon 的评论进行编辑

当字符串中不存在这样的值时，index() 方法会返回 值错误。使用的文本文件有 2 个空行。 find() 方法返回 -1 而不是错误。

请注意，问题中使用的代码块打印了第一对邮件和密码

当它在第二行（为空）中搜索值时，它会引发ValueError。另一方面，如果您在使用find() 方法时看到输出（下面附上的img），它只会打印出空字符串并继续前进！

因此，当您不确定要搜索的值是否在字符串中时，最好使用find()。

基于来自@Chris 的 cmets 进行的编辑

这是我使用的代码：

for line in f.readlines():
    # print(line)
    mail = line[0:line.find(':')]
    password = line[line.find(':') + 1:]
    print(mail)
    print(password)

我收到的您粘贴的文本的输出是这样的：

Answer 2

我认为在这种情况下split() 更容易理解和使用。 split() 按提供的字符拆分字符串并返回包含结果的值数组。在这种情况下，我们可以通过“：”进行拆分，得到两部分。

所以前几行变成了

for line in f.readlines():
    print(line)
    data = line.split(':')
    mail = data[0]
    password = data[1]
    ...

这应该可以解决您询问的问题，但查看您的其余代码，还会出现其他问题，我有一些建议。

1. 不要覆盖变量。

   line = line[line.index(':')+1:]
   

   它使您的代码更难遵循，例如你在这里用另一个值覆盖了line。现在您无法再访问 line 的原始值，并且随着脚本变得越来越长，其他读者（包括您自己）将难以调试代码。而是给它一个新名称，以最好地代表变量将包含的内容。在本例中为 password，就像您在下一行所做的那样。

   line = line[line.index(':')+1:]
   password = line
   

   变成

   password = line[line.index(':')+1:]
   

2. 使用最简单的定位器形式，例如如果您通过 ID 定位元素，则使用 .find_element_by_id()，而不是 XPath 或 CSS 选择器。这将使您的代码更易于阅读。

    email_input = driver.find_element_by_xpath('//*[@id="email"]')
   

   变成

    email_input = driver.find_element_by_id('email')
   

3. 您正在单击，然后将元素的键发送到同一元素。通常，这不是必需的，您可以省略单击。 （在某些情况下这是必要的，但根据我的经验，这种情况相当罕见）。

    email_input = driver.find_element_by_id('email')
    email_input.click()
    email_input.send_keys(mail)
   

   变成

    email_input = driver.find_element_by_id('email')
    email_input.send_keys(mail)
   

   甚至

    driver.find_element_by_id('email').send_keys(mail)
   

   因为您之后似乎不再使用email_input。

所以用上面的建议更新代码，它看起来像这样

for line in f.readlines():
    print(line)
    data = line.split(":")
    email = data[0]
    password = data[1]
    driver.find_element_by_id('email').send_keys(email)
    driver.find_element_by_id('password').send_keys(password)
    driver.find_element_by_id('sign-in').click()
    n_url = driver.current_url
    time.sleep(5)
    driver.refresh()