Skip to content

Meine Einsendung#5

Open
leofidus wants to merge 1 commit into
consistec:masterfrom
leofidus:master
Open

Meine Einsendung#5
leofidus wants to merge 1 commit into
consistec:masterfrom
leofidus:master

Conversation

@leofidus

@leofidus leofidus commented May 5, 2013

Copy link
Copy Markdown

Da es schon so viele Einsendungen in Java gibt, hier meine Version in C.
Für eine Cuda Variante hatte ich dann doch keine Zeit mehr.

@leofidus

leofidus commented May 7, 2013

Copy link
Copy Markdown
Author

You didn't require unit test. To quote, you asked us to send "all other code you wrote supporting the implementation", but you didn't require us to write any supporting code.

Unit tests are a good way to ensure that code quality stays at a constant level and no new bugs are introduced, but nobody will never touch this code again, so that's not an issue. For the limited testing during development and the tests before "shipping", manual tests were much faster than writing unit-tests and also covered more code. Therefore I saved myself a lot of time and probably reduced the code size by factor two.

Of course in a corporate environment, there is no throw-away code, so unit tests are (almost) always useful. I understand your intention, and suspected that that was the case. However every tool is very specific for it's job, and I didn't feel that unit-tests were the right tool for this job. Even though we developed rotary excavators, a shovel is often the better tool for a job.

(you are free to disagree, but those are my reasons)

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