А защо моя код е грешен и ми прави false Варна и Пловдив???

Защо стойностите ги караш в низходящ ред, какво е предимството?

Защо се налага да употребиш boolean  за всичко, което ще се въведе грешно?

Исках да знам причината за всичко това, а не само да приписвам.

Сърдечно благодаря!!!

import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        String city = scanner.nextLine();
        double sales = Double.parseDouble(scanner.nextLine());
        double profit = 0;

        if (city.equals("Sofia")) {
            if (sales > 0 && sales <= 500) {
                profit = 0.05;
            } else if (sales > 500 && sales <= 1000) {
                profit = 0.07;
            } else if (sales > 1000 && sales <= 10000) {
                profit = 0.08;
            } else if (sales > 1000) {
                profit = 0.12;
            }
            if (city.equals("Varna")) {
                if (sales > 0 && sales <= 500) {
                    profit = 0.045;
                } else if (sales > 500 && sales <= 1000) {
                    profit = 0.075;
                } else if (sales > 1000 && sales <= 10000) {
                    profit = 0.10;
                } else if (sales > 10000)
                    profit = 0.13;
            }
            if (city.equals("Plovdiv")) {
                if (sales > 0 && sales <= 500) {
                    profit = 0.055;
                } else if (sales > 500 && sales <= 1000) {
                    profit = 0.08;
                } else if (sales > 1000 && sales <= 10000) {
                    profit = 0.12;
                } else if (sales > 10000)
                    profit = 0.145;
            }
            if (profit > 0) {
                double result = profit * sales;
                System.out.printf("%.2f", result);
            } else {
                System.out.println("error");
            }


        }
    }
}

Не знам в какво IDE пишеш, но ако е intelliJ, както е по програма в Java basics, би трябвало да ти оцвети или подчертае по някакъв начин проверките за "Varna" и "Plovdiv", че са винаги false... или нещо от сорта. Това е така, защото са ти вложени в проверката за "Sofia"... ако хванеш къдравата скоба след if (city.equals("Sofia")) { <- тази, ще видиш къде се затваря, и че другите две за град са вложени в нея. Не е като да не може да се реши без boolean, но в Java-та има switch за разлика от Python, на който пиша по принцип и тъй като има и default към switch-a, а булевите са нещо много хубаво, което ни улеснява живота, а по условие трябва да се провери дали града е валиден - съм му сложил тази default стойност, която виждаш в решението ми. Последно за проверките и техния ред - въпрос на предпочитание е... това как ти изглежда най-лесно и най-логично за конструиране. Когато почнеш от голямо към малко можеш да правиш проверка само за ">" без да проверяваш дали числото е в някакъв диапазон. Няма особено значение като цяло...

На Python може да се направи нещо от сорта на:

valid_towns = ['Sofia', 'Varna', 'Plovdiv']

if town not in valid_towns or profit < 0:
    print('error')

Може и при градевете да сложиш else и направо да принти 'error' и накрая да проверяваш за валиден профит и пак да принтиш 'error', но сигурно ще има проблем ако и двете са невалидни - ще се изпринти два пъти 'error'... Пробвай различни неща и ще видиш кое работи и кое не.