Getting values from an array

Question

So let's say I have an array called arr with the values &&&&.&&. I want to find the number of ampersands (&) that are after the decimal point and store the value into numDecimalDigits.

int numDecimalDigits = 0;

char[] arr = new char[7]

for (int i = 0; i < str.length(); i ++)
{
    for (int decimal = (arr[pos] = '.'); decimal <= arr.length; decimal ++)
    {
        numDecimalDigits += 1;
    }
}

I'm not sure if this is the right approach. So the outside for loop runs through each index value of the array. The inner for loop starts at the decimal, and ends at the end of the array. Every time a new value is found, numDecimalDigits is added by one. However, in my code I think numDecimalDigits is returning an incorrect value.

What is this supposed to be doing -- (arr[pos] = '.')? What it does is set arr[pos] to '.' (whatever pos is). — Hot Licks
– Hot Licks, Commented Feb 10, 2014 at 2:33
Well what I tried doing was to start the for loop at where the decimal point began, and end the loop at the end of the array. pos is just short for "position" (i.e. position in array) — Evan
– Evan, Commented Feb 10, 2014 at 2:38
Using indexOf on a String would make more sense. Or a single loop from 0 to array.length, comparing to '.'. (And you compare for equal with ==. = is assignment.) — Hot Licks
– Hot Licks, Commented Feb 10, 2014 at 2:38
(arr[pos] = '.') doesn't tell you where the decimal point is, it assigns '.' to arr[pos]. — Hot Licks
– Hot Licks, Commented Feb 10, 2014 at 2:39

Weibo Li · Accepted Answer · 2014-02-10 02:39:55Z

1

You only need one loop：

boolean foundDot = false;
for (int i = 0; i < arr.length; i++) {
    if(arr[i] == '.') {
        foundDot = true;
    } else if(foundDot) {
        numDecimalDigits ++;
    }
}

answered Feb 10, 2014 at 2:39

Weibo Li

3,6054 gold badges27 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Baby · Accepted Answer · 2014-02-10 02:50:00Z

1

No need to use array. It would be easy like this:(Assuming str value must contains one '.' )

    int numDecimalDigits = str.split("\\.")[1].length();

Or you can do by subtracting str.length()-1 with indexOf(".")

    int numDecimalDigits = str.length()-1 - str.indexOf(".");

edited Feb 10, 2014 at 2:50

answered Feb 10, 2014 at 2:32

Baby

5,1123 gold badges33 silver badges53 bronze badges

4 Comments

Hot Licks Over a year ago

Wouldn't make a lot more sense to use indexOf and not bother creating new strings you don't need??

Baby Over a year ago

@HotLicks which new string you are referring to?

Hot Licks Over a year ago

The two strings created by split. (Not to mention the String array.)

Baby Over a year ago

Ok, I have added your suggestion.

Collectives™ on Stack Overflow

Getting values from an array

2 Answers 2

Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related