Delete the for loop because it is useless:
string text = File.ReadAllText(openFileDialog1.FileName); richTextBox1.Text = text;
text is a string that already contains all the text in the file to be sent to the text block.
Doing:
for(int i=0, i < text.Lengt-1; i++) richTextBox1.Text = text;
you assign the text read from the text.Length-1 times file ( Length is the number of characters of the string), and it is useless.
source share