java class呼び出しのエラー

Question

###前提・実現したいこと
google.google();を実行し ブラウザにgoogleを表示させたいです

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

表示されない 実行はされていている

###該当のソースコード

package hello;

import java.awt.Desktop;
import java.io.IOException;
import java.net.URI;
import java.net.URISyntaxException;

public class google{

    /**
    * @param args
    */
	public void google() {
		
		Desktop desktop = Desktop.getDesktop();
		String uriString = "http://www.google.co.jp";
		System.out.println("システムが起動しました");
		try {
			URI uri = new URI(uriString);
			desktop.browse(uri);
		} catch (URISyntaxException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
}
////
以下呼び出し
////

google google = new google();

google.google();

###試したこと
classの名前の変更など

###補足情報(言語/FW/ツール等のバージョンなど)
テキストエディタ＆実行環境は eclipse

Comments

[swordone]

この「以下呼び出し」としている周辺の状況が知りたいです。ただmainメソッドから起動しているということでいいのでしょうか？

[maguro027]

はい、mainから呼び出しています

Answer 1

String uriString = "http://www.google.co.jp";
ではなく
String uriString = "http://www.google.co.jp/";
で動きます

以下で表示されました

import java.awt.Desktop;
import java.io.IOException;
import java.net.URI;
import java.net.URISyntaxException;

public class google{

    public static void main(String[] args) {

google g=new google();
g.google();

}


    public void google() {

        Desktop desktop = Desktop.getDesktop();
        String uriString = "http://www.google.co.jp/";
        System.out.println("システムが起動しました");
        try {
            URI uri = new URI(uriString);
            desktop.browse(uri);
        } catch (URISyntaxException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Comments

[退会済みユーザー]

習慣でつい変数は短くしましたが、
google google=new google();
google.google();
でも動きました

Answer 2

訂正1：swordoneさんご指摘により以下は自分の知識の間違いであることがわかりました。

クラス名と同じ名前のメソッドはコンストラクターという特別なメソッドです。

大変失礼しました。

訂正2: コードにコンパイルエラーがあったので訂正

間違い部分========ここから=======

書いておられるコードはコンパイルエラーになるはずです。

クラス名と同じ名前のメソッドはコンストラクターという特別なメソッドです。

new google()

とするとgoogleクラスのインスタンスの領域が割り当てられた後自動的にコンストラクターメソッドgoogleが引数なしで呼び出されます。よって、以下のようにインスタンスを生成した後クラス名と同じメソッドを呼び出すというのは矛盾しています。

google google = new google();
google.google();

間違い部分========ここまで=======

まずは次のようにしてJavaのクラスの生成・コンストラクターの起動・通常のメソッドの呼び出しを区別して書いてみてはいかがでしょうか。

public class Google { // クラス名は大文字で始める習慣にしましょう
  public Google() {   // コンストラクター（戻り値の型は書いてはいけません）
    //とりあえず何もしないがコンストラクターが呼ばれたことを確認してみる
    System.out.println("インスタンスが作られました");
  }

  public void show() {
    Desktop desktop = Desktop.getDesktop();
    String uriString = "http://www.google.co.jp/";
    System.out.println("ブラウザーを起動します");
    try {
      URI uri = new URI(uriString);
      desktop.browse(uri);
      System.out.println("ブラウザーが起動できたみたいです");
    } catch (URISyntaxException e) {
      e.printStackTrace();
    } catch (IOException e) {
      e.printStackTrace();
    }
  }
}

// ブラウザーを表示するには・・・
// まずブラウザーを表示するためのGoogleクラスのインスタンスを作ってみる
Google google = new Google();
// そして表示してみる
google.show();

Comments

[swordone]

文法上は何も問題ありませんよ？
"返り値が設定されていない、クラス名と同名のメソッド"がコンストラクタになるだけです。
googleクラスのgoogleメソッドは返り値型がvoidで指定されているため、コンストラクタではなくメソッドです。
new google()でデフォルトコンストラクタが起動し、生成されたインスタンスのvoidメソッドgoogleを実行しているだけなので、何ら問題ないです。

実際、Ideoneでクラス名と同名のインスタンスvoidメソッドIdeoneを作っても問題なく動作しました。
→http://ideone.com/B88uTd

ただ命名規約上小文字始まりのクラス名は推奨されないでしょうがね。

[KSwordOfHaste]

ほんとですね！・・・自分はずっと勘違いしました。お恥ずかしい限りです。
ご指摘ありがとうございました。

[maguro027]

なるほど！わかりやすい解説ありがとうございます！
すごく勉強になりましたφ(..)メモメモ
しかしちゃんとインスタントも実行でき　URLの最後に/を追加してもコピペでくっつけても起動はできませんでした。。
恐らくソフト側の問題だったのかもしれません。。

[KSwordOfHaste]

む？・・・私の回答は(コードの中身はとりあえず動きはすると思いますが)指摘内容が間違っていますよ？本件の原因はtetratailさんが指摘されたとおりURLの末尾の"/"の有無だと思います。
よってBA(ベストアンサー)はどうぞtetratailさんの方へつけなおしてください(多分つけなおせた気がします)。BAは正しい回答の方へつけるのが順当と思いますので！
---
自己訂正：
> コードの中身はとりあえず動きはすると思いますが
コンパイルエラーがありました。粗忽な回答で恐縮です。