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

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

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

2 ответов

11 просмотров

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

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

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

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

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Вот еще странный косяк, подскажите как бороться. Я git clone сделал себе всего embassy и примеры там запускаю. Всё хорошо. Но вот решил в cargo.toml зависимости не как в приме...
Lukutin R2AJP
5
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
Карта сайта