minor to medium TODOs

[benjaminaaron]

Collecting some minor ToDo's that aren't worth opening an issue for by themselves.

• [ ] In human vs. human mode, opponents ships in setup-phase and hide both players ships in play-mode? Or just the opponents ships? In practice this means one has to look away while its the others turn :smile:
• [ ] introduce variable that determines how many shots per turn are allowed :arrow_right: allow multiple shots in the same turn as immediate rewards for hits?
• [ ] auto-naming of bots
• [ ] add ability to choose which bot type in the mode-select menu, maybe a jquery ui multiselect
• [ ] introduce boolean variable that determines whether or not touching ships are allowed

[benjaminaaron]

• [x] shadow while dragging doesn't quite work at borders sometimes
• [ ] setTimeout into fire() of AbstractBot?
• [x] better randomizer for firing randomly: randIndex in freeCells array instead of while(!ok){} loop

[benjaminaaron]

• [ ] new game object instead of reloading page on reset

[benjaminaaron]

Tip by @bbuecherl: to avoid the constant use of this.var in prototype-functions of an object, use this method:

var Obj = function() {
    var x = 1;
    return {
        y: 2,
        func: function() {
            console.log('x is ' + x); //can use x instead of this.x
        },
    }
};
var test = new Obj();
console.log(test);
test.func();
console.log('y is ' + test.y);

[benjaminaaron]

• [x] put content from info-alert into same popup window that highscore uses

[benjaminaaron]

• [x] clean highscore-div before building table to remove "can't fetch data"

[benjaminaaron]

• [ ] add duration of game in end of game display and to highscore

[benjaminaaron]

• [x] use setCellStatusesAroundShipToSpare from AbstractPlayer also for position verification in ship-setup-mode instead of the (weird) neighbourBlast method.
• [x] send ALLSHIPSDESTROYED via CellStatusMsg extraInfo instead of CellStatus

[benjaminaaron]

• [x] add overview-text from moodle-submission to README.md

[benjaminaaron]

• [x] after game is over, show remaining ships on winners board

[benjaminaaron]

• [x] move setup-done trigger from whitespace on board to button in game control bar

[benjaminaaron]

• [ ] allow parallel setup phase in human-vs-human mode

[benjaminaaron]

• [ ] Code kommentieren / Doku schreiben
• [ ] Default-Setup: Bot vs. Bot für Debugging Zwecke
• [ ] GameObserver ==> storeData() in kleinere Funktionen aufsplitten
• [ ] Pixelkoords nur noch in View
• [ ] Heatmap in DB schreiben: Schiffplatzierung auf Feld auch noch abspeichern

[benjaminaaron]

moving project and task management to external tool