Kapselung Übergabe von Objekten aus ArrayList

[arjoopy]

Heyho,
es existiert eine Klasse mit einer Menge Punkten:

public class ALotOfPoints {

    static Point p1 = new Point(-3.678,-83.689);
    static Point p2 = new Point(-73.440,81.76);
    static Point p3 = new Point(0.667,-18.607);
}

Hier schon die erste Frage: Das static vor den Points erfüllt nur den Zweck, die Fehlermeldung "Cannot make a static reference to the non-static field" zu umgehen - gibt's da vielleicht ne elegantere Lösung?
Weiterhin existiert eine zweite Klasse, die diese einzelnen Punkte einer ArrayList hinzufügt:

import java.util.ArrayList;
import java.util.List;

public class DefaultList {
   
    public DefaultList() {
       
        List<Point> pointlist = new ArrayList<Point>();
       
        pointlist.add(ALotOfPoints.p1);
        pointlist.add(ALotOfPoints.p2);
        pointlist.add(ALotOfPoints.p3);
    }
}

Mir stellt sich nun die nach ausreichend Internetrecherche immernoch nicht beantwortete Frage, wie ich jetzt aus der Mainclass auf einzelne Elemente der ArrayList zugreifen kann? Das hier erschien mir logisch, funktioniert aber jedenfalls nicht:

public class Mainclass {

    public static void main(String[] args) {
       
        System.out.println(DefaultList.pointlist.p1);   
    }
}

Danke im Voraus!

 

[Flown]

Die Frage ist so allgemein, dass ich dich fragen möchte, was möchtest du denn mit deinem Progamm erreichen/machen?
Am einfachsten wäre:

class ALotOfPoints {
  
  public static final Point P1 = new Point(-3.678d, -83.689d);
  public static final Point P2 = new Point(-73.440d, 81.76d);
  public static final Point P3 = new Point(0.667d, -18.607d);
  
  public static List<Point> defaultPoints() {
    return Arrays.asList(P1, P2, P3);
  }
}

Dann die Verwendung in deiner main:

System.out.println(ALotOfPoints.defaultPoints());

 

[Kababär]

"Cannot make a static reference to the non-static field"

Das ist der Fall in der Main, da du dich dort in einem statischen Kontext (public static void main ...) befindest.

Du kannst aber ein neues Objekt "ALotOfPoints" instanziieren.
Generell würde ich hier auf "static" komplett verzichten. Brauchst du die Klasse "DefaultListe" wirklich?
So fände ich es persönlich angenehmer.

public class ALotOfPoints {

    private Point p1;
    private Point p2;
    private Point p3;
    private List<Point> pointList;
    public ALotOfPoints(){
        pointList = new ArrayList<>();
        p1 = new Point(-3.678,-83.689);
        p2 = new Point(-73.440,81.76);
        p3= new Point(0.667,-18.607);

        pointList.addAll(p1,p2,p3);
    }


    public List<Point> getPoints(){
              return pointList;
    }
}

public class Mainclass {

    public static void main(String[] args) {
     
        System.out.println(new ALotOfPoints().getPoints().get(0));
    }
}

Je nachdem, ob sich die Punkte nochmal ändern sollen oder nicht, würde ich diese final machen (oder eben nicht).
Generell bin ich davon abgeneigt, Klassenattribute als public static zu deklarieren.

 

[arjoopy]

@Flown Obenstehendes ist nur Vorläufer für ein Programm zur Ermittlung der BoundingBox einer Punktwolke, ich als blutiger Anfänger wollte mich nur mit so einer grundlegenden Aufgabe mit dem Konzept von ArrayLists vetraut machen, da ich damit zuvor noch die gearbeitet habe.
@Kababär Die Klasse DefaultList ist in ihrer Form unumgänglich, da die Punkte aus einer anderen Quelle in dieser Form ausgegeben werden. Insofern löst eine Umstrukturierung mein Problem nicht, da aus der Mainclass auf die pointlist zugreifen muss.

Nochmal konkret das eigentliche Problem: Was ist an System.out.println(DefaultList.pointlist.p1); falsch, wenn man aus der Mainclass auf eine ArrayList in einer anderen Klasse zugreifen möchte?

 

[Kababär]

Ok. Also in DefaultList

public class DefaultList {

public DefaultList() {

List<Point> pointlist = new ArrayList<Point>();

pointlist.add(ALotOfPoints.p1);
pointlist.add(ALotOfPoints.p2);
pointlist.add(ALotOfPoints.p3);
}
}

deklarierst du eine Liste von Punkten im Konstruktor. Diese Liste müsste eigentlich als Klassenattribut deklariert werden, wenn du diese Liste außerhalb der Klasse benötigst.
Also ungefähr so:

public class DefaultList {

    public static List<Point> pointlist;
    public DefaultList() {
   
        pointList = new ArrayList<Point>();
   
        pointlist.add(ALotOfPoints.p1);
        pointlist.add(ALotOfPoints.p2);
        pointlist.add(ALotOfPoints.p3);
    }
}

Dann solltest du mit dem Sout/Sysout keine Probleme haben.
Um allerdings auf Attribute von Objekten zuzugreifen, verwendet man meistens Getter & Setter.