bigboy | A work-in-progress GameBoy emulator | Game Engine library

The bug here is actually just in the condition of your while loop:

while(this.players.length !== 0 || this.enemies.length !== 0)

Your condition says to loop while there is at least one player OR there is at least one enemy. So as long as ONE of the arrays is not empty, it will continue to loop.

But when you first create this.players and this.enemies, they start at different sizes. Then when you remove one entry from each array, eventually one of the arrays is empty before the other.

Then your code has var randomEnemyIndex = Math.floor(Math.random() * this.enemies.length); which will evaluate to 0 when the array is empty. And when you do this.enemies[0], it returns undefined. When undefined is passed into weapon.attack like weapon.attack(randomPlayer, undefined), then it tries to call applyDamage(dmg) on the undefined, which throws your exception.

If you modify your code to have the following console logs, you will see the issue:

You are very close and on the right track. You should initialize highest to Integer.MIN_VALUE and lowest to Integer.MAX_VALUE. Then when you find one lower or higher, reset those values to the newer ones and continue until no more input.

Continue reading in values and names, setting appropriate values until you're finished. Then print the results.

Here is a sample.

Make your firebase call like this -

I get an incorrect result, of 0. Why is this (?)

Insufficiently wide integer math.

2+ changes needed

Is long wide enough?

long may only be 32-bit, then long bigboy = 600851475143; is insuffceint to save the constant. recommend long long

Since you went with OOP as a paradigm for developing your user interfaces, you should have fully utilised all the benefits it has to offer. However, something tells me you're not quite familiar with classes, objects, instance variables and other OOP concepts.

1. I suggest you design your class to actually inherit from tkinter.Tk, since it makes sense that state and behavior of the user interface parts (widgets) are grouped into separate objects, which are all in turn in a separate namespace.

2. Apart from that, I also suggest you get rid of from tkinter import *, since you don't know what names that imports. It can replace names you imported earlier, and it makes it very difficult to see where names in your program are supposed to come from. Use import tkinter as tk instead, which is kind of an idiom here in Python.

3. You might want to read what PEP8 says when it comes to naming conventions, spaces around keyword arguments, and the rest there is to the Python coding style guide in general:

4. You had other minor issues in the code - for instance, you were doing object equality checking between two different data types, small typos, etc.

In any case, here's my view how you could slightly rewrite the code above:

Please try the following command with a single CSV file to see if it extracts the fields you want: