Skip to content

code cleanup/optimisation; virtualenv preparation; see README. - #1

Open
XenGi wants to merge 1 commit into
ij0n:masterfrom
XenGi:master
Open

code cleanup/optimisation; virtualenv preparation; see README.#1
XenGi wants to merge 1 commit into
ij0n:masterfrom
XenGi:master

Conversation

@XenGi

@XenGi XenGi commented Feb 28, 2016

Copy link
Copy Markdown

Sieht doch alles ganz hübsch aus. Konnts nich lassen noch ein paar sachen zu optimieren.

Die Module csv und configparser sind bereits in python integriert. Die musst du nicht nachinstallieren. Du musst auch nicht den code davon mit in dein repo tun. ;)

Kleiner Tipp für imports:

#!/usr/bin/env python3
# coding: utf-8

# import libs already in python
import math
import csv

# empty line; import libs not in the python std lib
import libais

# empty line; import your own libs and modules
import webcamcontrol
import nmea_util


#2 empty lines im globalen scope
class A:
    def__init__(self):
        pass

    #1 empty line in class scope
    def some_func(self):
        pass


def func_b():
    pass


if __name__ == '__main__':
    # your code here

Mit der Struktur sieht man im code besser durch. Das ist größtenteils in der PEP-8 abgebildet. An die sollte man sich möglichst halten. Ist aber kein Gesetz, eher guter Stil etc.

@ij0n

ij0n commented Feb 29, 2016

Copy link
Copy Markdown
Owner

äh sag mal warum hast du die ganzen libais-daten auch mit geändert? 403 changed files? WTF?

@XenGi

XenGi commented Feb 29, 2016

Copy link
Copy Markdown
Author

du hattest das libais mit in das verzeichnis gepackt. Aber du kannst die lib einfach in dein environment installieren, dann musst du es dort nicht mit reinpacken. Deshalb so viele gelöschte files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants