Closes #20 - transition to C++ and solve HTML templating #24

MichaelYick · 2023-03-13T10:17:19Z

MichaelYick commented

2023-03-13 10:17:19 +00:00

Java's verbosity and analness has made developing this tool not as efficient as it could be. So, I opted to (in one sitting), rewrite everything we had (and then some) while knowing none of the tools I was using.

This happened because of Vertx' poor documentation with its template system. Learning everything with poor documentation took forever, and since java is pedantic with its syntax, there was a lot more room for error, and that error held up actually writing the tool.

I swapped to C++ to just see if the issue was with my understanding of the tools or the tools I was using. I succeeded in writing all this in one sitting without blowing my brains out (although C++, at times, did make me want to throw myself out a window). So, I'm pretty sure the tools we're using are holding up development.

C++ sucks, but Java sucks way more. This rewrite has templating, SQL data insertion, SQL data reading, along with a make file for building the system (and the SQL database, assuming you're running Debian). It's not great in terms of security, but works as a proof of concept.

@t_a, I'd like to hear your opinion on this change (and if its worth going for), or if we should stick with Vertx despite its flaws.

Java's verbosity and analness has made developing this tool not as efficient as it could be. So, I opted to (in one sitting), rewrite everything we had (and then some) while knowing none of the tools I was using. This happened because of Vertx' poor documentation with its template system. Learning everything with poor documentation took forever, and since java is pedantic with its syntax, there was a lot more room for error, and that error held up actually writing the tool. I swapped to C++ to just see if the issue was with my understanding of the tools or the tools I was using. I succeeded in writing all this in one sitting without blowing my brains out (although C++, at times, did make me want to throw myself out a window). So, I'm pretty sure the tools we're using are holding up development. C++ sucks, but Java sucks way more. This rewrite has templating, SQL data insertion, SQL data reading, along with a make file for building the system (and the SQL database, assuming you're running Debian). It's not great in terms of security, but works as a proof of concept. @t_a, I'd like to hear your opinion on this change (and if its worth going for), or if we should stick with Vertx despite its flaws.

MichaelYick added the

labels 2023-03-13 10:17:19 +00:00

MichaelYick added 6 commits 2023-03-13 10:17:32 +00:00

add CI/CD pipeline d43dd509a7

rewrote to use C++ instead of Java 8d9d3df7eb

completed HTML assembler templates fc70b91b9d

c++ git ignore 9a894b00b0

remove this binary file b4fb6833b3

update readme c1478da85a

t_a added 3 commits 2023-03-20 17:09:36 +00:00

Modified Makefile potentially remove the include if it is generally not necessary. b79a513e62

-pthread is necessary

Database Manager namespace 74a707f938

Implementations:
struct representing an Requests Item with the apropriate JSON parser
a generic sql execution ideally something that is not necessary
prepared statment for the insertion of an item into Requests
prepared statment for the read out of Requests based on the ID
conversion of a pqxx::result into a wvalue JSON which is usable by the crowcpp context

Rewriting of test routes aea7bebbb5

t_a added 1 commit 2023-03-21 01:06:03 +00:00

Amendment of spec: Templates shall utilize only lower-case variable names fae75e65fc

Removal of workaround for lowercase column names
Apropriately changed the test.html

MichaelYick referenced this issue from a commit

2023-03-21 09:05:18 +00:00

Migrated to C++

MichaelYick merged commit b90f72f61a into master

2023-03-21 09:05:19 +00:00

MichaelYick deleted branch cpp

2023-03-21 09:05:49 +00:00

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Cavemanon/cavecomm#24