Faltou a função retornar um valor. No final deveria ter um return final
:
def dnaComplement(s):
complement = []
complement0 = reversed(s)
for character in complement0:
if character == 'G':
complement.append('C')
elif character == 'C':
complement.append('G')
elif character == 'A':
complement.append('T')
elif character == 'T':
complement.append('A')
final = ''.join(complement)
print(final)
return final # <-- aqui
Quando não há nenhum return
, a função retorna None
¹ (veja mais detalhes na documentação). Colocando um valor de retorno, ela se comporta como você espera, retornando a string.
Ou ainda, se não quiser imprimir e não for usar a variável final
para mais nada, nem precisa dela, pode retornar direto o resultado do join
:
def dnaComplement(s):
complement = []
complement0 = reversed(s)
for character in complement0:
if character == 'G':
complement.append('C')
elif character == 'C':
complement.append('G')
elif character == 'A':
complement.append('T')
elif character == 'T':
complement.append('A')
return ''.join(complement)
E como o retorno de join
já é uma string, não precisa usar str(dnaComplement(s))
, basta chamar a função diretamente que você já terá uma string:
s = input()
result = dnaComplement(s) # retorno da função é uma string, não precisa de str()
# etc...
Não diretamente relacionado, mas você pode simplificar a função usando um dicionário contendo os mapeamentos entre os caracteres:
def dnaComplement(s):
comps = { 'G': 'C', 'C': 'G', 'A': 'T', 'T': 'A' }
complement = []
for character in reversed(s):
complement.append(comps[character])
return ''.join(complement)
E pode ainda usar uma generator expression (bem mais sucinta e pythônica), aí nem precisa gerar a lista complement
:
def dnaComplement(s):
comps = { 'G': 'C', 'C': 'G', 'A': 'T', 'T': 'A' }
return ''.join(comps[character] for character in reversed(s))
1: a exceção a esta regra é quando a função tem um yield
em vez de return
, aí ela se torna um generator.