Javaのファイル配置とインポートについて

Question

Java初心者です。

###前提・実現したいこと
プログラムからエラーを消したいです
2のソースを1のソースにインポートしているだけだと思うのですが
うまく動きません。
###発生している問題・エラーメッセージ

Exception in thread "main" javax.imageio.IIOException: Can't read input file!
	at javax.imageio.ImageIO.read(ImageIO.java:1301)
	at Test.main(Test.java:9)

###該当のソース
import static img.ImageUtility.*;

import java.awt.image.BufferedImage;
import java.io.*;
import javax.imageio.ImageIO;
public class Test {
public static void main(String[] args) throws IOException {
File f = new File("test.jpg");
BufferedImage read=ImageIO.read(f);
int w = read.getWidth(),h=read.getHeight();
BufferedImage write =
new BufferedImage(w, h, BufferedImage.TYPE_INT_RGB);

    for(int y=0;y<h;y++){
        for(int x=0;x<w;x++){
            int c = read.getRGB(x, y);
            int r = 255-r(c);
            int g = 255-g(c);
            int b = 255-b(c);
            int rgb = rgb(r,g,b);
            write.setRGB(x,y,rgb);
        }
    }

    File f2 = new File("ret.jpg");
    ImageIO.write(write, "jpg", f2);
}

}

Comments

[LouiS0616]

１と２が同じに見えますが、貼り間違いでしょうか。また、エラーの出る場合はエラーメッセージを載せてください。

[LouiS0616]

肝心のソースコードが上手く表示できていないですよ。。。編集したら、ちゃんと思い通りになっているか確認するようにしてくださいね。

[unz.hori]

1のソースに2のソースはimportされていないですね。import OCR.ImageUtility;の文がないです。

[mashumaro0628]

入力してもなぜかeclipceに消されてしまいます。

[unz.hori]

あー、勘違いです。そもそもImageUtilityクラスはstaticではないので、newしてあげる必要がありますね。ImageUtility imageUtility = new ImageUtility();のようにそれでrメソッドとかを使用する場合は、imageUtility.r(c);と書いて呼び出してください。こちらで試したところ他にも例外処理などでエラーがでるみたいです。

[unz.hori]

あと、ソース1を見て思ったのですが、クラス名でmainはちょっとないと思います。

[unz.hori]

パッケージ名も基本は小文字にしてください。

[unz.hori]

あぁ、また勘違い。すみません。ImageUtilityクラスのメソッドはstaticなんですね。では、ソース１から呼び出す場合は、ImageUtility.r(c);のように記述してください。

[unz.hori]

public staticメソッドの場合は、クラス名.メソッド名()の形で呼び出します。newは必要ありません。newが必要なのはstaticでない場合です。

[mashumaro0628]

インスタンス生成はしなくていいんですね。元ソースです。http://nodamushi.hatenablog.com/entry/20111012/1318436587

[mashumaro0628]

もうどうすれば実行できるのかよくわかりません

[unz.hori]

元ソースを見るとメイン処理側はパッケージ指定が無いですね。ImageUtilityクラスはパッケージ指定しているのでメイン処理側でimportしています。

[unz.hori]

メイン処理の例外周りでエラーがでるのはmainメソッドにthrows IOExceptionが無いからです。

[unz.hori]

元ソースとの違いをよく見てみましょう。根気よくやれば動かせるはずです。

[mashumaro0628]

ありがとうございます

[mashumaro0628]

元ソースのパッケージ名が表示されてないのですがデフォルトパッケージに入ってるということでしょうか

[unz.hori]

そうだと思います。パッケージ記述がないので

[mashumaro0628]

エラーはすべて消えたのですがイメージファイルが読み込めないみたいなんです。イメージファイルは各パッケージと同じ階層に直接配置しています

[unz.hori]

開発はどういう環境でやってますか？Windows/Eclipseですか？jpgファイルは多分classファイルと同じ位置にないとだめだと思います。

[unz.hori]

上手くいかない場合はフルパスで指定してみましょう。例えばC:/tmpフォルダを作成して、その中にtest.jpgを配置してFile f = new File("C:/tmp/test.jpg");とか

[mashumaro0628]

Windows/Eclipseです！クラスファイルと同じ階層に置こうとしたらそこにおかさってしまいます

[unz.hori]

おかさって？

[unz.hori]

クラスファイルとはソースファイルではなく、.classファイルでソースファイルをコンパイルして出来るものですよ。

Answer 1

int r = 255-r(c); → int r = 255 - ImageUtility.r(c);
のように記述しましょう。

Comments

[unz.hori]

importについては下記のURLのページを参考にしてください。
http://neos21.hatenablog.com/entry/2016/06/08/010248

[退会済みユーザー]

Static インポートしてるから間違えはそこじゃないよ

クラスファイルと同じ位置にあるファイルを開こうとしたら開けなかったというエラーやよ、この質問は

[unz.hori]

その後、どうなりましたか？解決したのならクローズしてくださいね。