[Exam Problems] Advanced C# Exam 19 July 2015 - Problem{1} - Bunker Buster
Здравейте на всички.За първи път задавам въпрос във форума и се надявам след като прочeтох правилата да съм ги разбрал.
Имам проблем със задачата [Exam Problems] Advanced C# Exam 19 July 2015 - Problem{1} - Bunker Buster.
Пускам кода ми като приемам препоръки отностно него и критики ,защото ми се струва ,че стана малко нечетим.
http://pastebin.com/nqK6M95i
Нулевите тестове минават,но на следващите ми пише Runtime error и си мисля,че ми гърми при проверките,но като правя различни варианти не ми гърми.
Благодаря предварително.
Благодаря ти.Аз ще я довърша проблема ми беше само защо ми гърми.Сега вече знам каго я измисля ще напиша
Измислих я само до 40/100.
Така и не мога да разбера след всички корекции,защо продължава да ми гърми.
http://pastebin.com/nqK6M95i
Ако имаш други идеи.
В BombMatrix там, където си слагал break, го замених с continue. С break-овете прекъсваш проверката за други бункери по същата колона...
Извадих метода DestroyedCells от BombMatrix. Лоша практика е да имаш метод, който изпълнява две дейности - в случая не само се бомбардират клетките, но и се броят разрушените клетки. Вместо това го преработих да проверява цялата матрица за унищожени клетки и го извиквам от Main само един път - след като са извършени всички бомбардировки.
С тези промени става 60/100, ще търся още...
Не мога да намеря къде са останалите грешки...
Ако искаш виж моето решение - то дава 100/100. Аз използвам малко по-различна логика за бомбардирането и не мога само като сравня решенията да намеря, къде се дъни твоето![sad sad](https://softuni.bg/Scripts/ckeditor/plugins/smiley/images/sad_smile.png)
Една грешка която съм направил е в където чета входа съм сложил ToLower() а то не трябва да е така мисля че с тази поправка може да е вярна.
Твоето решение ще го видя,но искам още малко да я помъча.
В момента в който смених break с continue ми дава 70/100;
Мога ли да видя как си го преработил .
Благодаря за отделеното време.
Ето това даде 60/100 с преработката, за която споменах: http://pastebin.com/E2BBzzyN