can't see a thing running that code. fonts missing, data missing, black screen...

move all the loadStrings() into setup. that's probably the cause of slowness. that and the spheres - look at sphereDetail() or use box()

try and narrow down which of the 6 different line drawing things is the problem and only post that. 350 lines is a bit much to ask people to look through.

I just want to know what a if() loop is...

@kooogy: I fixed the font missing and data missing.

i can't get peasy cam working on linux so can't run it. tried something else but didn't see anything.

the problem:

1.     points.add(V);
2.     if(points.size() > 1) {

using this smaller set

1. 40.632788°N - 73.947612°W - Brooklyn College / Flatbush Avenue
2. 40.639912°N - 73.94846°W - Newkirk Avenue
3. 40.645122°N - 73.94906°W - Beverly Road

you read a line (Brooklyn) from the file and parse it. you then add it to points
points size is 1, so no drawing.

read another line (Newkirk) and add it to points
size is 2 so you draw a line between point 1 (Newkirk) and 0 (Brooklyn)

read another line (Beverley) and add it to points
size is 3 so you draw a line from point 2 (Beverley) to point 1 (Newkirk)

ok, end of file.

next draw loop:

you read a line (Brooklyn again) and add it to points
points is size 4 so you draw a line between point 3 (Brooklyn) and point 2 (Beverley)

and there's your closed loop. it's because you're not clearing points array between draw loops.

that said, you have bigger problems with the code - you are still calling loadStrings inside the draw loop for instance. and there's too much duplicate code - which is a sign you need a class, or at least a method.

Great thanks for your explanation.

See code above.

Kooogy,

1. import processing.opengl.*;
2. //import peasy.*;
3. //PeasyCam cam;
4. 
   
5. Station[] aLine;  // array of stations for aLine
6. 
   
7. void setup() {
8.   size(800,600,OPENGL);  // was 1200 x 800. my laptop isn't that big
9. //  cam = new PeasyCam(this, 2000);  // no peasycam here
10. //  cam.setMinimumDistance(500);
11. //  cam.setMaximumDistance(1000000);
12. 
    
13.   // read aLine stations from file
14.   String[] lines = loadStrings("a_line.txt");
15.   aLine = new Station[lines.length];    // allocate Stations array
16.   for(int i = 0; i < lines.length; i++) {
17.     aLine[i] = new Station(lines[i]);  // initialise this Station
18.   }
19. 
    
20.   background(0);
21.   smooth();
22.   frameRate(15);
23. }
24. 
    
25. void draw() {
26.   background(0, 0, 0, 75);
27.   lights();
28.   translate(-width * 2, -height / 2);
29.   lights();
30. 
    
31.   // draw line from one station to the next (stop one before end)
32.   for (int i = 0 ; i < aLine.length - 1 ; i++) {
33.     line(aLine[i].x, aLine[i].y, aLine[i + 1].x, aLine[i + 1].y);
34.   }
35.  
36.   noLoop();  // no need to redraw static scenes
37. }
38. 
    
39. // station class
40. class Station {
41.   float x, y;   // no point using PVector if you then just use v.x and v.y
42.   String name;  // there was a name in earlier example 8)
43.  
44.   // default constructor takes a line from the file
45.   // works out x and y (and name) from this line
46.   Station(String s) {
47.     // parse line (look into splitTokens())
48.     int pos = s.indexOf('°');
49.     String sx = s.substring(0, pos);
50.     s = s.substring(pos + 1);
51.     pos = s.indexOf('-');
52.     String niets = s.substring(0, pos);
53.     s = s.substring(pos + 1);
54.     pos = s.lastIndexOf('°');
55.     String sy = s.substring(0, pos);
56.     float xTmp = float(sx.trim());
57.     float yTmp = float(sy.trim());
58.     x = ((xTmp - 40) * 10000) - 5000;  // these are values for aLine
59.     y = ((yTmp - 73) * 10000) - 9000;  // think they were different for other lines
60.                                        // if so move them to the draw loop and keep Station common
61.     // name parsing here
62.   }
63. }

ok, i don't have peasycam so i can't tell if this works. the line coords look ok when i print them.

note Station class, you can use this for all your stations although i think the multipliers at the bottom were different per line. in which case store xTmp, yTmp in x and y and apply magnification in the draw loop (which isn't ideal but keeps Station useable for all stations on all lines)

your a_line[] is local to a_line() which is fine if you're reading and using the data in the same place. except it isn't because that way you have loadStrings() in the draw(). if you move the data loading outside the method then you have to make the array global. (i've bypassed this by having a global stations array...)

btw, having an array called a_line and a method called a_line will just confuse yourself.

Thanks so much for your help!

i don't have easy cam and didn't want the thing to keep looping because i was printing lots of debug information with every frame. you can remove it.

(although personally i'd leave it in there and add

void mouseMoved() {      // or maybe mouseDragged()
  redraw();
}

so that it doesn't redraw unless you're moving the mouse - no point eating up cpu drawing identical frames)

okay thanks for the explanation! 

you can access members of a class using variableName.memberName (subject to some caveats)

>     line(aLine[i].x, aLine[i].y, aLine[i + 1].x, aLine[i + 1].y);

like i did when drawing the lines...

Sorry for the late response!

I think I know where the problem is.

String names on line 3 is a member variable to hold the name

String station_names on line 16 is local to the constructor, you don't need it (plus it'll disappear as soon as the constructor ends, leaving you with nothing)

just change line 16 to

names = s.substring(pos+1);

(compare with x and y, they are defined as member variables and not in the constructor, but they are set there and available from the outside)

Great thanks!

those rotations are based on rotating around the origin. to rotate around an arbitrary point you need to

a) translate the object to the origin
b) rotate
c) translate back

so, inside the pop/push you'd have

translate(-aLine[i]x, -aLine[i].y);
rotateX(PI);
translate(aLine[i]x, aLine[i].y);
text("words", 0, 0);

i think. i can't test this as i've said before. the 0, 0 in the text is because you've just translated to the correct position anyway. you might be able to combine these two statements. and the 'origin' of your string may not be where you think it is (bottom left, i think)

tutorial on 2d transforms
http://processing.org/learning/transform2d/

1. PFont font;
2. float angle = 0;
3. 
   
4. void setup() {
5.   size(400, 400);
6.   font = createFont("Arial Black", 32);
7.   textFont(font);
8. }
9. 
   
10. void draw() {
11.   background(255);
12.   stroke(255, 0, 0);
13.   fill(0, 255, 0);
14.   ellipse(0, 0, 5, 5); // draw an ellipse the origin
    
15.  
16.   angle += .01;
17.   pushMatrix();
18.   translate(200, 200);
19.   rotate(angle);
20. //  translate(-200, -200);
21.   text("acd", 0, 0);      // you can combine the translate(-200, -200); text("acd", 200, 200); as text("acd", 0, 0);
22.   popMatrix();
23. }

ok, the thing i forgot is that transformations are applied in reverse order...

Hi Kooogy,