problem z matura 2018

Question

Witam, uczę się programować. Mam problem z rozwiązaniem zadania 4.3 z matura rozszerzona informatyka 2018. Mógłby ktoś sprawdzić mój kod i napisać co zrobiłem źle w funkcji odleglosc ze wyświetla wszystko? 

Link do arkusza: https://cke.gov.pl/egzamin-maturalny/egzamin-w-nowej-formule/arkusze/2018-2

 

import java.util.*;
import java.io.File;
import java.io.FileNotFoundException;

public class zad4{
    public static void litery(String[]a){

        char[]alfabet={'A','Ą','B','C','Ć','D','E','Ę','F','G','H','I','J','K','L','Ł','M','N','Ń','O','Ó','P','R','S','Ś','T','U','W','Y','Z','Ź','Ż','X','V','Q'};

        int z=0;
        int ile=0;
        int licznik=0;
        int max=0;
        while(z<a.length){
            char[] lit=a[z].toCharArray();
            for(int i=0;i<alfabet.length;i++){
                boolean tak=false;
                for(int j=0;j<lit.length;j++){
                    if(alfabet[i]==lit[j])
                        tak=true;
                }
                if(tak==true){
                    ile++;
                }

            }

            if(ile>max){max=ile;

                licznik=z;}
            ile=0;
            z++;
        }
        System.out.println("\n\nZad4.2");
        System.out.println(max);
        System.out.println(a[licznik]);

    }

    public static void odleglosc(String []a){
        System.out.println("zad4.3");
        for(int i=0;i<a.length;i++){
            boolean tak=true;
            char[] lit=a[i].toCharArray();
            for(int j=0; j<lit.length-1&&tak!=false;j++){
                for(int z=j+1;z<lit.length&&tak!=false;z++){
                    int roznica=(int)lit[j]-(int)lit[z];
                    //System.out.println(i+", "+j+", "+z+", "+roznica);
                    if((roznica>=10)&&(roznica<=-10)){
                        tak=false;}
                }

            }
            if(tak==true)System.out.println(a[i]);
        }
    }

    public static void main()throws FileNotFoundException{
        File file=new File("sygnaly.txt");

        Scanner in=new Scanner(file);
        String[]wiersze=new String[1000];
        String wiersz="";
        for(int i=0;i<1000;i++){
            wiersz=in.nextLine();
            wiersze[i]=wiersz;
            //System.out.println(wiersze[i]);
        }
        System.out.println("zad4.1");
        for(int i=39;i<1000;i+=40){
            String litera=wiersze[i].substring(9, 10);
            System.out.print(litera);

        }
        litery(wiersze);
        odleglosc(wiersze);

    }
}

 

Answer 1

Wkleiłem to do Intellij i od razu powiedział, że ten warunek:

if((roznica>=10)&&(roznica<=-10))

jest zawsze fałszywy. Zamień  && and ||.

Podkreśla też 

&&tak!=false

can be simplified to:

&&tak

A tak na marginesie to troche przesadzasz z porównywaniem każdej liczby ze sobą. Wystarczy różnica między min i max, którą można wyciągnąc na kilka sposobów. W najmniejszej liczbie linijek Twoje zadanie to:

for(String line : Files.readAllLines(Paths.get("sygnaly.txt"))){
    IntSummaryStatistics stats = line.chars().summaryStatistics();
    System.out.println(stats.getMax() - stats.getMin() > 10);
}

 

Comments

dzięki, pomogło:) wiedziałem, że popełniłem jakiś głupi błąd, ale nie sądziłem, że aż tak głupi.

Temat do zamknięcia

---

Polecam zapoznać się ze stremami albo chociaz z podstawowymi kolekcjami List, Map, Set, bo jak zerknąłem z ciekawości na 4.2 to można je napisać o wiele wiele prościej:

        List<Long> distinctChars = Arrays.stream(a)
                .map(s->s.chars().distinct().count())
                .collect(Collectors.toList());

I masz listę zawierającą ilosc różnych liter w każdym słowie.

---

Polecam też język python, w którym zrobisz większość tych zadań w jednej linijce, np 4.2:

print(*max(enumerate(open('przyklad.txt').readlines()), key=lambda a: len(set(a[1]))))

 

---

Myślę, że warto ogarnąć pythona na mature dla samego trollingu sprawdzających. Wszyscy się produkują a tu jeb jedna, dwie linijki na większość zadań :D