ProjectExplorer

[GrilledSheep]

man kann im mommetn beliebige dateien dropen. bei z.B. exe dateine stürtzt das programm ab.

[rtur]

hmm, stimmt wahrscheinlich. Habe ich nicht bedacht.

Aber abstürzen wird es nicht weil er es nicht checkt, sondern da der Handler.consistencyCheck ist die sache mit der fileSize im verhältnis zur frameByteSize prüft. Die ausnahme muss abgefangen und behandelt werden (im vidImport).

Kannst du ja fertigstellen wenn du magst. Der Plan ist die ausnahme abzufangen und es dem nutzer (nicht unbedingt mit dem modalen Dialog, vlcht irgendwas rot unterstreichen und ne Meldung auf das bestehende Dialog drucken) bekanntmachen

[GrilledSheep]

ich wäre jetzt im projektexplorer hingegangen und hätte die endung des videos überprüft

[rtur]

Das sowieso, aber das allein reicht nicht. Consistency chek prüft das video file an sich, wenn du z.b. exe zu yuv umbenennst würde er in den meisten Fällen sagen, dass etwas nicht in Ordnung ist

[GrilledSheep]

was für ne exceptions is das genau? er sagt nur exception und catch exception is net schön. aber im moment gehts so

[rtur]

hab gerade nochmal nachgeschaut, wie es aussieht werfe ich da gar keine Exception. Es wird nur das consistency flag auf false gesetzt.

Was steht in der Exception ? Und woher wird Sie geworfen ?

[GrilledSheep]

Cant process given video file format. und ansonstenw irklich nur null

[rtur]

das wird vom Video geworfen, wird geworfen wenn kein passender pluginType gefunden wird um das Format zu lesen (ist eig. klar aber lässt sich schnell vergessen, du solltest jetzt nicht einfach im ProjectExplorer nur yuvs Zulassen)

[rtur]

vielmehr sollten wir sogar exe zulassen (ins import) und wenn dort die cant Process exception kommt den Nutzer darauf aufmerksamm machen. Vlcht ist die Endung die in seinem Video verwendet wurde einfach nur nich standart und er kann ja die dropBox mit den known Formats dazu nutzen einen anderen Format auszuwählen

[GrilledSheep]

jettzt is mir auch klar warum da nur exceptions steht. ich habs mal in file format exception umbenannt, weil wenn man catch exception schreibt is das das selbe wie catch throwable aus java.

[GrilledSheep]

es amcht für mich keinen sin exe dateien zuzzulassen. aber du kannst dir ja mal meine jetzige lösung anschauen und sie danach umschreiben. von mir aus kannst du closen

[sleidig]

Unbekannte Fileextensions wuerde ich auch nicht zulassen, allerdings sollte das Programm mit einer "kaputten" Datei (oder einer exe die eine .yuv Endung hat) halt nicht abstuerzen.

[rtur]

die falsche endung wird erkannt, und es wird eine exception geworfen (weiß nicht ob die inzwischen abgefangen wird).

Über die unbekannten fileExtensions bin ich prinzipiell eurer Meinung. Aber dafür fehlt uns zurzeit eine Liste der bekannten fileextensions, und da ich weiß wie hier manch einer auf "lasst uns das vervollständigen" Vorschläge reagiert habe ich die einfachere alternative (alles zulassen, wenn beim import consistency falsch gesetzt ist -> nutzer bescheid geben) vorgeschlagen.

[GrilledSheep]

naja doof finde ich im momment nur dass die message box manchmal hinter oqat auftaucht, und es dann trotzdem so ausschaut als ob es abgestütz wäre. ich würde da mit dem illegalen file auch im video options dialog machen, und keine exceptionw erfen. grund: user möchte mehrere videos hinzufügen und eines is ramsch, dann kann man hier noch schauen dass die richtigen reinkommen und dass flasche nicht(hoffe ich zumindest).

[GrilledSheep]

ich habe mal ne lösung gepusht. sie is zwar im projek explorer realisiert aber sollte gehen. plöd is das manchmal die messageboxne hinter oqat auftauchen, vermutlich weil alles auf vordergrund ist.

[GrilledSheep]

nee msus nochw er nen blick drauf werfen. auserdem stützt der vid import dialog noch ab wenn man 0/0 einträgt...

[rtur]

"auserdem stützt der vid import dialog noch ab wenn man 0/0 "

Ich bin mir sicher du schaffst es das auszumerzen.

Mein erster Gedanke wäre, dass es vielleicht helfen würde die Division durch null zu verhindern.

[GrilledSheep]

dann setze deine gedanken um und denk dir das nexte mal, vielleciht hat es einen grund dass er es hier reinschreibt und nicht selber macht.

[rtur]

das width/height Problem ist fixed.

[GrilledSheep]

dann sind iwr hier fertig.