質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

解決済

1回答

539閲覧

特定の文字列が何回目で初めて出てくるか調べたい

退会済みユーザー

退会済みユーザー

総合スコア0

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

0クリップ

投稿2018/07/22 12:31

前提・実現したいこと

MStringObj で firstOccurence(MString s) を実装したいのですが、
1.以下のエラーが発生します。
2.78行目の if の () の中で、「もし s を含まない場合は」という条件を表したいのですが、いい方法が思い浮かびません。

発生している問題・エラーメッセージ

MStringObj.java:1: エラー: MStringObjはabstractでなく、MString内のabstractメソッ ドfirstOccurrence(MString)をオーバーライドしません public class MStringObj implements MString { ^ MStringObj.java:78: エラー: charは間接参照できません if (chars[i].length == a.length) { ^ エラー2個

該当のソースコード(MStringObj)

Java

1public class MStringObj implements MString { 2 3 private char[] chars; 4 5 public MStringObj(char[] chars) { 6 if (chars == null) { 7 throw new IllegalArgumentException(); 8 } 9 10 this.chars = chars; 11 } 12 13 public MStringObj() { 14 this.chars = new char[0]; 15 } 16 17 public int length() { 18 return this.chars.length; 19 } 20 21 public boolean contains(char c) { 22 for (int i = 0; i < chars.length; i++) { 23 if (chars[i] == c) { 24 return true; 25 } 26 } 27 return false; 28 } 29 30 public int count(char c) { 31 int x = 0; 32 for (int i = 0; i < chars.length; i++) { 33 if (chars[i] == c) { 34 x++; 35 } 36 return x; 37 } 38 } 39 40 public char[] toCharArray() { 41 return this.chars; 42 } 43 44 public MString append(MString s) { 45 char[] a = s.toCharArray(); 46 char[] b = new char[this.chars.length + a.length]; 47 return new MStringObj(b); 48 } 49 50 public MString replace(MString find, MString replace) { 51 char[] a = find.toCharArray(); 52 char[] b = replace.toCharArray(); 53 char[] c = new char[this.chars.length - a.length + b.length]; 54 return new MStringObj(c); 55 } 56 57 public Integer firstOccurrence(char c) { 58 int x = x; 59 if (this.contains(c) == false) { 60 return null; 61 } else { 62 for (int i = 0; i < chars.length; i++) { 63 if (chars[i] == c) { 64 x = i; 65 } 66 return x; 67 } 68 } 69 } 70 71 public Integer firstOccurence(MString s) { 72 int x = x; 73 char[] a = s.toCharArray(); 74 if (a.length == 0) { 75 return null; 76 } 77 for (int i = 0; i < chars.length; i++) { 78 if (chars[i].length == a.length) { 79 return null; 80 } else { 81 x = i; 82 return x; 83 } 84 } 85 } 86 87}

補足情報

よろしくお願い致します。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

エラーの一つ目はスペルミスです。

java

1public Integer firstOccurence(MString s)

rが一個足りません。
オーバーライドしたい場合はアノテーション@Overrideを付けましょう。

java

1@Override 2public Integer firstOccurence(MString s)

こうすることで、このメソッドの宣言でコンパイルエラーが発生し、メソッドのスペルミスをしていることが明確にわかります。

エラー二つ目は、(char型の変数).lengthとしているためのエラーです。
正しくはこうでは?

java

1if (chars.length == a.length) {

なお、このチェックをするのであればforの前であるべきです。
そもそもこのfirstOccurrenceメソッドは部分文字列の検索ですよね(StringクラスのindexOfに相当)?
なんか言ってることとやってることが完全に違って見えますよ。
このメソッドに関しては最初から考え直すべきでは…?

投稿2018/07/22 12:52

swordone

総合スコア20649

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2018/07/22 20:38

詳しい解説、ありがとうございました。もう一度考えてみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問