C# строка

[Nickolai47]

нужно упростить код!
по заданию найти максимальное число, число считать последовательность до букв, т.е в строке "при324мер34" должно вывести 324.
Код работает но надо как то упростить слишком уж много действий лишних...

private void button1_Click(object sender, EventArgs e)
        {
            string s = textBox1.Text;
            string ss = null;
            int max = 0; 
            int k = 0; 
            for (int i = 0; i < s.Length; i++)
                if ((s[i] >= '0') && (s[i] <= '9'))
                {
                    k = 1;
                    ss += s[i];
                    if (max < Convert.ToInt32(ss))
                        max = Convert.ToInt32(ss);
                }
                else
                {
                    ss = null;
                }
            if (k == 1)
                 textBox2.Text = Convert.ToString(max);
            else 
                 textBox2.Text = "Цифр нет";
        }
    }
}

[Serg94]

мой вариант

            string line = textBox1.Text;
            int max = 0;
            string text = "";
            bool flag = false;
            for (int i = 0; i < line.Length; ++i)
            {
                if (Char.IsDigit(line[i]))
                {
                    text += line[i];
                    flag = true;
                }
                else
                {
                    if (text != "")
                        if (int.Parse(text) > max)
                            max = int.Parse(text);
                    text = "";

                }
            }
            if (flag)
                textBox2.Text = Convert.ToString(max);
            else
                textBox2.Text = "Нет цифр";

[Stilet]

Упростим?:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            string s = "при324мер34";
            int max=0;
            foreach(Match m in new Regex(@"\d+").Matches(s)){
                int k=Convert.ToInt32(m.Value);
                max=(k>max)?k:max;
            }
            Console.Write("{0}", max);
            Console.ReadKey();
        }
    }
}

Я уверен что можно еще проще, если через LINQ сделать.

P.S. Вот собственно я оказался прав:

            string s = "при324мер34";
            var a = from Match m in new Regex(@"\d+").Matches(s) select Convert.ToInt32(m.Value);
            Console.Write("{0}", a.Max());
            Console.ReadKey();

Привыкайте, ребятки, к хорошему. )

[Rififi]

Nickolai47

Код работает но надо как то упростить слишком уж много действий лишних...

var s = "1при324мер34_1";
var max = Regex.Matches(s, @"(\d+)").OfType<Match>().Select(p => int.Parse(p.Value)).Max();

Console.WriteLine(max);