New answers tagged object-oriented
2
votes
Accepted
Calculator (Java/Beginner)
For a beginner this isn't terrible, but unfortunatly from a OOP point of view it's structured quite badly.
First i'm going to concentrate on the Database class. (...
- 10.6k
1
vote
OOP methods/functions that can return a value or an exception
I suggest you don't need methods like HasValue() & HasException() hardcoded in it as they can be checked by the caller when ...
- 129
5
votes
Accepted
Making a game using python and pygame
It looks like you're actually mixing a few unrelated things here. Consider: how many of these are actually settings? That is, how many of these does it make sense for the end user to modify in some ...
- 8,842
1
vote
Test generator I made for practice
Namespacing
You have test_generator and test_dictionary. It might be better to put these in one package, so they're ...
- 323
1
vote
Wordlist formatter code
Follow the single responsibility principle
Your WordListCleaner class is doing too many things. It is in fact not just a word list cleaner but also a word list file ...
- 336
4
votes
Test generator I made for practice
test_*.py is a bad name for the file. As per convention test_*.py files are for unit tests. A better word would be 'quiz'.
There ...
- 336
1
vote
2
votes
Python Module for representation, calculus and rich comparisons between temperature scale instances
Tests
You test that a value in corresponds to an expected value out, but where did you get the calculated value out? Was it from your code? If so, it's only testing regression.
I'd prefer to see some ...
- 865
2
votes
Java program for complex math utilities
For now, I only feel like reviewing complex. I may come back for FormBus_builtin later.
Canonical java has several naming and ...
- 6,475
2
votes
Accepted
GPG class for AJAX calls
My opinion is that variable variables and variable function/method names should be avoided as much as possible (I nearly want to say: "NEVER use them"). These features often have a negative ...
- 8,369
Top 50 recent answers are included