task - 3 sales - Софтуерен университет

danyboy 2 Точки

task - 3 sales

Здравейте колеги имам нужда от малко помощ . немога да разбера защо judge не изписва нищо в: Your output: " нищо" при положение че аз на конзолата получавам отговор. изглежда че работи няма грешки предупреждения и уви резултата е правилен " на моят телевизор"

Solution #13008190 by danyboy for problem 03. Sales

https://pastebin.com/yPPjxpv1

ако някои може да помогне с идея ще съм му много благодарен

Тагове:

30/10/2019 16:26:04 C++ Fundamentals

galin_kostadinov 166 Точки

Best Answer

Привет!

bool find_town(vector<sales> &inputVector, string city, double volume_) {

    for (vector<sales>::iterator it = inputVector.begin(); it != inputVector.end(); ++it) {
        if (it->getTown() == city) {
            double sum = it->getVolume();
            sum += volume_;
            it->setVolume(sum);
            return true;
        }
    }

 return false; // добавих го
}

Трябва да гарантираш, че винаги ще се върне някаква стойност, тъй като имаш само if(), но нямаш какво да върне в противен случай.

(Пробвал съм решението след добавктата и при мен минава 100/100, макар, че първия път ми гръмна с "Лимит време" )

Поздрави!

30/10/2019 17:06:16 30/10/2019 17:10:16

danyboy 2 Точки

благодаря ти трябвало ми е само един ред за да си реша задачата макар и по наи-смотания начин

31/10/2019 08:47:19 31/10/2019 09:09:51

kolioi 641 Точки

Ето ти едно примерно решение https://pastebin.com/4PmCmkp6

30/10/2019 17:47:30

danyboy 2 Точки

евала толкова изчистено благодаря за примера

31/10/2019 09:11:27

j.petrov_90 373 Точки

Привет, колега,

Използвам направо тази темата за да ти отговоря на въпроса и на предната.

Имаш доста вярни неща. Продължавай и не се отказвай! :)

Имаш и доста грешни неща.
Колегата Галин е посочил една от най-големите грешки.

Предупреждавам и че getter-ите ти връщат референция към оригинала.
Внимавай, че голяма лопата ще настъпиш.

Решението на задачата (доста изчистено такова) е споделено от колегата kolioi.
Гледай него и го сравни с твоето.

Поднасям ти градивна критика и искам да я приемеш сериозно.
Отказах се да гледам решението ти след 2-3 минути. Не за нещо друго, а просто наименуването на променливите ти е под всякаква критика. За читателя на твоя код (в случая) става невъзможно да разбера какво си искал да направиш.

vector<string> & first_town - това сега 1 град ли е или поредица от градове?
vector<sales> & inputVector - аз го виждам, че е вектор. Напиши ми какво значи тоя вектор
Функция "print", която сортира ... няма начин просто.

bool find_shit = sale.find_town(vec,townIn,volume);
if(!find_shit ){
vec.push_back(sale);
}

За това съм без коментар просто.

Знам, че това е просто курс и това е единична задача от този курс, но започни от сега да градиш навици.
Не започнеш ли сега - никога няма да дойде "после".

Жив и здрав
И не се притеснявай, ще борим OOП-то в детайли в рамките на C++ Advanced

30/10/2019 21:30:56

danyboy 2 Точки

Живко благодаря за отговора. за променливите и особено името на една от функците..... но не това ме притеснява а: " Предупреждавам и че getter-ите ти връщат референция към оригинала.
Внимавай, че голяма лопата ще настъпиш." какво значи това без & ли да ги пиша ?

погледнах и на колегата kolioi кода и аз искам да пиша така като порасна но както каза по нагоре : Продължавай и не се отказвай! :) все пак това ми е първия Метод мерси още веднъж

31/10/2019 09:08:02 31/10/2019 09:08:40

j.petrov_90 373 Точки

Ами връщаш референция от getter функция.
Това е опасно поради 2 причини:
- ако върнеш локална променлива(която не е част от класа) имаш отромен бъг
- по този начин "счупваш" правилото за енкапсулация (т.е. хем имаш нещо private, а хем му даваш публичен достъп като функцията ти връща оригинала).

01/11/2019 10:43:21