Как выдрать из строки нужные символы зная позиции?

Question

[Envywewok]

Вроде не трудная задача, но что то я не понял как лучше сделать. Вот пример названия файла - N54E026.hgt
Формат названия всегда одинаковы - 1 буква / 2 цифры / 1 буква / 3 цифры / + расширение. (а значит и расположение нужных символов тоже).

Задача в том что повыдергивать все эти позиции (1 буква / 2 цифры / 1 буква / 3 цифры) в 4 разные переменные. Как это лучше сделать?

Answer 1

[Роман]

"Лучше" бывает разное: скорость работы, затраты памяти, понятность кода.

По скорости работы и затратам памяти этот вариант самый оптимальный:

static void Parse(string name = "N54E026.hgt")
{
    char c1 = name[0];
    int n1 = (name[1] - '0') * 10 + (name[2] - '0');
    char c2 = name[3];
    int n2 = (name[4] - '0') * 100 + (name[5] - '0') * 10 + (name[6] - '0');
}

Comments

[Envywewok]

Как это работает? ) - "- '0') * 100".

[#]

"Лучше" бывает разное: скорость работы, затраты памяти, понятность кода.

в данном примере точно проигрывает читаемость ))

нашел ошибку в замерах.. тем не менее - обычно больше в цене легкая сопровождаемость
а выигрыш становится серьезным где то на 10 миллионах парсингов ))
(сам был когда то фанатом низкоуровневых оптимизаций, по тому не вытерпел

и проверил ))

using System;
using System.Diagnostics;
using System.Text;

namespace parse_fn_bench
{
    static class Program
    {
        static (char, int, char, int) Parse1(this string fn)
        {
            var c1 = fn[0];
            if (!int.TryParse(fn.Substring(1, 2), out var n1))
                throw new Exception("not integer value");
            var c2 = fn[3];
            if (!int.TryParse(fn.Substring(4, 3), out var n2))
                throw new Exception("not integer value");
            return (c1, n1, c2, n2);
        }

        static (char, int, char, int) Parse2(this string name)
        {
            char c1 = name[0];
            int n1 = (name[1] - '0') * 10 + (name[2] - '0');
            char c2 = name[3];
            int n2 = (name[4] - '0') * 100 + (name[5] - '0') * 10 + (name[6] - '0');
            return (c1, n1, c2, n2);
        }

        static string[] bulk = default;

        static void fill(int n = 10)
        {
            bulk = new string[n];
            var rnd = new Random();
            for (var i = 0; i < bulk.Length; i++)
                bulk[i] = (new StringBuilder()
                    .Append((char)('A' + (char)rnd.Next(0, 26)))
                    .Append(rnd.Next(10, 99).ToString())
                    .Append((char)('A' + (char)rnd.Next(0, 26)))
                    .Append(rnd.Next(100, 999).ToString())
                    .ToString());
        }

        static void test(int n = 10)
        {
            if (n > bulk.Length)
                n = bulk.Length;
            for (var i = 0; i < n; i++)
                Console.WriteLine($"file name '{bulk[i]}'\n"
                    + $"\t test parsings {bulk[i].Parse1()}\n"
                    + $"\t parsing result is equal {bulk[i].Parse1() == bulk[i].Parse2()}");
        }

        static void Main(string[] args)
        {
            // для проверки идентичности парсинга на малом массиве - 10 значений
            //fill();
            //test();

            var n = 100_000;
            // массивы что бы убрать влияние динамического создания объектов
            var r1 = new object[n];
            var r2 = new object[n];

            var sw = new Stopwatch();
            sw.Start();

            fill(n);
            var tsf = sw.Elapsed;
            sw.Reset();
            sw.Start();

            for (var i = 0; i < n; i++)
                r1[i] = bulk[i].Parse1();
            var ts1 = sw.Elapsed;
            sw.Reset();
            sw.Start();

            for (var i = 0; i < n; i++)
                r2[i] = bulk[i].Parse2();
            var ts2 = sw.Elapsed;
            sw.Stop();

            // весь вывод выносим за пределы замера
            // вывод r1.cnt/r2.cnt что бы сборщик мусора не успел возбудиться ))
            Console.WriteLine($"{tsf.Hours:00}:{tsf.Minutes:00}:{tsf.Seconds:00}.{tsf.Milliseconds:000}");
            Console.WriteLine($"{ts1.Hours:00}:{ts1.Minutes:00}:{ts1.Seconds:00}.{ts1.Milliseconds:000} r1.cnt {r1.Length}");
            Console.WriteLine($"{ts2.Hours:00}:{ts2.Minutes:00}:{ts2.Seconds:00}.{ts2.Milliseconds:000} r2.cnt {r1.Length}");
        }

    }
}

[#]

Envywewok,

Как это работает?

https://duckduckgo.com/?q=c%23+%D1%8F%D0%B2%D0%BD%...

в данном случае (name[1] - '0') неявно преобразуется к int. что бы понимать и использовать такие приемы, надо накопить некоторую базу программирования вообще, и знания конкретного языка в частности ))

и повторюсь - выигрыш в скорости очень сомнительный. выигрыш хороший и стабильный )) но вот читаемость кода (в корпоративе - сопровождаемость кода) сейчас в большей цене. низкоуровневая оптимизация по скорости, выполняется только для очень дорогостоящих проектов

[#]

Envywewok,

Как это работает?

name[1]это строго говоря число. но интерпретируется как символ. если name[1] '0'..'9' предложенный алгоритм сработает верно. если name[1] буква .. он тоже сработает, вот только вы получите не корректное значение.. но программа вам не сообщит об ошибке

[Envywewok]

А все теперь дошло зачем умножать на 10 и 100) Спасибо помог ваш коммент. Интересные приемчики, сам бы не пришел к этому)

[#]

Envywewok, см камент про "проверку", я туда код довесил, может пригодится ))
(оба приема реализованы ))

Answer 2

[#]

...
  var fn = "N54E026.hgt";
  var c1 = fn[0];
  Console.WriteLine($"первый символ {c1}");
...
  var d2 = fn.Substring(1, 2);
  Console.WriteLine($"фрагмент из 2х символов, начиная со второй позиции {d2}");
  if(int.TryParse(d2, out var n2))
    Console.WriteLine($"численное значение d2 {n2}");
...

Answer 3

[Рональд Макдональд]

string one_char = name[0];
string two_numbers = name[1] + name[2];
string another_one_char = name[3];
//и так далее

Answer 4

[Владимир Ковалёв]

Используйте метод ToCharArray() для перевода строки в массив символов. Например:

string fileName = "N54E026.hgt";
    char[] chars = fileName.ToCharArray();
            
    var first = chars[0];
    var second = chars[1] + chars[2];
    var third = chars[3];
    var fourth = chars[4] + chars[5] + chars[6];

Comments

[#]

в данном случае избыточно. строка и так прекрасно отдает символы по индексу. здесь приведение к массиву символов - лишняя операция по созданию объекта и копированию данных

преобразование к массиву, целесообразно, только когда есть необходимость изменять порядок символов в массиве, или что то подобное

строка в дотнет вообще (и сишарп в частности) - неизменяемый тип. принято создавать новые экземпляры строк при любом изменении. кстати, это отчасти уже элементы функционального стиля - алгоритм любой сложности над символами строки, может быть выполнен без приведения к массиву. и будет более читаемым/надежным