Java Tablice + Pętla

Question

Witam mam zrobić tak żeby wczytać od użytkownika 10 liczb i potem żeby dodało je do tablicy i pokazało na wyniku końcowym tylko te liczby które są podzielne przez 3 i jestem troche w martwym punkcie ;/

package com.company;

import java.util.Arrays;
import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        int[] tablica= new int[10];
        int i;
        int t=3;
        System.out.println("Podaj 10 liczb");
        for(i=0; i< 10;i++){
            tablica[i]=getInt();
          if(tablica[i]%t==0){
                  System.out.println(tablica[i]);
              }else{System.out.println("ta liczba nie dzieli sie przez 3");}
          }System.out.println(Arrays.toString(tablica));
    }
    public static int getInt(){
        return new Scanner(System.in).nextInt();
    }
}

Answer 1

Proszę całość zoptymalizowana pamiętaj że indeksowanie jest od 0, zatem od 0-9 będzie 10 elementów jest to też optymalniejsze niż dodatkowa rezerwa 1 elementu w pamięci 
 

int[] a=new int[9];
    Scanner sc=new Scanner(System.in);
        System.out.println("Wprowadz liczby...");
        for(int j=0;j==9;j++)
        a[j]=sc.nextInt();

dodatkowo 1 instancja skanera- nie takowe tworzenie nowego obiektu za każdym wywołaniem, potem zaczytanie odpowiednio i zapis 

Answer 2

Byłeś blisko, uporządkujmy kod.  

 

przykład  ( on-line )

import java.util.Arrays;
import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        int length = 10;
        int numbers[] = new int[length];

        System.out.println("Podaj " + length + " liczb");
        for (int i=0; i<length; i++)
            // Pobranie 10 liczb od użytkownika i dodanie ich do tablicy
            numbers[i] = new Scanner(System.in).nextInt();
            
        System.out.println("\nWprowadzone liczby podzielne przez 3");    
            
        for (int i=0; i<length; i++)
            // liczby które są podzielne przez 3 z pominięciem zera
            if (numbers[i] % 3 == 0 && numbers[i] != 0)
                  System.out.println(numbers[i]);
    }
}

 

Comments

   int length = 10;
   int numbers[] = new int[length];

To jest niepotrzebnie ... Boilerplate code 

---

IMO. zmienna length jest przydatna, gdyż została użyta 4 razy. Jeżeli ktoś będzie chciał zmienić length z 10 na 5 i nie powiąże 10 w pozostałych linijkach kodu z tą samą wartością, co jest częstsze niż się wydaje, to buga mamy gwarantowanego.

---

to również możesz zmienić te wartość inline, i korzystać z numbers.length. wcale nie musisz alokować dodatkowej pamięci na prymitywna wartosc ... inta, to jest java to nie jest c++ i to jest język obiektowy a jvm nie jest wydajna