Prośba o ocenę kodu i wskazanie błędów.

0

Chciałbym zrobić sklep rowerowy w którym klient może wybrać sobie rodzaj ramy po wpisaniu odpowiedniej komendy. Mój main wygląda teraz tak:

https://pastebin.com/AMAY9Kyc

0

Po pierwsze, jeśli o coś prosisz to pokaż swoim postem, że ci zależy rzeczywiście na tym, a nie wklejasz tak paskudny kod, z masą pustych linijek. Jeśli korzystasz z IDE to ogarnij coś takiego jak auto alignment i jak już doprowadzisz ten kod do porządku to wykorzystaj możliwości formatowania na forum, dopiero później cokolwiek na nie wstawiaj.

0

i to działa w ogóle, czy wrzucasz bez sprawdzenia? Może coś powalę, ale tak na szybkiego to:

  • "char grade ='Rower'" - char to znak, a rower to ciąg znaków, więc String.
  • "char grade ='Rower'; switch (grade)" - definiujesz grade jako 'Rower', a potem chcesz tym switchować, chociaż nigdzie nie zmieniasz wartości zmiennej. Ona zawsze będzie jako 'Rower' i zawsze wywoła defaulta, co byś zauważył, gdybyś odpalił kod. Zmienną do switcha powininieś przypisać inputem, jeśli to ma tak działać.
  • brakuje mi break w case 3 i default
  • wywal te puste linie i nie wstawiaj więcej, niż jeden pusty wiersz, jeśli potrzebujesz zwiększyć czytelność kodu
  • wcięcia zrób jak człowiek

1 użytkowników online, w tym zalogowanych: 0, gości: 1