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

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

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

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

Q&A

解決済

1回答

697閲覧

インスタンスの失敗,Java

Nabekirin

総合スコア1

Java

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

0グッド

0クリップ

投稿2021/11/28 01:34

編集2021/11/28 02:08

前提・実現したいこと

複数のクラスファイル(12ファイル)にあるメソッドを、メインメソッドファイルの中に作ったdoItメソッドで実行しようとしています。1つのinstructionについて実行が成功したので、次は配列に複数のinstructionを入れて実行しようと考えましたがうまくいきません。

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

エラーは発生していません。該当ソースコードの中の変数instが、doItメソッドによって配列instructの要素を順次読み込んで変わっていく(最初は0で、次に123、その次に456、最後に789の)ようにしたいのですが、変数instは0のままで変化しません。
該当ソースコードは問題の発生している個所を抽出しました。

該当のソースコード

コード public class Test1 { public static void main(String[] args) { Test1 me = new Test1(); me.doIt(); } public int []instruct = {123, 456, 789}; public int inst; public void doIt() { Test2 istg = new Test2(); for(int i=0; i<2; i++) { inst = instruct[i]; istg.I_stage(); } } public class Test2 { int message;     public void I_stage() { Test1 pl = new Test1(); message= pl.inst; } } }

試したこと

関連する情報を当たりましたが、解決には至りませんでした。
宜しくお願いいたします。

補足情報(FW/ツールのバージョンなど)

eclipseを使っています。

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

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

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

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

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

BeatStar

2021/11/28 01:39 編集

コードは"<code>"または"<コード>"のボタンを押して出てくるヤツの中に書きましょう。 今のままでは読みづらいです。 今の状態は、日本語の文章だと、すべてひらがなで左詰め状態で句読点無しのようなものです。 英文であればすべて小文字(あるいはすべて大文字)でスペース無しのようなものです。 thisisnotanswerbutitisadvise とかみたいなものです。読みづらいですよね。 質問は編集できるので編集しましょう。
BeatStar

2021/11/28 01:44

それと、日本語でいいので、「どういう流れなのか」も書いてください。なんかやりたいことがいまいちわからない…
jimbe

2021/11/28 01:58 編集

> 複数のクラスファイル(12ファイル)にあるメソッド Test2 と同様のものが Test13 まであるということでしょうか。 「メインメソッドファイル」とは何でしょう。「main メソッドのあるファイル」= Test1 クラスのことでしょうか。 > 1つのinstructionについて実行が成功したので、次は配列に複数のinstruction 「instraction」? インスタンスのことでしょうか。それとも 0 や 123、456 のことでしょうか。 Test1 で Test2 を作りながら Test2 で Test1 を作っていたり、ファイルとクラスを混同されているようにも思いますし、どこか用語の使用が微妙に感じます。 結局のところ、どのような順にどのクラスのインスタンスのどのメソッドが実行されてどう表示される仕様としたいのでしょうか。
Nabekirin

2021/11/28 01:58

ご指摘ありがとうございます。 「流れ」ですが、 1.mainメソッドでdoItメソッドを呼び出します。 2.doItメソッドの中には実際は12個のメソッドがコードされており、連続して実行させます。 3.問題が発生したのは、その最初のメソッド(ソースコードの中のI_stageメソッド)なので、その他のメソッドは省略しました。 4.I_stageメソッドでTest1クラスの中の変数instを、message= pl.inst;のように参照します。 5.doItメソッドの中のfor文で、instに配列instructの要素を順次読み込ませることで、I_stageメソッドの参照する変数instを変化させたいと考えています。
jimbe

2021/11/28 02:01 編集

> 2. doItメソッドの中には実際は12個のメソッドがコードされており メソッドの中にメソッドは書けません。「メソッドがコードされており」とはどういう意味でしょうか。
BeatStar

2021/11/28 02:04

繰り返します。コードは"<code>"または"<コード>"のボタンを押して出てくるヤツの中に書きましょう。
jimbe

2021/11/28 02:05 編集

I_stage にパラメータとして inst を渡すか、 Test1 を渡せば良いだけに思いますが、ダメなのでしょうか。
BeatStar

2021/11/28 02:07

とりあえず、「コードは"<code>"のやつの中に書く事」と『それぞれのクラスやフィールド等の関係性を提示』しましょう。 なんか、何がしたいのかよくわからないコードになっています。Test1,2 はそれぞれどういう関係でしょうか。名前からも想像がつかないし、コードを読んでも何がしたいのかが不明です。 それに他の方も仰っているように「メソッドの中にメソッド」は書けません。何か勘違いされていませんか?
Nabekirin

2021/11/28 02:17

失礼しました。コードを修正しました。
Nabekirin

2021/11/28 02:25

解決しました。アドバイスありがとうございました!
guest

回答1

0

ベストアンサー

inst = instruct[i];

ここで内容が変わると期待してると思いますが、

そのあとの

istg.I_stage();

では、

public void I_stage() {
Test1 pl = new Test1();
message= pl.inst;
}

と、Test1の新しいインスタンスを作成しているので、

message= pl.inst;

の部分は、

inst = instruct[i];

とは異なるものです。

違うインスタンスで値を共有するならstaticにする方法もありますが・・・
staticだと予期せぬタイミングで変数が更新されることもあるので、

public void I_stage() {

の部分を

public void I_stage(int inst) {

のように引数で受け取るようにします。

投稿2021/11/28 02:04

momon-ga

総合スコア4826

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

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

Nabekirin

2021/11/28 02:24

ご回答ありがとうございます。 public void I_stage(int inst) に加えて、istg.I_stage(inst);と修正することで解決しました。大変助かりました。ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問