Get specific words from string c#

Question

I am working on a final year project. I have a file that contain some text. I need to get words form this file that contain "//jj" tag. e.g abc//jj, bcd//jj etc.

suppose file is containing the following text

ffafa adada//bb adad ssss//jj aad adad adadad aaada dsdsd//jj dsdsd sfsfhf//vv dfdfdf

I need all the words that are associated with //jj tag. I am stuck here past few days. My code that i am trying

  // Create OpenFileDialog
        Microsoft.Win32.OpenFileDialog dlg = new Microsoft.Win32.OpenFileDialog();

        // Set filter for file extension and default file extension
        dlg.DefaultExt = ".txt";
        dlg.Filter = "Text documents (.txt)|*.txt";

        // Display OpenFileDialog by calling ShowDialog method
        Nullable<bool> result = dlg.ShowDialog();

        // Get the selected file name and display in a TextBox
        string filename = string.Empty;
        if (result == true)
        {
            // Open document
            filename = dlg.FileName;
            FileNameTextBox.Text = filename;
        }

        string text;
        using (var streamReader = new StreamReader(filename, Encoding.UTF8))
        {
            text = streamReader.ReadToEnd();
        }

        string FilteredText = string.Empty;

        string pattern = @"(?<before>\w+) //jj (?<after>\w+)";

        MatchCollection matches = Regex.Matches(text, pattern);

        for (int i = 0; i < matches.Count; i++)
        {
            FilteredText="before:" + matches[i].Groups["before"].ToString();
            //Console.WriteLine("after:" + matches[i].Groups["after"].ToString());
        }

        textbx.Text = FilteredText;

I cant find my result please help me.

Before I try to help with this, can I just ask you to confirm that the requirements of the project allow you to ask for help from outside sources like SO? — John Clifford
– John Clifford, Commented Jan 14, 2016 at 16:09

Ian · Accepted Answer · 2016-01-14 16:24:47Z

7

With LINQ you could do this with one line:

string[] taggedwords = input.Split(' ').Where(x => x.EndsWith(@"//jj")).ToArray();

And all your //jj words will be there...

answered Jan 14, 2016 at 16:24

Ian

30.9k20 gold badges76 silver badges112 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Equalsk Over a year ago

Nice! You've reminded me to go and read my "LINQ for Dummies" book again ;-)

Equalsk Over a year ago

I just mean I'm jealous that it didn't occur to me to use LINQ as I've been learning to use it recently :-)

Ian Over a year ago

@Equalsk ah, I see.. ;) so that is what you mean. lol, no worries I will still vote your answer up then. :D This way, you won't feel your answer go wasted. :) your answer is actually easier to understand for new programmer than LINQ. I mean: it is still a nice answer. :D

Shahbaz Shafiq Over a year ago

much helpful. Thanks

Shahbaz Shafiq Over a year ago

I have stuck again because text that I have in the file is in other languages like Urdu and Arabic, Not in English, In both of your code failed to get specific words because it can't split

|

Equalsk · Accepted Answer · 2016-01-14 16:14:58Z

3

Personally I think Regex is overkill if that's definitely how the string will look. You haven't specified that you definitely need to use Regex so why not try this instead?

// A list that will hold the words ending with '//jj'
List<string> results = new List<string>();

// The text you provided
string input = @"ffafa adada//bb adad ssss//jj aad adad adadad aaada dsdsd//jj dsdsd sfsfhf//vv dfdfdf";

// Split the string on the space character to get each word
string[] words = input.Split(' ');

// Loop through each word
foreach (string word in words)
{
    // Does it end with '//jj'?
    if(word.EndsWith(@"//jj"))
    {
        // Yes, add to the list
        results.Add(word);
    }
}

// Show the results
foreach(string result in results)
{
    MessageBox.Show(result);
}

Results are:

ssss//jj
dsdsd//jj

Obviously this is not quite as robust as a regex, but you didn't provide any more detail for me to go on.

answered Jan 14, 2016 at 16:14

Equalsk

8,2643 gold badges47 silver badges71 bronze badges

6 Comments

lex82 Over a year ago

In how far is Regex overkill?

Equalsk Over a year ago

Because it's complex to read and understand unless you know what you're doing, which it seems OP does not. The above code is more of a beginners approach to get the same results and is much easier to understand, IMHO. I'm not saying Regex is slower, if that's what you mean.

PaulF Over a year ago

My solution was similar but using Linq to get the same result results = words.Where(s => s.Trim().EndsWith(@"//jj")).ToArray();

Equalsk Over a year ago

I love a good bit of LINQ, I just wanted to keep it nice and simple because, well, I'm simple!

Shahbaz Shafiq Over a year ago

I have stuck again because text that I have in the file is in other languages like Urdu and Arabic, Not in English, In both of your code failed to get specific words because it can't split.

|

James Boyden · Accepted Answer · 2016-01-14 16:15:28Z

2

You have an extra space in your regex, it assumes there's a space before "//jj". What you want is:

 string pattern = @"(?<before>\w+)//jj (?<after>\w+)";

answered Jan 14, 2016 at 16:15

James Boyden

4544 silver badges8 bronze badges

Comments

lex82 · Accepted Answer · 2016-01-14 16:11:39Z

0

This regular expression will yield the words you are looking for:

string pattern = "(\\S*)\\/\\/jj"

A bit nicer without backslash escaping:

(\S*)\/\/jj

Matches will include the //jj but you can get the word from the first bracketed group.

answered Jan 14, 2016 at 16:11

lex82

11.4k2 gold badges48 silver badges72 bronze badges

1 Comment

SeeSharp Over a year ago

If abc abc//ss abc//gg //jj should return 0 matches then (\w+)\/\/jj works also.

Collectives™ on Stack Overflow

Get specific words from string c#

4 Answers 4

6 Comments

6 Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

6 Comments

6 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related