170 похожих чатов

Привет. Никак не могу придумать, как сделать лучше вторую функцию

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

2 ответов

7 просмотров

Рекомендую сразу перейти с массивов статических размеров на vector<>, если пишете на плюсах, а не на C. Варианты ответа на ваш вопрос: 1. Не особо беспокоиться о производительности, а беспокоиться об отсутствии багов. Тогда стоит сделать .sales приватным полем и никогда в него напрямую не писать — только вызывая setSales. Тогда поля max, min, average будут гарантированно корректными. В setSales, который считавает, можно завести отдельный массив и записывать в него. 2. Разделить setSales на две функции: одна копирует в приватное поле sales, а вторая пересчитывает min, max, average. Тогда нижняя функция будет выглядеть как "считать всё в поле, вызвать функцию пересчёта". А верхняя будет выглядеть как "скопировать из аргумента в поле, вызвать функцию пересчёта".

для начала посоветую убрать из функций ненужный return

Похожие вопросы

Обсуждают сегодня

Есть какой-нибудь для Delphi/FPC T*Compression(Decompression)Stream на базе LZ4/Zstd/любой другой быстрый(и хорошо сжимающий) алгоритм А ещё лучше в pure pascal А ещё лучше од...
notme
48
Такой вопросец - есть функция function MySuperDuperConcat(const a: array of AnsiString): AnsiString; Как мне в её теле сделать вот так? Result:=Concat(a); А не грустный вариан...
notme
15
type TObj = object procedure Init; virtual; end; TObj1 = object(TObj) procedure Init; override; end; procedure TObj1.Init; begin inherited; end; procedur...
Alexander 👋
29
А чем вам питонисты не угодили?😂
.
79
Всем привет. Поделитесь, пожалуйста, опытом. Есть форма, на которой имеется dbgrid и кнопки: добавить, редактировать, удалить. Если нет записей в dbgrid, то кнопки редактирова...
Евгений
4
Всем привет, написал код ниже, но он выдает сегфолт, в чем причина? #include <stdio.h> #include <stdlib.h> #include <string.h> struct product { char *name; float price; };...
buzz базз
86
Вот смотрите у меня есть два стрима сейчас fs, cs: TStream; fs := TFileStream.Create('filename'); cs := TCompressionStream.Create(compression_level, fs); Я хочу сделать так: ...
notme
5
type TExtensions<GExtender>=class function GetExtension<GEntityExtenderType>:GEntityExtenderType; end; function TExtensions<GExtender>.GetExtension<GEntityExtenderType...
zamtmn
8
Всем здравствуйте! Я хотел узнать сколько стоит средняя месячная зарплата у Electron js разработчиков? Мне очень это важно и нужно, плиз помогите узнать эту инфу! Для Джунио...
U.K.
10
Можно ли загрузить скрипт py в бота чтобы он работал по нему? как это сделать?
huskadam #RCC Фанат? @hitlerpvp
13
Карта сайта