들여쓰기에 따라서 return 값이 왜 다르게 나오는지 궁금합니다

왜 None 값이 나오는 걸까요?:

for문을 사용하지 않으면 의미가 없나요?

제 코드 한번 봐주실 수 있나요?ㅜㅠ

좀 더 효율적인 코드로 바꾸는 방법은 뭐가 있을까요?