文字の部分一致について

Question

以下のコードでうまく動作しません
文法に誤りがあるようなのですが何がいけないのか
わかりません、どなたか御教授いただけませんでしょう。

import java.util.*;
public boolean isMatch(String S, String W) {
if(S.matches(W)) {
return true;
}
else {
return false;
}
}
public class Main {
public static void main(String[] args) {
Scanner sc = new Scanner(System.in);
String S = sc.nextLine();
String W = sc.nextLine();
isMatch(S,W);
if (isMatch = true){
System.out.println(NG);
}
else {
System.out.println(W);
}
}
}

Answer 1

修正しました。
Javaにおける比較演算子は "==" です。
また、Scannerインスンタスが閉じられていません。

質問者様はEclipseなどのIDEを使ってコードを書いてないのでしょうか？

IDEなしでJavaソースを記載することはオススメではありません。
IDEが気づいてくれる間違いはIDEに任せてしまった方が楽です。

import java.util.Scanner;

public class test {

	// isMatch呼び出しメソッド
	public static void main(String[] args) {

		// オブジェクト初期化
		Scanner sc = new Scanner(System.in);

		// 検索対象文字列を取得
		String S = sc.nextLine();
		String W = sc.nextLine();

		// スキャナーを閉じる
		sc.close();

		// 部分一致か判定(boolean型は == true などの記載は必要ない)
		if (isPartialMatch(S, W)) {
			System.out.println(W + " is " + "partial match of " + S + ".");
		} else {
			System.out.println(W + " is " + "NOT partial match of " + S + ".");
		}
	}

	// WがSの部分一致であるかを検索(部分一致は PartialMatch です)
	public static boolean isPartialMatch(String S, String W) {
		if (S.indexOf(W) != -1) {
			return true;
		} else {
			return false;
		}
	}
}

Comments

[sankakusan]

某サイトで問題を解いている最中でしたのでコンパイラなど使わずに質問してしまいました。
またコンパイルエラーの内容も理解できていませんでした。
ご回答いただきありがとうございました。

Answer 2

isMatchがクラスの外側にあることが原因です。
Javaでは、クラスに属さない、他の言語でいう関数を定義出来ません。

---

シンタックスエラーを消すために、次の三ステップが必要でした。

• isMatchをMainクラスのメソッドとして定義する

前述の理由に寄るものです。

• isMatchをstaticにする

staticメソッドであるmainから呼び出されるためです。

• isMatchの結果を受け取る

次の部分、明らかにおかしいのがわかりますかね？
メソッドの返り値を受け取っていません。

isMatch(S,W);
if (isMatch = true){
    ...

このように修正すべきかと。

boolean matchResult = isMatch(S,W);
if (matchResult){
    ...

ついでに言うと、boolean型の条件式に==trueが含まれるのは冗長です。

• NGを文字列とする

次の部分です。

System.out.println(NG);

単にNGと表示したいのなら、"NG"を引数として与えてください。

---

質問の際は、エラーメッセージを出来るだけ載せるようにしてください。
また、コードはバッククオート三つで括ってください。
よくわからない場合は、マークダウン記法について調べてみてください。

Comments

[sankakusan]

的確に間違いを指摘していただきありがとうございました。
質問に乗せたコードと言い質問内容と言いわかりづらく、聞く側として大変失礼でした。
ご回答いただいた内容を自分なりに言葉にすると
まずメソットをmainクラス囲う必要がある
staticにしてクラスに所属？させる必要がある
メソットの返り値をboolean型の変数に格納してif 文で評価に使うということで間違いないでしょうか？

[LouiS0616]

> まずメソットをmainクラス囲う必要がある
> staticにしてクラスに所属？させる必要がある
mainクラスで囲った時点で、メソッドはクラスに所属しています。
staticという概念は多少分かりづらいのですが、後々分かってくるかと。
一応簡単に説明しておくと、『インスタンスを介しないメソッド』です。

> メソットの返り値をboolean型の変数に格納してif 文で評価に使う
結果的に式の評価が一回であればいいので、今回の場合直接if文に与えるのもアリです。

[LouiS0616]

あと、メソットではありません。メソッド(method)です。
本を買うなりして、体系的に基礎を勉強した方がいいかもしれませんね。