also als ersten tipp für dich hätt ich folgendes, wenn du schon als constructor-parameter eine form übergibst, diese dann auch zu benutzen
dann kannst du nämlich die referenz auf unit2 schonmal weglassen. außerdem ne andere sache, ich frage mich inwieweit du den sinn des constructors nicht überstrapaziert hast indem du so ziemlich jede mögliche einstellung direkt im constructor aufrufst, wobei vor allem dinge wie SetBounds eigentlich dort nichts zu suchen haben... und noch was anderes, wie kommst du eigentlich auf die idee das ein TStringGrid mit der bezeichnung StringGrid1 zweifelsfrei existiert?
eigentlich sollte man ein stringgrid im constructor erstmal initialisieren...