Course All lectures for JA purposes - Lecture: 初心者プログラマーがよく犯す間違い

1. オブジェクトを比較する`==`

初心者プログラマーのよくある間違いは、==演算子を使用してオブジェクト (特に文字列) を比較することです。例えば：

Scanner console = new Scanner(System.in);
String s1 = console.nextLine();
String s2 = console.nextLine();
if (s1 == s2)
{
   System.out.println("The strings are equal");
}

このコードでは、2 つの異なる文字列オブジェクトへの参照を比較するため、「文字列は等しい」とは表示されませんif。

コードの正しいバージョンは次のとおりです。

Scanner console = new Scanner(System.in);
String s1 = console.nextLine();
String s2 = console.nextLine();
if (s1.equals(s2))
{
   System.out.println("The strings are equal");
}

2.`String`オブジェクトの変更

初心者プログラマは、クラスのすべてのオブジェクトがimmutableであること、およびStringクラスのすべてのメソッドが新しいオブジェクトを返すこと、つまり現在のオブジェクトは決して変更されないことを忘れがちです。

例：

String s = "Hello";
s.toUpperCase (); // Convert to uppercase

このコードは正しいコードとよく似ていますが、期待どおりに動作しません。このtoUpperCase()メソッドは、呼び出されるオブジェクトを変更しません。正しいコードは次のようになります。

String s = "Hello";
String result = s.toUpperCase(); // Convert to uppercase

3. 配列の要素であるオブジェクトの初期化を忘れる

もう 1 つのよくある間違いは、配列変数の初期化を忘れることです。例：

int[] array;
array[0] = 1;
array[0] = 2;

このコードは機能しません。配列の要素を格納するコンテナオブジェクトへの参照と等しい配列変数を明示的に設定する必要があります。

int[] array = new int[10];
array[0] = 1;
array[0] = 2;

4. インスタンス変数の代わりにローカル変数を使用します。

初心者は、変数に長くて意味のある名前を考えることを好みません。a彼らは、b、、などの 1 文字の名前を頻繁に使用します。iコードに次のような変数がいくつかある場合、これは残酷な行為です。

配列の 100 個のセルに数値 99 を入れます
`class Solution { public static int a = 99; public static int i = 100; public static void main(String[] args) { int[] a = new int[i]; for (int i = 0; i < 10; i++) { a[i] = a; } } }`

上記のコードはコンパイルされません。正しいバージョンは次のようになります。

配列の 100 個のセルに数値 99 を入れます
`class Solution { public static int value = 99; public static int count = 100; public static void main(String[] args) { int[] a = new int[count]; for (int i = 0; i < count; i++) { a[i] = value; } } }`

5. コレクションアイテムの削除

特定の要素をコレクションから削除する必要がある状況がよくあります。コードはおおよそ次のようになります。

ArrayList<Integer> list = new ArrayList<Integer>();
Collections.addAll(list, 0, -5, -7, -12, 5, 15);

for (Integer value: list)
   if (value < 0)
      list.remove(value);

for-each ループを使用してコレクションの要素を同時に走査し、そのコレクションを変更することはできないため、このコードは機能しません。

解決策はいくつかあります。まず、1 つのコレクションを調べて、別のコレクションを変更します。

解決策 1

ArrayList<Integer> list = new ArrayList<Integer>();
Collections.addAll(list, 0, -5, -7, -12, 5, 15);

ArrayList<Integer> copy = new ArrayList<Integer>(list);
for (Integer value: copy)
   if (value < 0)
      list.remove(value);

次に、Java 8 以降、コレクションにremoveIf()はメソッドがあり、どの要素を削除するかを示すルール (ラムダ関数) を渡すことができます。

例：

解決策 2
`ArrayList<Integer> list = new ArrayList<Integer>(); Collections.addAll(list, 0, -5, -7, -12, 5, 15); list.removeIf( x-> x<0 );`

6.`public`修飾子を含む複数のクラスを 1 つのファイルに配置する

ファイル内に存在できるパブリッククラスは 1 つだけです。ファイル内でさらに多くのクラスを宣言できますが、それらはパブリッククラスの内部クラスであるか、修飾子を持たない必要がありますpublic。例：

