crypto icon indicating copy to clipboard operation
crypto copied to clipboard

Coding-Style initiative

Open tassadarius opened this issue 4 years ago • 1 comments

Coding-Style initiative

Hello everyone, a I want to start a discussion / project to improve our JCrypTool code quality.

Why

JCryptool has specified Coding conventions and tools to apply them. Our automated checking tool currently counts 105.528 violations against it as can be seen in this screenshot.

image

This leads to code which is: inconsistent, hard-to-read and therefore hard-to-understand, more error prone and painful to maintain (you name it). The goal is to improve the quality of existing code, as well as ensure a certain code quality for newly written parts.

What

We as core developers should begin to enforce our coding guidelines in the code we produce. The tools are there (checkstyle, auto-formatter etc.), we just have to apply them properly. We should also try to enforce these conventions before new code is adopted from external contributors.

Possible steps.

I know it's not the nicest topic, but everybody is happier with cleaner code.

  • [ ] Discuss whether we are happy with our coding guidelines or if it can be improved.
  • [ ] Tell people (we to ourselves, external contributors) explicitly about these tools and ask them to use them.
  • [ ] Place the wiki page for it more prominently
  • [ ] Start a refactoring initiate to make more code conform to our coding conventions.
  • [ ] (Far Future goal) enforce a checkstyle check at build level and reject non-conform code.

HTML coding conventions

We currently have rules only for Java in place. It would be cool to cure our inconsistent and bad HTML code as well. There are HTML formatters available and standards as well.

What do you think? Please comment and/or let's discuss this in a meeting.

tassadarius avatar Mar 15 '21 20:03 tassadarius