[java] Prośba o sprawdzenie poprawności

dulaman
Użytkownik
Użytkownik
Posty: 1
Rejestracja: 28 lut 2011, o 21:25
Płeć: Mężczyzna
Lokalizacja: Sosnowiec

[java] Prośba o sprawdzenie poprawności

Post autor: dulaman »

Wkrótce mam egzamin z programowania dostałem w związku z tym zadnie które muszę odesłać do północy. Zadanie jest skończone, ale potrzebuje pomocy, aby ktoś je zweryfikował czy jest dobrze zrobione. Za wszelką pomoc z góry dzięki.

Polecenie:
Zdefiniować klasę zarządzającą stosem danych typu String zapisanym w tablicy. Klasa powinna zawierać metody:
-konstruktor tworzący pusty stos;
-NaStos( łańcuch ) - wstawia łańcuch na stos
-ZeStosu() - zwraca łańcuch i usuwa go ze stosu
-Kowertuj() - łańcuch na szczycie stosu zostaje odwrócony (nie stosować funkcji reverse() klasy StringBuffer)
-PiszStos() - wypisanie zawartości stosu na konsolę

Kod:

Kod: Zaznacz cały

class Zad2 {
      int i;
      String stos[] = new String[i];
      String łańcuch;
      public Zad2(String stos[],
              int i){
              this.stos=null;
              this.i=i;
              this.łańcuch=łańcuch;
      }
      public void NaStos (){
          this.stos[1]=this.łańcuch;
      }
      public String ZeStosu (){
          String s = this.stos[1];
          this.stos[1] = null;
          return s;
      }
      public void Konwertuj (){
          this.stos[1] = this.łańcuch;
          String s1 = this.stos[1];
          String s2 = null;
          for (int j; j < s1.length(); j++){
              s2 = s1 + s2;
          }
          this.stos[1] = s2;
      }
      public void PiszStos (){
          for (int j; j < this.i; j++){
              System.out.println(this.stos[j]);
          }
      }
}
Ostatnio zmieniony 28 lut 2011, o 22:31 przez Afish, łącznie zmieniany 1 raz.
Powód: Kody umieszczaj w znacznikach code. Staraj się lepiej nazywać tematy.
Afish
Moderator
Moderator
Posty: 2828
Rejestracja: 15 cze 2008, o 15:45
Płeć: Mężczyzna
Lokalizacja: Seattle, WA
Podziękował: 3 razy
Pomógł: 356 razy

[java] Prośba o sprawdzenie poprawności

Post autor: Afish »

No raczej nie jest dobrze. Źle do tego podszedłeś. Po pierwsze zmienna łańcuch jest niepotrzebna, bo w założeniu łańcuch do wrzucenia na stos powinien pochodzić z zewnątrz. Po drugie rozmiar tablicy nie jest nigdzie ustalony, bo zmienna i jest niezainicjalizowana. Najlepiej zrób tablicę o stałej długości i tylko sprawdzaj, czy nie przekraczasz zakresu. Po trzecie łańcuch dodajesz zawsze do tego samego elementu tablicy, a powinieneś na jej koniec. Swoją drogą nigdzie nie trzymasz informacji, ile jest elementów na stosie. Funkcja konwertuj też nie wygląda dobrze.
ODPOWIEDZ