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

Prośba o ocenę kodu i wskazanie błędów.
T1
  • Rejestracja:ponad 8 lat
  • Ostatnio:ponad 6 lat
  • Postów:3
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

GregoryI
Już lepiej było wstawić tutaj - raptem 40 linijek, z czego 1/4 jest pusta.
OL
  • Rejestracja:ponad 8 lat
  • Ostatnio:prawie 8 lat
  • Postów:13
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.

EV
  • Rejestracja:prawie 13 lat
  • Ostatnio:7 miesięcy
  • Postów:8
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

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.