要素の検索を行うプログラム

Question

###実現したいこと
私はJavaを始めてまだ数日の初心者です。
整数の配列の要素の検索で、配列x1の要素の整数aが現れる最初の位置 (x1[i]==a を満たす最小のi),aがなければ要素数を表示するプログラムを作り、半日自分で考えて以下のようなプログラムを作成しました。
しかし、最小のiだけでなくその他の該当するiが表示されてしまったり、x1の要素数が常に表示されてしまったりと、何が原因なのか考えても一向に分からず、自分の力量では手に負えないと感じ、質問するに至った次第です。
できるだけ簡単に、どこが違うのか、どこが足りないかを教授していただけると幸いです。
(ただし条件があって、メソッドとしてsearchを使用、戻り値はintです。)
###実際書いたプログラム
public static void main(String[]args) {
int[]x1={4,2,4,4,3};
System.out.println(search(x1));
｝
public static int search(int[]x1) {
int g=x1.length;
int a=3;
for (int i=0; i<g; i++){
if (x1[i]==a) {
System.out.println(i);
}else if(x1[i]!=a){
System.out.println();
}else{
System.out.println(x1.length);
}
}
return g;
}

Answer 1

コードが読みづらいのでマークダウン記法を使用して質問文を修正してください。
ヘルプ

最小のiだけでなくその他の該当するiが表示されてしまったり

「最小のi」を見つけた時点でループを終了させていないから。

x1の要素数が常に表示されてしまったり

System.out.println(search(x1));

上記が原因です。

Comments

[kt-]

ループを終了させるには`break`は要りますか？それとも違うものが必要ですか？
そこだけ、いくら考えても分からずにいるのですが…。

[javahack]

考えて分からなければ検索。
「java ループ処理 抜ける」の検索結果↓
https://www.google.com/search?q=java+%E3%83%AB%E3%83%BC%E3%83%97%E5%87%A6%E7%90%86+%E6%8A%9C%E3%81%91%E3%82%8B&rlz=1C1AVFC_enJP830JP830&oq=java+%E3%83%AB%E3%83%BC%E3%83%97%E3%81%97&aqs=chrome.2.69i57j0i4l3j0i8i30.9844j1j7&sourceid=chrome&ie=UTF-8

[kt-]

わざわざ申し訳ございません。ありがとうございます。

[kt-]

度々申し訳ございません。
最後の質問ですが、System.out.println(search(x1));の、何が原因でしょうか？
x1の部分を変えても『〇を変数に解決できません』とのエラーメッセージが出るのですが…。