From: Kevin D. <ke...@tr...> - 2005-08-27 15:07:57
|
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN"> <HTML><HEAD> <STYLE type=text/css> P, UL, OL, DL, DIR, MENU, PRE { margin: 0 auto;}</STYLE> <META content="MSHTML 6.00.2900.2668" name=GENERATOR></HEAD> <BODY leftMargin=1 topMargin=1 rightMargin=1><FONT face=Tahoma size=2> <DIV><FONT face=Arial></FONT></DIV> <DIV><FONT face=Arial>Some architectural thoughts that just occured to me:</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial>Alex - instead of scattering the NULL_RECID all over the place, what do you think about adding two methods to RecordHeader:</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial>boolean isFree()</FONT></DIV> <DIV><FONT face=Arial>and</FONT></DIV> <DIV><FONT face=Arial>void setFree()</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial>?</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial>That way the mechanism that the header uses to mark itself as free can be completely encapsulated . I still think that marking the length as -1 is a good approach, but the fact that we are having such difficulty finding all the code that needs to be updated is a good indication that encpasulation has been violated.</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial></FONT> </DIV> <DIV><FONT face=Arial>- K</FONT></DIV> <DIV><FONT face=Arial></FONT> </DIV></FONT></BODY></HTML> |