Comment suivez-vous DRY avec du code en utilisant les coordonnées x-y?

Question

Comment voulez-vous obtenir l'idéal DRY rêvé dans cet exemple, dans la langue de votre choix:

drawLine(Point(0, 0), Point(w, 0)); 
int curRowY = 0; 
for(int row=0; row<rowHeights.size(); row++) { 
    curRowY += rowHeights[row]; 
    drawLine(Point(0, curRowY), Point(w, curRowY)); 
} 

drawLine(Point(0, 0), Point(0, h)); 
int curColX = 0; 
for(int col=0; col<colWidths.size(); col++) { 
    curColX += colWidths[col]; 
    drawLine(Point(curColX, 0), Point(curColX, h)); 
} 

Note: Une multitude d'ad-hoc préprocesseur macros est susceptible d'être extrêmement moins lisible & écriture si c'est parti. Peut-être, dans ce cas, vous prenez DRY trop à l'extrême?

Answer 1

La réponse est simple: les vecteurs. Par exemple.

repeatLines(Point start, Point end, Vector direction, int[] gaps) 
    { 
    drawLine(start, end); 
    for (int i = 0; i < gaps.Length; i++) 
     { 
     Vector vector = direction * gaps[i]; 
     start += vector; 
     end += vector; 
     drawLine(start, end); 
     } 
    } 

repeatLines(Point(0, 0), Point(0, w), Vector(1, 0), rowHeights); 
repeatLines(Point(0, 0), Point(h, 0), Vector(0, 1), colWidths); 

Answer 2

Néanmoins, en tant qu'exercice académique, je suis impatient de voir si quelqu'un peut trouver une solution aussi lisible que votre code actuel - mais sans la duplication apparente!

Answer 3

[. Je suis d'accord avec Stewart, mais j'appuie sur un exercice académique]

Tricky ...

D'une certaine façon, vous n'êtes pas vraiment vous répéter; vous faites deux choses semblables (littéralement et figurativement) orthogonales les unes aux autres.

Je suppose que vous pouvez faire ce qui suit, mais il est plus lisible et certainement pas plus performant:

[pseudo C#]:

void DrawGrid() 
{ 
    DrawLines(w, rowHeights, true); 
    DrawLines(h, colWidths, false); 
} 

void DrawLines(int lineLength, int[] lineSeparations, bool isHorizontal) 
{ 
    MyDrawLine(Point(0, 0), Point(lineLength, 0), isHorizontal); 
    int offset = 0; 
    for (int i = 0; i < widths.length; i++) 
    { 
     offset += lineSeparations[i]; 
     MyDrawLine(Point(offset, 0), Point(offset, lineLength), isHorizontal); 
    } 
} 

void MyDrawLine(Point startPoint, Point endPoint, bool isHorizontal) 
{ 
    if (isHorizontal) 
    { 
     SwapXAndYCoordinates(startPoint); 
     SwapXAndYCoordinates(endPoint); 
    } 

    drawLine(startPoint, endPoint); 
} 

A la réflexion, je pense que cela est juste un idiot idée ... :-)

Answer 4

Si votre grille est carré, je pense que ce qui suit pourrait fonctionner:

void drawGrid() 
{ 
    for(int i = 1, offset = 10; i <= numPoints; i++, offset += 10) 
    { 
     Point p = new Point(i * offset, i * offset); 

     drawHorizontal(p); 
     drawVertical(p); 
    } 
} 

void drawHorizontal(Point p) 
{ 
    drawLine(new Point(0, p.y), new Point(width, p.y)); 
} 

void drawVertical(Point p) 
{ 
    drawLine(new Point(p.x, 0), new Point(p.x, height)); 
}