LNU python else i finally

Question

Zadanie:

obok zakładki z kodem źródłowym znajdziesz plik data.txt. Do funkcji addData() przez parametr przekazano pewną wartość. Jeżeli wartość jest typu string (łańcuch), funkcja powinna dopisać ją w nowym wierszu do pliku data.txt. Jeżeli wartość jest innego typu, funkcja powinna zwrócić wartość
​"Type error". Jeżeli plik został otwarty i nie zgłoszono wyjątku, funkcja powinna zamknąć plik i zwrócić wartość "Done". 

Przykłady:

data.txt:

Monty⤶ Python⤶

1.
Wejście: "The larch"

Wyjście: "Done"

data.txt

Monty⤶ Python⤶ The larch⤶

2. 
Wejście: 123

Wyjście: "Type error"

data.txt:

Monty⤶ Python⤶

 

Nie rozumiem dlaczego program nie zwraca mi "done".

def addData(data):
    if isinstance(data, str):
        try:
            with open("data.txt", "a") as file:
                file.write(data + "\n")
            return "Done"
        except Exception:
            return "Error writing to file"
    else:
        return "Type error"

 

Comments

Nie sprawdzałem kodu, ale może to sprawa wcięć (spacji) - problem powszechny w pythonie :)

Answer 1

IMO Twój kod działa poprawnie on-line.

W dokumentacji jest nawet

The isinstance() built-in function is recommended for testing the type of an object, because it takes subclasses into account.

 

BTW, w zależności od kontekstu i potrzeb, można zdecydować, która funkcja jest bardziej odpowiednia. Jeśli chcesz sprawdzić konkretny typ obiektu, type() może być bardziej odpowiednie. Jeśli natomiast chcesz uwzględnić dziedziczenie i sprawdzić, czy obiekt jest instancją danego typu lub jednego z jego podtypów, isinstance() jest lepszym wyborem.

Ja użyłbym w tym zadaniu, on-line.

if type(data) == str:

 

Comments

niestety nadal nie działa

PASS addData((58, ('Faith', {'Ebony': 74}))): Result: 'Type error' Expected: 'Type error'

FAILED addData(Joy Cotton lives in Leczna since 19 years.): Result: Wrong data.txt content Expected: 'Done'

---

No bo zapis:

addData(Joy Cotton lives in Leczna since 19 years.):

nie jest string-iem, zapisz tak

addData("Joy Cotton lives in Leczna since 19 years."):

---

Jeżeli taki zapis jest

addData((58, ('Faith', {'Ebony': 74}))):

to w tym przypadku IMO lepiej użyć: isinstance().

 

 

BTW, jak ciekawostka, całość zapisana w try, on-line.

Answer 2

Problem polega na tym, ze isinstance(data, str) nie jest wystarczający, aby wykryć, czy zmienna data jest liczbą
Z uwagi na to, że liczba 123 nie jest instancją typu str, nie zostanie zgłoszony błąd typu, a kod będzie kontynuowany...

Comments

Jeśli celem twoim jest np. sprawdzenie czy cos tylko jest stringiem, spróbuj przez wymuszenie konwersji 

 

str_data = str(data)  # Próba konwersji na łańcuch znaków