Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 5.00/7: Рейтинг темы: голосов - 7, средняя оценка - 5.00
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47

Улучшить код разбора аргументов

08.11.2016, 18:24. Показов 1392. Ответов 11
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Разрабатываю программу, и не хочу использовать костыли и говнокод, но делая одну функцию я встал в тупик:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public static string[] ParseArgs (string full)
        {
            if(!full.Contains(" "))
                throw new ArgumentException("Args not found", "full");
            var args = new Dictionary<int, string>();
            var chars = full.ToCharArray();
            bool nospace=false;
            int index=0;
            for(int i=0; i<chars.Length; i++)
            {
                var ch = chars[i];
                if(ch==' ' && !nospace)
                    index++;
                else if(ch=='"')
                    nospace=!nospace;
                else
                    args[index]+=ch;
            }
            if(nospace)
                throw new Exception("You must close the quotation marks");
            var resp = string[args.Count];
            for(int i=0; i<args.Count; i++)
            {
                resp[i] = args[i];
            }
            return resp;
        }
Функция парсит строку типа: command "argument1 and spaces" argument2 3
И возвращает массив: "command", "argument1 and spaces", "argument2", "3"

Как можно оптимизировать этот код?
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
08.11.2016, 18:24
Ответы с готовыми решениями:

Как улучшить код?
В Main() код для вызова функции У меня есть много классов которые должны работать через Thread например: Thread s = new...

Как улучшить код?
Как улучшить? static double Power(double x, int y) { double b = x; double c; ...

Улучшить код с кучей ветвлений
Доброго времени суток, форумчане! Есть вариант написания подобной логики без огромного колличества Ifов и без мега switchей? if (Stime...

11
08.11.2016, 18:28

Не по теме:

Можно посмотреть на код готового https://github.com/gsscoder/commandline

0
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47
08.11.2016, 18:37  [ТС]
Там вообще целая система, а мне нужна небольшая функция.
0
 Аватар для LeniumSoft
1454 / 847 / 150
Регистрация: 06.06.2012
Сообщений: 2,370
08.11.2016, 18:56
Цитата Сообщение от elektro-volk Посмотреть сообщение
а мне нужна небольшая функция
https://github.com/OrchardCMS/... eParser.cs

возьми этот класс и просто вызови у него Parse.

Вроде он должен выдать именно твои три строчки.
0
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47
08.11.2016, 19:01  [ТС]
Чтобы сократить костыльную функцию, нужно создать целый класс?
0
 Аватар для kesean
292 / 291 / 108
Регистрация: 04.09.2010
Сообщений: 638
08.11.2016, 19:02
elektro-volk, а регулярку не рассматривал? Как-то так:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;
 
namespace ConsoleApplication3
{
    class Program
    {
        static void Main(string[] args)
        {
            string s = "command "argument1 and spaces" argument2 3";
            string[] parts = Regex.Split(s, " (?=(?:[^"]*"[^"]*")*[^"]*$)");
            foreach(string part in parts)
                Console.WriteLine(part);
            Console.ReadLine();
        }
    }
}
почему- то не экранируются кавычки
1
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47
08.11.2016, 19:06  [ТС]
Тут только 3 переменные?
0
 Аватар для kesean
292 / 291 / 108
Регистрация: 04.09.2010
Сообщений: 638
08.11.2016, 19:10
Цитата Сообщение от elektro-volk Посмотреть сообщение
Тут только 3 переменные?
Не совсем понял вопрос
0
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47
08.11.2016, 19:12  [ТС]
Команды у меня могут любыми
c arg1 arg2 "arg 3"
c arg "arg zachem nujni"
c arg arg arg arg arg arg arg arg
c "slishkom mnogo argumentov"
0
 Аватар для kesean
292 / 291 / 108
Регистрация: 04.09.2010
Сообщений: 638
09.11.2016, 09:00
Ты попробуй сперва.
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
        static void Main(string[] args)
        {
            string full="command \"argument1 and spaces\" argument2 3";
            Console.WriteLine(full);
            foreach(string arg in ParseArgs(full))
                Console.WriteLine("\t" + arg);
 
            full="c arg1 arg2 \"arg 3\"";
            Console.WriteLine(full);
            foreach(string arg in ParseArgs(full))
                Console.WriteLine("\t" + arg);
 
            full="c arg \"arg zachem nujni\"";
            Console.WriteLine(full);
            foreach(string arg in ParseArgs(full))
                Console.WriteLine("\t" + arg);
 
            full="\"c arg arg arg arg arg arg arg arg\"";
            Console.WriteLine(full);
            foreach(string arg in ParseArgs(full))
                Console.WriteLine("\t" + arg);
 
            full="c \"slishkom mnogo argumentov\"";
            Console.WriteLine(full);
            foreach(string arg in ParseArgs(full))
                Console.WriteLine("\t" + arg);
 
            Console.ReadLine();
        }
        public static string[] ParseArgs (string full)
        {
            return System.Text.RegularExpressions.Regex.Split(full, " (?=(?:[^\"]*\"[^\"]*\")*[^\"]*$)");
        }
1
0 / 0 / 2
Регистрация: 11.12.2015
Сообщений: 47
09.11.2016, 09:43  [ТС]
Работает, и последний вопрос тогда, как в ответе кавычки убрать? Я сделал так:
C#
1
2
3
4
for(int i=0; i<parts.Length; i++)
            {
                parts[i] = parts[i].Replace(@"""", "");
            }
Хочу делать свой код красивее, и боюсь за костыли.

c "arg 1" =
c
arg 1
0
[Bicycle Reinventor]
 Аватар для Exerion
332 / 270 / 109
Регистрация: 19.10.2011
Сообщений: 668
Записей в блоге: 2
09.11.2016, 10:30
Лучший ответ Сообщение было отмечено elektro-volk как решение

Решение

В .net уже есть всё, что нужно. Нет необходимости качать чужие библиотеки.

C#
1
2
3
4
5
6
7
8
9
10
using Microsoft.VisualBasic.FileIO;
 
string rawCmd = "command "argument1 and spaces" argument2 3";
using (StringReader sr = new StringReader(rawCmd))
using (TextFieldParser parser = new TextFieldParser(sr))
{
    parser.Delimiters = new string[] { " " };
    string[] tokens = parser.ReadFields();
    foreach (var s in tokens) Console.WriteLine(s);
}
Необходим лишь добавить референс на Microsoft.VisualBasic.dll

Отличный парсер csv форматов (для данного случая тоже подойдёт), использую во всех проектах, где надо разбирать csv, ни разу не подводил. Корректно проглатывает все варианты: кавычки, удвоенные/утроенные кавычки, разделители внутри кавычек, пробелы, csv внутри csv и прочее и прочее.


Не по теме:

p.s. когда уже на форуме пофиксят обратные слеши в теге [CSHARP]?

1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
09.11.2016, 10:30
Помогаю со студенческими работами здесь

Системная информация - как улучшить код?
Проблема в том что выходит слишком много foreach, как всё это можно сократить? string savePath = @&quot;C:\Sys.txt&quot;; ...

Найти элемент в массиве которого больше 50%. Улучшить код
Новый русский Колян любит две вещи: деньги и порядок. У Коляна много денег, но в них нет порядка. Одним прекрасным утром Колян понял, что...

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

улучшить код
добрый день. Начал недавно изучать angularjs, попросил людей скинуть задачки, чтобы потренироваться. Скинуть - скинули, но вот пропали...

Улучшить код
Написал код, хотелось бы узнать, как можно его улучшить. Буду благодарен если поможете, заранее спасибо! Код: Engine = 0 ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru