Hay muchas cosas que contar respecto al código que has incluido en la pregunta. Vamos por pasos:
- Tienes una función para leer el fichero. Tal como la has escrito lo que retorna es una lista, cuyos elementos serían las líneas
- Tienes una función que toma una lista de cadenas y las concatena, para convertirlo todo en una larga cadena única.
- Llamas a ambas funciones en
text(read())
, lo cual no tiene demasiado sentido. ¿Por qué partir el fichero en líneas dentro deread()
para después volver a juntar todo en una sola cadena dentro detext()
? Para eso mejor lees el fichero conread()
en vez de conreadlines()
, pues de ese modo ya lo obtienes en una sola cadena, y te ahorras la llamada atext()
- A la hora de dividir el texto en palabras usas
.split(" ")
, pasando un espacio como parámetro. Esto funciona sólo si cada palabra está separada de la siguiente por un espacio y solo uno. Pero te dará problemas si el separador son varios espacios, o un tabulador, o un carácter invisible. Es mejor llamar a.split()
sin parámetros para que funcione también en los casos anteriores. Además, resulta que.split()
sin parámetros también considera el\n
como separador, por lo que funcionará con palabras al final de la línea, y evitarás que esos\n
aparezcan en el resultado. - Para no tener que andar abriendo y cerrando el fichero "a mano" (es decir, usando
.close()
) se recomienda usar un contexto (sentenciawith
). De este modo no necesitas cerrar el fichero porque python lo hace por ti cuando el contexto finaliza. - No necesitas pasar la opción
"r"
al abrir el archivo, ya que ese es el modo por defecto.
Juntando todo lo anterior, esta sería mi solución:
with open("Names.txt") as f:
contenido =f= f.read()
data = contenido.split()
print(data)