Dev 56 add jvm args editor

[D1edDuck]

No description provided.

[IgorShaposhnikov]

А в использование StackPanel есть какая-та секретная логика или можем просто удалить?

[IgorShaposhnikov]

Думаю все содержимое этой компанды лучше вынести в отдельный метод для читабельности

[IgorShaposhnikov]

можно добавить sealed

[IgorShaposhnikov]

думаю стоит вынести за метод как поле класса в формате:

private static readonly Regex SplitArgsRegex = new(
    @"[\""'].+?[\""']|[^\s]+", 
    RegexOptions.Compiled
);

[IgorShaposhnikov]

Тоже самое для для всех regex, что и в методе SplitArgs выше.

[IgorShaposhnikov]

думаю стоит все viewmodel запечатывать через sealed

[IgorShaposhnikov]

Стоит подумать можно ли большую часть логики отсюда вынести в JvmArgsEditorModel. Во viewmodel останется только команды и ссылка на Model.

В качестве примера можно взять GeneralSettingsViewModel

[IgorShaposhnikov]

думаю можно вернуть string.Empty заместо ""

[IgorShaposhnikov]

думаю лучше вот так метод сделать

private void OnFocusNewEntryRequested(JvmArgEntry entry)
{
    // Fail-fast guard clauses to prevent NullReference and ArgumentOutOfRange exceptions
    if (entry == null || ArgsDataGrid == null || ArgsDataGrid.Columns.Count == 0)
        return;

    // 1. Instantly select and start the scroll action
    ArgsDataGrid.SelectedItem = entry;
    ArgsDataGrid.ScrollIntoView(entry);

    // 2. Queue the remaining work until after the DataGrid finishes rendering the row
    Dispatcher.BeginInvoke(System.Windows.Threading.DispatcherPriority.Background, new Action(() =>
    {
        // Re-verify the item is still valid after the brief async delay
        if (ArgsDataGrid.Columns.Count == 0) return;

        var column = ArgsDataGrid.Columns[0];
        ArgsDataGrid.CurrentCell = new DataGridCellInfo(entry, column);

        // 3. Move keyboard focus to the grid and start editing the cell
        ArgsDataGrid.Focus();
        ArgsDataGrid.BeginEdit();
    }));
}