two squares - Fehler unklar

[Craven]

Hi,

Ich komm grad aus unerfindlichen Gründen an dem Script nicht weiter, da mich ein Fehler daran hindert, den ich grad nicht nachvollziehen kann ... Die Uhrzeit kann auch ein Faktor sein.

Code: Alles auswählen

def twosquares():
    t = raw_input()
    numbers = []
    for a in range(0, t):
        numbers.append(int(raw_input()))
    for a in range(0, t):
        z = int(sqrt(numbers[a])
        if type(numbers[a] - (z ** 2.0)) == type(1.0):
            print("No")
        elif ((z ** 2) + (numbers[a] - (z ** 2))) == numbers[a]:
            print("Yes")

Das Script dürfte selbsterklärend sein. Hier der Fehler:

Traceback (most recent call last):
File "<string>", line 1, in <string>
invalid syntax: <string>, line 1003, pos 68

... also in der Zeile:

Code: Alles auswählen

        if type(numbers[a] - (z ** 2.0)) == type(1.0):

Weiß wer, was den Fehler auslöst, und noch wichtiger, wie ich ihn beheben kann? Die Frage ist bei dem Script eigentlich rhetorisch ...

Gruß, Phillip

[BlackVivi]

Auch wenn mir die Codeauszug einige Fragen aufwirft, weiß ich trotzdem wo dein Problem liegt:

Code: Alles auswählen

type(float(numbers[a]) - (float(z) ** 2.0))

Schau nochmal genau... Du versucht von einem Type etwas abzuziehen. Also eher:

Code: Alles auswählen

type(float(numbers[a] - (float(z) ** 2.0)))

Wobei ich nich den blassesten Schimmer einer Ahnung habe, was das bewerkstelligen soll. Du könntest auch einfach == float schreiben, anstatt type(1.0) oder... isinstance(foobar, float) O_o Allgemein ... print in einer Funktion?.... Ach, ich hab keine Ahnung. Kenn ja das Skript nicht.

Außerdem wird der type _IMMER_ float sein, du sagst es ja direkt davor oO' Du müsstest schauen ob nachkommastellen vorhanden sind...

//Edit:

Wieso ist oben anderer Code als unten oO'? Ich bin vollkommen verwirrt. Im oberen müsste es funktionieren, im unteren auf keinen Fall. Aber das ändert nichts daran, das's immer Float ist, also die erste Bedingung immer erfüllt wird.

[Craven]

So, jetzt ist es mir aufgefallen ... und nachdem ich den Editor zum Teufel gejagt hab, post ichs:

Code: Alles auswählen

def twosquares():
    t = raw_input()
    numbers = []
    for a in range(0, t):
        numbers.append(int(raw_input()))
    for a in range(0, t):
        z = int(sqrt(numbers[a])) # Die gottverdammte Klammer ...
        if type(numbers[a] - (z ** 2)) == type(1.0):
            print("No")
        elif ((z ** 2) + (numbers[a] - (z ** 2))) == numbers[a]:
            print("Yes")

Dass der Code oben und unten verschieden war, lag daran, dass ich in der Zwischenzeit noch herumprobiert und die Zeile kopiert hatte ...

Edit: Mir fällt grade auf, nachdem ich es jetzt endlich ausführen kann, dass es immer ein int ist. Halb vier ist doch zu spät.

[BlackVivi]

Gut, das mit'r Klammer erscheint mir noch logischer, manchmal sieht man eben den Wald vor lauter Bäumen nicht. Aber trotzdem wird das Ding defintiv immer Float sein.

//Edit: Ach, jetzt auch nich mehr. Bin so verwirrt. 3 verschiedene Codes in wenigen Minuten, zwischen Antwort und Frage. *seufz..

Aber du kannst trotzdem

Code: Alles auswählen

type(numbers[a] - (z ** 2)) == float:

schreiben, dass is'n bissel offensichtlicher.

//Edit2: oder...

Code: Alles auswählen

if isinstance(numbers[a] - (z ** 2), float):

[Craven]

Code: Alles auswählen

        if type(sqrt(numbers[a] - (z ** 2))) == type(1.0):
            print("No")

Auf das wollte ich raus.

Thx @ BlackVivi, ohne ihn wäre das nicht möglich gewesen.

[Leonidas]

Oft hilft es der Übersichtlichkeit, wenn man komplizierte Bedingungen in einfachere Teile aufspaltet.

Hint: `print`ist ein Statement

[BlackJack]

Das bleibt trotzdem sehr wirrer Code. Was soll das denn bewirken? Das kann man mit Sicherheit einfacher ausdrücken. Ausserdem kann man direkt über die Liste iterieren statt einen Index zu benutzen. Und wieso steckt das in einer Datei mit mehr als 1000 Zeilen!? Fragen über Fragen…

Edit: Mal so geraten was das Ganze soll:

Code: Alles auswählen

def two_squares():
    numbers = [int(raw_input()) for __ in xrange(int(raw_input()))]
    for number in numbers:
        root = sqrt(number)
        print 'Yes' if root == int(root) and number > 1 else 'No'