Member 13874636 Ответов: 2

Не знаю, почему это не сработает.


Я написал этот код. Я должен инициировать массив так, как он есть. Профессор хочет, чтобы мы это сделали. Но он не будет проходить через код и заканчиваться. Не знаю почему. Любая помощь, пожалуйста.

Что я уже пробовал:

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */

import java.io.File;
import java.io.FileNotFoundException;
import java.util.Scanner;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import static java.lang.System.out;
import java.util.Arrays;
import java.util.List;
import java.util.Scanner;
import java.io.PrintWriter;
import java.util.ArrayList;
import java.util.Collections;

/**
 *
 *
 * @author tsadams
 */
public class Program2 {
    private int n = 235886;
    private int mid = n/2;
    int correctWords = 0;
    int incorrectWords = 0;
    public Program2() {
        mid = 0;  
    
    }

    /**
     * @param args the command line arguments
     */
    /**
     *
     * @author tsadams
     */
    /**
     * @param args the command line arguments
     */
  

    public static void main(String[] args) throws FileNotFoundException, IOException {
        File inf = new File("dictionary.txt");
        String[] sArray = new String[238886];
        Program2 a = new Program2();
        a.readDictionary(sArray);

        Arrays.sort(sArray);
  int correctRec = 0;
    int incorrectRec = 0;

// String data[] = new String[10];
// data = s.split("\\n");
//System.out.print(contains(data, "A"));
        FileInputStream infO = new FileInputStream(new File("oliver.txt"));
        char let;
        String str = ""; //word to be processed
        int n = 0;
        while ((n = infO.read()) != -1) {
            let = (char) n;
            if (Character.isLetter(let)) {
                str += Character.toLowerCase(let);
            }
            if ((Character.isWhitespace(let) || let == '-') && !str.isEmpty()) {
                if (a.bSearch(sArray, str, 0, 235886) >= 0) {
                    correctRec++;
                } else {
                    incorrectRec++;
                }
                str = "";

            }

        }
        infO.close();
        a.print(correctRec, incorrectRec);

        /**
         *
         *
         */
        /**
         *
         * @param correctRec
         * @param incorrectRec
         * @param incorrectWords
         * @param correctWords
         */
    }

    public void print(int correctRec, int incorrectRec) {
        System.out.println("Out of total words" + (incorrectWords + correctWords));
        System.out.println("Correc" + correctWords);
        System.out.println("incorrect" + incorrectWords);
        System.out.println("Total number of recursive steps is" + (correctRec + incorrectRec));
        System.out.println("The average number of comparisons for a word found =" + correctRec/correctWords);
        System.out.println("The average number of comparisons for a word not found = " + incorrectRec/incorrectWords);
    }

    public void readDictionary(String[] sArray) {
        try {
            int i = 0;
            File f = new File("dictionary.txt");
            Scanner inf = new Scanner(f);
            while (inf.hasNext()) {
                sArray[i] = inf.nextLine();
                i++;
            }
        } catch (FileNotFoundException ex) {
            System.out.println("The dictionary file was not found");
        }
    }

    public int bSearch(String[] sArray, String key, int lowIndex, int highIndex) {
        int rec = 1;  
        if (lowIndex > highIndex) {
            System.out.println(key + "is possibly misspelled");
            incorrectWords++;
            return rec * -1;
        }
        mid = (lowIndex + highIndex) / 2;
        if (sArray[mid].equals(key)) {
            correctWords++;

            return rec;
        } else if (sArray[mid].compareToIgnoreCase(key) > 0) {
            rec++;
            return bSearch(sArray, key, lowIndex, mid - 1);
        } else {
            rec++;
            return bSearch(sArray, key, mid + 1, highIndex);
        }
    }

}

2 Ответов

Рейтинг:
2

OriginalGriff

Компиляция не означает, что ваш код верен! :смеяться:
Подумайте о процессе разработки как о написании электронного письма: успешная компиляция означает, что вы написали письмо на правильном языке - например, на английском, а не на немецком, - а не то, что письмо содержало сообщение, которое вы хотели отправить.

Итак, теперь вы входите во вторую стадию разработки (на самом деле это четвертая или пятая, но вы перейдете к более ранним стадиям позже): тестирование и отладка.

Начните с рассмотрения того, что он делает, и как это отличается от того, что вы хотели. Это важно, потому что это дает вам информацию о том, почему он это делает. Например, если программа предназначена для того, чтобы позволить пользователю ввести число, а затем удвоить его и напечатать ответ, то если бы ввод / вывод был таким:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16
Тогда совершенно очевидно, что проблема заключается в бите, который удваивает его - он не прибавляет себя к себе или умножает его на 2, он умножает его на себя и возвращает квадрат входного сигнала.
Таким образом, вы можете посмотреть на код, и очевидно, что он находится где-то здесь:
private int Double(int value)
   {
   return value * value;
   }

Как только у вас появится идея, что может пойти не так, начните использовать отладчик, чтобы выяснить, почему. Поместите точку останова в первую строку метода и запустите приложение. Когда он достигнет точки останова, отладчик остановится и передаст управление вам. Теперь вы можете запускать свой код построчно (так называемый "одноступенчатый") и просматривать (или даже изменять) содержимое переменных по мере необходимости (черт возьми, вы даже можете изменить код и повторить попытку, если вам это нужно).
Подумайте о том, что должна делать каждая строка кода перед ее выполнением, и сравните это с тем, что она действительно делала, когда вы использовали кнопку "Step over" для выполнения каждой строки по очереди. Он сделал то, что вы ожидали? Если да, то переходите к следующей строке.
Если нет, то почему? Чем это отличается?
Надеюсь, это поможет вам определить, в какой части этого кода есть проблема и в чем она заключается.
Это навык, и его стоит развивать, поскольку он помогает вам как в реальном мире, так и в развитии. И, как и все навыки, он только улучшается при использовании!

Мы не можем сделать это за вас: у нас нет доступа к вашему файлу данных, и мы даже не знаем, что ваш наставник попросил вас произвести, поэтому мы вообще не знаем, что такое "правильно" или "отжимать" для приложения!


Рейтинг:
1

Patrice T

Цитата:
Но он не будет проходить через код и заканчиваться. Не знаю почему.

Как насчет того, чтобы иметь код, говорящий вам, что он делает, а что нет ?

Ваш код ведет себя не так, как вы ожидаете, и вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что вы должны делать, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.
Отладчик - Википедия, свободная энциклопедия[^]
http://docs.oracle.com/javase/7/docs/technotes/tools/windows/jdb.html[^]
https://www.jetbrains.com/idea/help/debugging-your-first-java-application.html[^]
Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.