How to return a variable from inside a for loop

Question

I am trying to return the index of the element containing character 'r' which has charcater 'u' as its successor. The following piece of code always returns the varialbe ruindex =0. Please suggest edits in the code.

    int ruindex=0;
    if (s1[0]=='h') {
        s2[0] = 'h';
        s2[1] = 't';
        s2[2] = 't';
        s2[3] = 'p';
        s2[4] = ':';
        s2[5] = '/';
        s2[6] = '/';

        for (int i=4; i < s1.length-1; i++) {
            if (s1[i]=='r' && s1[i+1]=='u') {
                ruindex = i;
                break;
            }
        }
    }

It is just a part of my overall code. Yes there is a reason why I want to start from i=4. What I wish to do is that whenever I encounter 'r' and 'u' together in the character array, I want to return the index of that 'r'.

Is there a reason you're starting from i=4, you're only setting up s2 if the first value of s1 is h, there isn't this r in s2 you're looking for and what does s1 and s2 mean here, why are there 2 separate variables? Please elaborate on your requirements and provide an SSCCE. — Bernhard Barker
– Bernhard Barker, Commented Sep 26, 2013 at 10:01

aviad · Accepted Answer · 2013-10-02 15:21:18Z

3

How about

String s2 = new String(s1);
s2.indexOf("ru");

?

edited Oct 2, 2013 at 15:21

answered Sep 26, 2013 at 10:02

aviad

8,2789 gold badges55 silver badges100 bronze badges

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

2 Comments

AmanArora Over a year ago

Given a string "hlolrulolru", would the above piece of code return 4?

aviad Over a year ago

do you have google "installed" to search for the java doc of indexOf method of String class? :)

Limbo Exile · Accepted Answer · 2013-09-26 10:13:10Z

0

Your code does affect index of first encountered "ru" to ruindex variable if arrays are initialized and populated. However you should change the loop to

for (int i = 4; i < s1.length - 1; i++) {

or you'll get an ArrayOutOfBoundsException if 'r' is the last character of the array and there are no matches before.

Anyway it seems to me that you should consider working with Strings, rather than arrays. You'll be able to use dedicated methods to achieve your goal or even use regular expressions if your goal becomes more complicate than this.

answered Sep 26, 2013 at 10:13

Limbo Exile

1,3512 gold badges21 silver badges43 bronze badges

Collectives™ on Stack Overflow

How to return a variable from inside a for loop

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related