Эту строчку я не могу найти   

Судя по всему я её успела отредактировать как

Класс Grids я скомпелировала так 


Где-то закралась ошибка потому что окошко выходит без решётки и не по центру экрана . Закрывается нормально но есть Terminal window с текстом

На кнопке старт я изменила вызов конструктора с  new Grids("Points", 800, 800, 20, 20).setVisible(true);    на new Grids( );

DrawablePoints выглядят теперь так 
И остался ещё класс BattleField который нам не нужен но там прописан метод
 который теперь надо вызывать прописать в GridCanvas/

Эта плоха. Быть она должна.

Вот после этой строчки у вас был там цикл инициализации массива в котором я видел ошибку... Если вы ее уже поправили - тогда ладно.


Одно замечание:
canvas.setSize(rows*sellSize , cols*sellSize);
можно убрать. Для JPanel размер устанавливается через setPreferredSize(), и делать это нужно в конструкторе GridsCanvas.


Видимо ему не нравится extends класса GridsCanvas. Пока там не будет extends JPanel ошибка так и останется.


Норм. 

Метод makePoints() нужно перенести из BattleField в GridsCanvas. Класс BattleField можно после этого убрать совсем.

Нашла ! Эта строчка была в классе BattleField который мы уже не используем .Там мы инициализировали массив точек в методе  makePoints
 Теперь мы  уже не можем это делать потому что класс DrawablePoint  в настоящий момент не содержит такого конструктора . Там теперь конструктор 
 

нужно видимо ввести  соординаты x,y как аргументы  , либо  сделать два разных конструктора DrawablePoint  . Один только с цветом а второй только с координатами .

class GridsCanvas extends JFrame  было ещё вчера .. Сегодня ошибка повторилась . 
"adding a window to a container" может означать что мы добавляем add(canvas); неправильно с точки зрения компайлера ..?

extends JPanel. JFrame - это окно, нельзя добавить окно в окно.

Конструктор точки пусть остается как есть. Цикл инициализации массива нужно модифицировать - передавать точке вместо координат цвет. Кроме того x и y мы будем передавать точке во время отрисовки.

C extends JPanel всё получилось замечательно.Сообщение о ошибке изчезло и решётка вышла нормально . 

Ну вот, дальше собственно берем цикл

и вставляем его в GridsCanvas в метод paintComponent(). Точки должны рисоваться цветом, который вы передаете им при создании.
И не удивляйтесь если точки окажутся неправильно расположены на сетке.

Я изменила цикл инициализации массива   и вызвала paint() для каждой точки 
 на строчке  компайлер не может найти переменную cellSize которая является public членом того же класса. ..
 

Это сообщение отредактировал(а) Qawra - 13.1.2009, 18:34

Qawra, 
[


опечатка. должно быть cellSize

Угу.
И переменные rowHt и rowWid в методе paintComponent() это тот же cellSize, так что от них можно избавиться.

В остальных местах где эти переменные используются меняем их на cellSize.

Кстати, переменные-члены класса у нас везде должны быть private, потому что пользуется ими только сам класс. Да и вообще разумно спрятать переменные класса от остальной программы.

Если отключить  то надо как-то модифицировать  paintComponent потому что
без них ничего нарисовать в данный момент нельзя.
  

и дальше в том же духе

скомпелировалось так  без rows и cols в аргументах drawLine не компелировалось поэтому я их там прописала. Как вы и предупреждали  рисует косые линии  там где предполагается решётка. Есть сообщение о NullPointerException в 34 строкe:
 Array у нас инициализирован как private  член класса  

Вот этого я не ожидал   


А почему именно их?

Давайте разберемся:
Лини рисуются при помощи drawLine() + Линии рисуются неправильно = Неправильно используется метод drawLine()

drawLine() принимает четыре аргумента:
drawLine(x1, y1, x2, y2)
x1 и y1 это координаты начала линии, x2 и y2 - координаты ее конца. Нам, чтобы начертить сетку, нужно нарисовать вертикальные и горизонтальные линии на поле

У горизонтальных линий y1 и y2 должны быть одинаковые, иначе они не будут горизонтальными. Вообще любая точка на горизонтальной прямой имеет ту же координату Y. У вертикальных линий должны быть одинаковые x1 и x2.
Дальше - первую горизонтальную линию нам надо начертить в самом верху поля, все остальные - ниже предыдущей на cellSize. Т.е. координата Y кратна cellSize и равна cellSize*номерСтроки; у нас в цикле уже есть переменная i которая считает строки и содержит номер текущей строки. Записываем:

Теперь x1 и x2. Каждая линия должна идти от одного края поля до другого, для этого один икс должен быть равен нулю(у левого края поля x=0), а другой равен ширине поля (т.е. cols*cellSize); вписываем иксы:


Аналогично для вертикальных линий:
X начала и конца каждой линии одинаковый.
X кратен cellSize и равен cellSize*номерСтолбца (вместо номера столбца опять используем переменную i).
Y начала линии - ноль.
Y конца линии - расстояние от верха до низа поля = высота поля = количествоСтрок*cellSize

===========================

В этой строке:

такое исключение может возникнуть, только если points[x][y] не содержит точки.
Скорее всего что-то случилось с циклом инициализации массива points.
Вообще с этим массивом мы много чего делаем, напомню самые важные моменты:

Когда мы создаем массив, в его ячейки раскладывается значение null. Если не инициализировать массив, null так там и останется. Если после этого попытаться что-то сделать с элементом в массиве, например

возникает NullPointerException.

Когда инициализация у вас будет работать как надо, на экране сразу появятся точки.

Зашла в интернет утром перед школой и убегаю ..В школе попробую переделать . У меня часовой пояс на два часа раньше чем московкий .Поэтому я иногда не успеваю 

Решётка теперь рисуется нормально.
Цикл инициализации массива points ...
Мне кажется чего - то нехватает в классе  DrawablePoint ..
Но начнём по порядку :
В классе GridsCanvas если мы инициализируем как член класса 
 а затем указываем его размер  компайлер на этой строчке требует идентифайер . Если сделать всё в одну строчку  то это компелируется и мы переходим к созданию и раскладыванию точек по ячейкам .У нас есть для этого  метод makePoints но конструктору к которому он нас отсылает  нужен аргумент color.
Если мы передадим цвет при создании точек (не знаю правда какой реально  это цвет с точки зрения компайлера ..наверно firstPlayerColor..) то сообщение об ошибке изчезает 

    

Осталась нарисовать точки...Для этого нам нужны аргументы. В классе  DrawablePoint у нас прописано почему- то даже два метода
 
когда я пытаюсь нарисовать точки как  класс компелируется но при запуске программы выходит NullPointerException на этой строчке. Я думаю  нам надо переделать метод для рисования . Может быть проблема в Graphics g ..