Does this code is right or need to some changes? It basically search the Given word by user in a String and print the word inde

Question

I am new learner of java. Kindly help to improve ny logic and its a humble request to see my code. And guide me. What possible Our teachers give task related to this code? What modification should be done to this code? How can i search word if it happend twice or thrice in string?
Any help and advice would be greatly appreciated
Thankyou All
#HappyProgramming
https://code.sololearn.com/cMFoyJMk17xd/?ref=app

Answer 1

Haha, I am certainly not a magician and can't guess what your teacher might be thinking in the future :)
It would make sense to ask for the inputs at runtime, so the user can type in any sentence and search word. For that, you would probably use the Scanner class.

Answer 2

First you have this loop
for (int i =0;i<sent.length();i++)
And then you have these:
sent.charAt(i+1)
so if the word is at the end, i will exceed the string length and you will get an error.
So you should adjust your loop like
i<sent.length()-word.length()

Also your program works only if the word is exactly 3 characters long. You may want to make another loop inside the first one, to extract each character, same length as the word, starting from index i

Answer 3

Yeah. Got. It. Man thankew soo much. I was getting this error now. I understand.
One. More question how can i make this program generic so it can work for every input.
Thanks

Answer 4

Lothar chek this when u got free
Thanks

Answer 5

It is better if you do not change either the string nor the search phrase.
You could adjust the for loop to consider the size of 'sent' and 'word' . The index should check only until sent.length()-word.length()
This way you can avoid ArrayOutOfBoundsException, when the word is at the very end.
I recommend to use the substring() method to capture parts of the sent string and compare them with word.
https://www.javatpoint.com/substring

Answer 6

Tibor Santa Our teacher have strictly forbid to use any builtin function in program except charAt(i)

Answer 7

Sent.length-word.length?
Can u explain little bit more
Thanks

Answer 8

It will work for any input, if you do the changes I suggested. It will be able to detect multiple occurrences of the word too, just don't use break when you find the first one.

Answer 9

Ok Thanks Allot Man
Do u have any idea what could possible Our teacher asks to some more Modification in this code?

Answer 10

Wht could possible our Sir can give more tasl in this program. Any Task in ur mind so i will try to solve it now

Answer 11

Hahaha thanks for the help