Solution.java ファイルの内容	ノート
`public class Solution { } public class Main { }`	これは許可されていません。1 つのファイル内に 2 つのパブリッククラスがあります。
`public class Solution { } class Main { }`	しかし、これならできます。Mainクラスはパブリックではありません
`public class Solution { public static class Main { } }`	そして、これを行うことができます。Main クラスはネストされたクラスです

`main()`7. 静的メソッドからクラスの通常の (非静的) メソッドを呼び出す

main()初心者プログラマは、メソッドまたは他の静的メソッドから非静的変数やメソッドにアクセスしようとすることがあります。もちろん、そのようなコードは機能しません。

public class Solution
{
   public int n = 100;
   public int[] createArray()
   {
      return new int[n];
   }

   public static void main(String[]args)
   {
      int[] array = createArray();
   }
}

main メソッドは静的メソッド/変数のみを参照できます。そうでなければ、最初にクラスのインスタンスを作成してSolutionから、そのオブジェクトの非静的メソッドを呼び出す必要があります。例：

解決策 1 解決策 2

public class Solution
{
  public static int n = 100;

  public static int[] createArray()
  {
    return new int[n];
  }

  public static void main(String[]args)
  {
    int[] array = createArray();
  }
}

public class Solution
{
  public int n = 100;

  public int[] createArray()
  {
    return new int[n];
  }

  public static void main(String[]args)
  {
    Solution sol = new Solution();
    int[] array = sol.createArray();
  }
}

8. メソッドのようなコンストラクターの宣言

もう 1 つのよくある間違いは、クラスコンストラクターを誤って宣言することです。コンストラクターの名前はクラスの名前と同じである必要があり、コンストラクターには結果の型がありません。最も一般的な間違いは次のとおりです。

`public class Person { private String value; void Person(String value) { this.value = value; } }`	ここには戻り値の型があってはなりません
`public class Person { private String value; constructor(String value) { this.value = value; } }`	コンストラクター名が無効です。コンストラクターの名前はクラス名と一致する必要があります
`public class Person { private String value; Person(String value) { value = value; } }`	`this` 不足している。変数`value`はそれ自体に割り当てられます
`public class Person { private String value; Person(String value) { this.value = value; } }`	それはすべて正しいです

9. インターフェースの不正な継承

Java の作成者は Java を英語に非常に近づけようとしたため、特定の関連概念に対して異なるキーワードを選択しました。

クラスがクラスを継承するときは、次のextendsキーワードを使用する必要があります。

class Pet
{
}

class Cat extends Pet
{
}

クラスがインターフェイスを継承する場合、次のimplementsキーワードを使用する必要があります。

interface Meow
{
}

class Cat implements Meow
{
}

インターフェイスがインターフェイスを継承する場合は、extends次のキーワードを使用します。

interface Meow
{
}

interface Voice extends Meow
{
}

10.文中`break`の省略`switch`

breakそして、今日の私たちにとっての最後の間違いは、初心者にとっての最後の間違いではありませんが、ステートメントにステートメントを含めないことですswitch。例

間違い右

LocalDate date = LocalDate.now();
DayOfWeek day = date.getDayOfWeek();
switch (day)
{
   case MONDAY:
      System.out.println("Monday");
   case TUESDAY:
      System.out.println("Tuesday");
   case WEDNESDAY:
      System.out.println("Wednesday");
   case THURSDAY:
      System.out.println("Thursday");
   case FRIDAY:
      System.out.println("Friday");
   case SATURDAY:
      System.out.println("Saturday");
   case SUNDAY:
      System.out.println("Sunday");
}

LocalDate date = LocalDate.now();
DayOfWeek day = date.getDayOfWeek();
switch (day)
{
   case MONDAY:
      System.out.println("Monday");
      break;
   case TUESDAY:
      System.out.println("Tuesday");
      break;
   case WEDNESDAY:
      System.out.println("Wednesday");
      break;
   case THURSDAY:
      System.out.println("Thursday");
      break;
   case FRIDAY:
      System.out.println("Friday");
      break;
   case SATURDAY:
      System.out.println("Saturday");
      break;
   case SUNDAY:
      System.out.println("Sunday");
      break;
}

初心者プログラマーがよく犯す間違い

1. オブジェクトを比較する==

2.Stringオブジェクトの変更