C# Numbers game

OriginalGriff

No, it's l

void oznaci_po_redu(int s)
{
l = 0;
while (s > 0)
{
l++;
if (!uzet[l]) s--; -----Index was outside the bounds of the array--------
}
uzet[l] = true;
}

But it gets increased while s is non zero, and when does s get reduced?

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Lost User

My bad. And looking more closely at the code, I now see what you were alluding to.

OriginalGriff

:-D A quick visit with Mr Debug would have shown the OP as well! :laugh:

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Pavlex4

I changed this loop for (prog.u = 1; prog.u < 6; prog.u++) to for (prog.u = 0; prog.u < 5; prog.u++) now I have this error: imageshack.com/a/img923/8677/WIBoHG.png

Lost User

And the message is telling you exactly what is wrong.

OriginalGriff

And what error is that? It's less work for you to copy and paste the actual text, than to post a screen shot on a dodgy image site that I'm not planning on visiting...

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Pavlex4

Error is: Attempted to divide by zero.

OriginalGriff

Well, the error message pretty much explains itself. Mathematics does not allow you to divide by zero, so you should probably make sure your application doesn't try to do that, wouldn't you say? The fun bit is finding out why your application is trying at all. And - again - that's where you and Mr Debugger get to be good friends, because he's the one that tells you how the divisor came to be zero: which is pretty much essential to preventing it in future. Seriously, get used to the debugger - it's a powerful, flexible tool that most developers spend a considerable amount of time with. It's a lot more fun - and a lot quicker - to find the information than it is to ask here. Especially as most of the time we have to say "use the debugger to get information" anyway! :laugh:

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Pavlex4

When I changed all loops to start from 0 to n-1,those that start with other number than 0 I set to start from value reduced by 1 to n-1 I still get error: Index was outside the bounds of the array

OriginalGriff

And? What did the debugger show you?

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Pavlex4

I fixed that error but when I start program and input target value and values I get wrong expression. When I type 100 for target number and numbers 5,8,9,4,20,50 for making expression I get: 5150-(' + w + ')+5355+5150 Instead of: (6-1)*4*5

OriginalGriff

At the risk of repeating myself:

Quote:

And? What did the debugger show you?

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Lost User

OriginalGriff wrote:

visit with Mr Debug would have shown the OP

Why bother when you can be a help vampire?

Speed of sound - 1100 ft/sec Speed of light - 186,000 mi/sec Speed of stupid - instantaneous.

Dave Kreskowiak

OK. Sit down and listen up. Your'e trying to get everyone here to debug your code for you. That's not going to happen. The debugger is there to debug YOU, to give you a tool to understands how your code works, what it's really doing, to increase your understanding. USE IT. The problem with the code you have is that you didn't write it and have no idea how it's supposed to work. This is where the debugger is an indispensable tool, to better your understanding of how the code works. We can't do that for you.

A guide to posting questions on CodeProject

Click this: Asking questions is a skill. Seriously, do it.
Dave Kreskowiak

OriginalGriff

The amount of time and effort he's putting into avoiding work ... :sigh:

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

OriginalGriff

Maybe we need a special badge for 'em :laugh:

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Lost User

OriginalGriff wrote:

special badge

Good idea....Bob with vampire fangs.

Speed of sound - 1100 ft/sec Speed of light - 186,000 mi/sec Speed of stupid - instantaneous.

Patrice T

if (!uzet[l]) s--; -----Index was outside the bounds of the array--------

This error message tells you that your code try to access an element rgat do not exist in the array uzet because uzet us too small. With the debugger, check the size of uzet.

Patrice “Everything should be made as simple as possible, but no simpler.” Albert Einstein

Mycroft Holmes

ppolymorphe wrote:

rgat

Missed by that much :laugh:

Never underestimate the power of human stupidity RAH

Richard Deeming

Couldn't find a vampire; will this do? 🦇 🦇 Bat Emoji[^]

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer