Здравейте,
На задача с условие https://judge.softuni.bg/Contests/Practice/Index/2032#1 и решение https://pastebin.com/VKqzk0D0, Judge ми дава 90/100. Моля за Вашата помощ. Благодаря предварително!
Имате много повтарящ се код (почти всичко в 4-те метода за придвижване), което комбинирано с използването навсякъде в програмата на статични глобални променливи с близки (rowCheck, checkRowT и colCheck, checkColT) или твърде общи като смисъл имена (isTrue), прави допускането на грешки твърде лесно и затруднява ненужно намирането им.
Силно Ви препоръчвам да рефакторирате основно кода спрямо по-горните ми забележки и грешката ще стане очевидна, ако не бъде оправена междувременно.
Позволявам си да направя този по-критичен коментар, тъй като задачата е към изпит след C# Advanced и се очаква кодът, който пишете след това ниво, да бъде с по-високо качество.
Знам, че коментара Ви е основателен, но съм в началото на модула и все още имам пропуски. Ще опитам да си рефакторирам кода и ще го върна към дискусията. Благодаря за коментара.
Направих някои промени по кода си, но продължавам да не си виждам грешката и пак получавам 90/100 https://pastebin.com/nN0Ji9xd.
https://pastebin.com/X4cyBhes - Работещо решение и доста по-кратко. Успех!
@Milko123
Тест №6 не минава заради Runime error, а при работа с масиви, най-честата причина за такива exception-и е невалиден индекс.
В кода забелязвам, че след превъртане и попадане на 'T', променяте индекса в грешната посока.
Например в MoveUp, ако сме на ред 0 и получим тази команда, тя ще ни изпрати на последния ред. Aко там има 'Т' вместо да ни върне на ред 0, ще увеличи реда с 1, което вече е извън размера на matrix и като сигне до if (matrix[rowCheck, colCheck] == 'F') ще хвърли IndexOutOfRangeException
Ето примерен тест, който демонстрира бъга:
MartinBG
Много благодаря за подхода Ви към моя проблем. Последвах Вашите препоръки и открих начин, примерния вход, който сте дали да не връща грешка, но в случая на задачата това не е нужно. Това е ценен съвет за мен, тъй като в друга подобна задача би ме подвело. Та с малко помощ от код на друг колега преправих моя и постигнах 100/100 в Judge https://pastebin.com/v0DhPPJA. Голяма част от проблема ми се дължеше на грешни оператори в if проверките за движение.