getting specific string error? [modified]

[maifs]

i want to fetch the characters from file for example : 1st example: // mehmood// hello ,how r u? just within comments and replace it to all caps 2nd example: /* hi mehmood ahmed */ i want to fetch the characters within sigle line comment and multi line comments . just within comments not outside the comment. -------------------------------------------------------------------------------- i am trying this code, kindly correct my code:

FileStream fs = new FileStream(path, FileMode.Open, FileAccess.Read, FileShare.Read);

        StreamReader sr = new StreamReader(fs);
        int countComment=0;
        while(sr.Peek() >= 0 )
        {
            str =sr.ReadLine();
            strTemp = new char\[str.Length\];
           #region From String method
            int count=0;
            for (int i = 0; i < str.Length; i++)
            {
                if (str\[i\] == '/')
                {
                    ++countComment;          
                    count=i;
                }
                if (char.IsLetterOrDigit(str,count+1) && countComment==2)
                {
                    strTemp2 += str.Substring(count+1).ToUpper();                       
                }                  
                
            }
            #endregion

           
            countComment = 0;
            this.textBox2.Text += strTemp2 + Environment.NewLine; ;
            this.textBox1.Text += str + Environment.NewLine;
        }
        sr.Close();
        fs.Close();

it generates an exception : System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. Parameter name: index

Maifs

modified on Monday, July 13, 2009 5:08 PM

[Christian Graus]

Issues 1 - the code is a mess 2 - it won't work 3 - it's hard to read Do you not know how to use the debugger ? The error message is telling you what is wrong, what is the issue ?

maifs wrote:

if (char.IsLetterOrDigit(str,count+1) && countComment==2) { strTemp2 += str.Substring(count+1).ToUpper(); }

Why do you use -1 AND +1 ? What if +1 is out of range ( wait, isn't that what the error message says ? ) This needs a lot of work. I'd read a C# book to learn some basics, then rewrite it using regex and the split method of the string class.

Christian Graus Driven to the arms of OSX by Vista. Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

[Luc Pattyn]

Hi, you need to debug your code, and you should do that yourself. Here are some hints: - an exception contains a message and a stack traceback indicating class and method names, file names, and line numbers (when available; use debug mode!) - when you try-catch an exception, make sure to show the ENTIRE exception, use Exception.ToString() - teach your IDE to always show line numbers in editor windows; for Visual Studio, look under menu Tools/Options/TextEditor/AllLanguages and if you must show us some code please use PRE tags (that's under the "code block" widget above the text edit box) :)

Luc Pattyn [Forum Guidelines] [My Articles]

---

The quality and detail of your question reflects on the effectiveness of the help you are likely to get. Show formatted code inside PRE tags, and give clear symptoms when describing a problem.

---