1. Warto zacząć stosować addEventListener zamiast onload in onclick
2. Kiedy pobierasz na początku nie musisz tworzyć 2 pętli choć jest to jakieś rozwiązanie. Lepiej może by było pobrać wszystkie komórki TD i dla nich wykonać pętlę w której zmienisz ich kolor w zależności od wartości w niej.
3. Nazwa funkcji funkcja nie jest zbyt ciekawa ;)
4. W celu zmiany kliknięcia na daną komórkę wykorzystaj Event delegation czyli dodaj event dla całej tabeli a potem w funkcji sprawdź czy kliknąłeś w komórkę i zmień jej kolor.