Лишняя инициализация

@foxwizard · Регистрация: 18.04.2016

Студворк — интернет-сервис помощи студентам

C#
1
2
3
4
 public static Vector Add(Vector vector1, Vector vector2)
        {
            return new Vector() { X = vector1.X + vector2.X, Y = vector1.Y + vector2.Y };
        }

Ест замечание по кодревью
Лишняя инициализация –> проблемы с памятью
Используй новый метод
Что можно подправить?

@КОП · 19.12.2021, 21:14

Сообщение от foxwizard

Что можно подправить?

для начала стоит огласить исходную постановку задачи

@kypiwindy · 19.12.2021, 21:21

foxwizard, не вижу много опций. Можно выделить конструктор и использовать перегрузку оператора +

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
 struct Vector{
        public int X { get; set; }
        public int Y { get; set; }
        public Vector(int x,int y)
        {
            X = x;
            Y = y;
        }
        public static Vector operator + (Vector a, Vector b)
        {
            return new Vector(a.X+b.X, a.Y+b.Y);
        }
    }

@OwenGlendower · 20.12.2021, 08:28

Сообщение от foxwizard

Лишняя инициализация –> проблемы с памятью

Одно из другого здесь не следует.

Сообщение от foxwizard

Что можно подправить?

Сделайте инициализацию через конструктор. kypiwindy, уже показал как.

Добавлено через 17 минут
Хотя если Vector это класс, то потенциальная проблема с памятью в самом деле есть и можно переписать так

C#
1
2
3
4
5
public void Add(Vector vector)
{
    X += vector.X;
    Y += vector.Y;
}

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук 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 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .