«

This function keeps giving me negative values (esp. if I feed it
64,153,160)

What gives?

On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> This function keeps giving me negative values (esp. if I feed it
> 64,153,160)
>
> What gives?

int quantity(char chrs[],int length)
{
int q = 0;
int x;
int c = 0;

for (x=0; x<length;x++)
{
c = (int)chrs[x];
q += c;
}
return q;
}

"Mr. Politics" <> wrote in message
news: ups.com...
> On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
>> This function keeps giving me negative values (esp. if I feed it
>> 64,153,160)
>>
>> What gives?
>
> int quantity(char chrs[],int length)
> {
> int q = 0;
> int x;
> int c = 0;
>
> for (x=0; x<length;x++)
> {
> c = (int)chrs[x];
> q += c;
> }
> return q;
> }
>

char is probably signed on your platform (and with 8 bits, it can possibly
only hold values between -128 and 127 inclusive)

- Sylvester

On Feb 12, 8:18 am, "Sylvester Hesp" <s.h...@oisyn.nl> wrote:
> "Mr. Politics" <mrpolit...@gmail.com> wrote in message
>
> news: ups.com...
>
>
>
> > On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> >> This function keeps giving me negative values (esp. if I feed it
> >> 64,153,160)
>
> >> What gives?
>
> > int quantity(char chrs[],int length)
> > {
> > int q = 0;
> > int x;
> > int c = 0;
>
> > for (x=0; x<length;x++)
> > {
> > c = (int)chrs[x];
> > q += c;
> > }
> > return q;
> > }
>
> char is probably signed on your platform (and with 8 bits, it can possibly
> only hold values between -128 and 127 inclusive)
>
> - Sylvester

For posterity, the fix... (Thank you!)

int quantity(unsigned char chrs[],int length)
{
int q = 0;
int x;
int c = 0;

for (x=0; x<length;x++)
{
c = (int)chrs[x];
q += c;

}
return q;
}

On Feb 12, 2:35 pm, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> On Feb 12, 8:18 am, "Sylvester Hesp" <s.h...@oisyn.nl> wrote:
>
>
>
>
>
> > "Mr. Politics" <mrpolit...@gmail.com> wrote in message
>
> >news: oups.com...
>
> > > On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> > >> This function keeps giving me negative values (esp. if I feed it
> > >> 64,153,160)
>
> > >> What gives?
>
> > > int quantity(char chrs[],int length)
> > > {
> > > int q = 0;
> > > int x;
> > > int c = 0;
>
> > > for (x=0; x<length;x++)
> > > {
> > > c = (int)chrs[x];
> > > q += c;
> > > }
> > > return q;
> > > }
>
> > char is probably signed on your platform (and with 8 bits, it can possibly
> > only hold values between -128 and 127 inclusive)
>
> > - Sylvester
>
> For posterity, the fix... (Thank you!)
>
> int quantity(unsigned char chrs[],int length)
> {
> int q = 0;
> int x;
> int c = 0;
>
> for (x=0; x<length;x++)
> {
> c = (int)chrs[x];
> q += c;
>
> }
> return q;
>
>
>
> }
Instead of pulling hair out, you should pull your code in to a
debugger and see whats going on. One hint is that "casts are evil" is
almost always true.

/Peter

"peter koch" <> wrote in message
news: oups.com...
> On Feb 12, 2:35 pm, "Mr. Politics" <mrpolit...@gmail.com> wrote:
>> On Feb 12, 8:18 am, "Sylvester Hesp" <s.h...@oisyn.nl> wrote:
>>
>>
>>
>>
>>
>> > "Mr. Politics" <mrpolit...@gmail.com> wrote in message
>>
>> >news: oups.com...
>>
>> > > On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
>> > >> This function keeps giving me negative values (esp. if I feed it
>> > >> 64,153,160)
>>
>> > >> What gives?
>>
>> > > int quantity(char chrs[],int length)
>> > > {
>> > > int q = 0;
>> > > int x;
>> > > int c = 0;
>>
>> > > for (x=0; x<length;x++)
>> > > {
>> > > c = (int)chrs[x];
>> > > q += c;
>> > > }
>> > > return q;
>> > > }
>>
>> > char is probably signed on your platform (and with 8 bits, it can
>> > possibly
>> > only hold values between -128 and 127 inclusive)
>>
>> > - Sylvester
>>
>> For posterity, the fix... (Thank you!)
>>
>> int quantity(unsigned char chrs[],int length)
>> {
>> int q = 0;
>> int x;
>> int c = 0;
>>
>> for (x=0; x<length;x++)
>> {
>> c = (int)chrs[x];
>> q += c;
>>
>> }
>> return q;
>>
>>
>>
>> }
> Instead of pulling hair out, you should pull your code in to a
> debugger and see whats going on. One hint is that "casts are evil" is
> almost always true.
>
> /Peter
>

While I agree with you, I don't really see how this particular situation
changes when you just remove the cast to int

- Sylvester

On Feb 12, 3:13 pm, "Sylvester Hesp" <s.h...@oisyn.nl> wrote:
> "peter koch" <peter.koch.lar...@gmail.com> wrote in message
>
> news: oups.com...
>
>
>
>
>
> > On Feb 12, 2:35 pm, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> >> On Feb 12, 8:18 am, "Sylvester Hesp" <s.h...@oisyn.nl> wrote:
>
> >> > "Mr. Politics" <mrpolit...@gmail.com> wrote in message
>
> >> >news: oups.com...
>
> >> > > On Feb 12, 8:05 am, "Mr. Politics" <mrpolit...@gmail.com> wrote:
> >> > >> This function keeps giving me negative values (esp. if I feed it
> >> > >> 64,153,160)
>
> >> > >> What gives?
>
> >> > > int quantity(char chrs[],int length)
> >> > > {
> >> > > int q = 0;
> >> > > int x;
> >> > > int c = 0;
>
> >> > > for (x=0; x<length;x++)
> >> > > {
> >> > > c = (int)chrs[x];
> >> > > q += c;
> >> > > }
> >> > > return q;
> >> > > }
>
> >> > char is probably signed on your platform (and with 8 bits, it can
> >> > possibly
> >> > only hold values between -128 and 127 inclusive)
>
> >> > - Sylvester
>
> >> For posterity, the fix... (Thank you!)
>
> >> int quantity(unsigned char chrs[],int length)
> >> {
> >> int q = 0;
> >> int x;
> >> int c = 0;
>
> >> for (x=0; x<length;x++)
> >> {
> >> c = (int)chrs[x];
> >> q += c;
>
> >> }
> >> return q;
>
> >> }
> > Instead of pulling hair out, you should pull your code in to a
> > debugger and see whats going on. One hint is that "casts are evil" is
> > almost always true.
>
> > /Peter
>
> While I agree with you, I don't really see how this particular situation
> changes when you just remove the cast to int

My hope was that the OP would realise that there would have to be a
"real" conversion, and not just a cast. I was seemingly to optimistic.

/Peter

* peter koch:
> [umpteen lines quoted]

Please.

Quoting that much just to add a single line is /worse/ than top-posting.

IMHO.


Cheers,

- Alf

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?

Alf P. Steinbach wrote:

> * peter koch:
> > [umpteen lines quoted]
>
> Please.
>
> Quoting that much just to add a single line is worse than top-posting.
>
> IMHO.

Except that top-posters would almost certainly have quoted the whole as
well.




Brian

Not quite sure what you are trying to accomplish? Are you trying to
cast the char array into 1 int or into a combination of 3 or so ints?
It seems like 1 int.. if so, you need to do a lot more.

If its one int, this would work.

int quantity(unsigned char chrs[],int length)
{
int q = 0;
int x;
int c = 0;

for (x=0; x<length;x++)
{
c = (int)chrs[x];
c -= 48;
int pos = (length - x);
c *= pow(10, pos-1);
q += c;
}
return q;
}