Удалите цикл for, поскольку он бесполезен:
string text = File.ReadAllText(openFileDialog1.FileName);
richTextBox1.Text = text;
text
- это строка, которая уже содержит весь текст файла для передачи в textBox.
Выполнение:
for(int i=0, i < text.Lengt-1; i++)
richTextBox1.Text = text;
вы назначаете текст, прочитанный из файла text.Length-1
раз (Length
- количество символов в строке), и это бесполезно.