Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added File Connector #9

Merged
merged 9 commits into from Jun 11, 2019

Conversation

Projects
None yet
2 participants
@derNiklaas
Copy link
Contributor

commented May 22, 2019

Added basic file i/o.

Current functions:

  • Read/Write plaintext files (.txt, .json, ...)
  • Read/Write binary files
  • Read/Write Images

derNiklaas added some commits May 22, 2019

@sebinside

This comment has been minimized.

Copy link
Member

commented May 23, 2019

Danke für den Pull-Request! Ein paar Verbesserungsvorschläge / Anmerkungen vor dem Merge:

  • Was genau steckt hinter der Veränderung in fixPath? @joblo2213 hat hier ja auch erst kürzlich was geändert, die ursprüngliche Version hat aber (glaube ich) auch funktioniert. Ggf. nochmal kurz abklären
  • ImageFormat in der API: Java (sollte) unterstützt nativ JPEG, PNG, GIF, BMP and WBMP [1]. ImageFormats ist somit unvollständig und evtl. auch nicht notwendig: ImageIO.getWriterFormatNames();.
  • Die Dokumentation Requires a FileSystemActor that has to be created by the framework user. verstehe ich nicht. Warum muss ein Plugin Entwickler über den Actor im Connector bescheid wissen? Er nutzt selbst nur Inputs & Outputs
  • Die Implementierung von getFile (und analg) im FileInputImpl ist schlechter Stil. Exceptions dürfen niemals für den Kontrollfluss eingesetzt werden. Hier lieber mit den Scala Option Methoden isDefined bzw. isEmpty arbeiten, bevor du get aufrufst. Ebenfalls fraglich ist, ob man einen String "ERROR" zurückgeben sollte, oder einfach das Option-Objekt weiter reicht und nicht mit dem Paradigma bricht

Das war der erste Blick, ansonsten eine gute Erweiterung!

[1] https://docs.oracle.com/javase/tutorial/2d/images/saveimage.html

@derNiklaas

This comment has been minimized.

Copy link
Contributor Author

commented May 23, 2019

Hi. Es fehlt soweit nur noch der 2. Punkt (und evtl. weitere Verbesserungen von deiner/eurer Seite).

Den Konflikt werde ich nachdem beheben, sobald das Problem mit @joblo2213 soweit behoben worden ist.

derNiklaas added some commits May 31, 2019

@derNiklaas

This comment has been minimized.

Copy link
Contributor Author

commented May 31, 2019

Hi. Soweit sollte alles dabei sein. Hab folgende Sachen getestet:

  • Lesen: Textdatei, JPG, PNG
  • Erstellung: Textdatei, JPG, PNG, Ordner
  • Erstellung bzw. das lesen von Dateien außerhalb des data Ordners funktioniert nicht.

derNiklaas added some commits Jun 11, 2019

@sebinside

This comment has been minimized.

Copy link
Member

commented Jun 11, 2019

Well done!

@sebinside sebinside merged commit ac09a8c into codeoverflow-org:master Jun 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.