I think I need a lobotomy... strCH = strData.Substring(intLoop, 1) this is to extract the char of strData in each iteration, rigth? Argh! X|

I sort of agree, but rather than: MenuItemNode child = tvMenu.SelectedNode as MenuItemNode; if (child == null) return; MenuItemNode parent = child.Parent as MenuItemNode; if (parent == null) return; MenuItemNode swap = child.PrevNode as MenuItemNode; if (swap == null) return; //Code Here I would prefer: MenuItemNode child = tvMenu.SelectedNode as MenuItemNode; if (child == null) { return; } MenuItemNode parent = child.Parent as MenuItemNode; if (parent == null) { return; } MenuItemNode swap = child.PrevNode as MenuItemNode; if (swap == null) { return; } //Code Here Just to remind me if I add a statement before the return.