danivenk/uva_minor-programming_programming-theory

Case: RailNL, Team: Shibayama (芝山)

Portfolio

Let's first list some courses that were separated into multiple repos or were saved somewhere completely different:

• Web App Studio (project 0; project 1; project 2; project 3)
• Machine Learning (week 1; week 2; week 3; week 4; week 5; week 6)

Next, let's list all ongoing projects:

• simutrans
• Simutrans screenshot maker
• Simutrans Dataweb
• project cycling

Finally, these are some projects I'm proud of (some aren't finished yet, and some didn't even work out as I wanted):

• Programming Project
• Programming Theory
• Extreme Astrophysics - Final Project
• Master Project
• simutrans
• Simutrans screenshot maker
• Simutrans Dataweb

uva_minor-programming_programming-theory's Issues

Shebangs

al jullie files hebben een python shebang. Waarom is dit? Hoort elke file vanuit de terminal te kunnen worden aangeroepen?

Classes uitsplitsen

Nu staan alle classes in 1 bestand. Beter om deze in losse files te zetten, zodat ze rustig kunnen uitgroeien.

Lange functies

Astar (en andere algoritmes) heeft een ontzettend lange functie 'create_line'. Kijk of je daar losse functionaliteiten kunt identificeren die eventueel een losse methode zouden kunnen zijn.
Dat zou ontzettend helpen om de code te lezen en begrijpen.

Readme

De readme is nog niet af! Er otbreekt onder andere de installatie/gebruik/intro van de case.

Kijk nog even op de voorbeeldrepo!

teamwork

Een deel van de files heeft alleen Dani als auteur. Jullie leveren de code echter in als team.

Dead code

In hill_climber staan een aantal blokjes weggecommente code. Verwijder deze, of geef ze een comment over waarom ze zijn weggecoment voordat je de final version inlevert