Както каза колегата @Rogneda. Аз само ще добавя, че предвид въпроса ти, трябва да отделиш повече внимание на условните конструкции.
else ще се изпълни само ако условието на прилежащия му if е false. В случая това е if (sales >= 0). Винаги когато не си сигурен защо се случва нещо, го пусни през дебъгера ред по ред. Изключително е полезно и в 99% от случаите ще разбереш сам къде ти е грешката.

Ако искаш да изпишеш "error" ако не бъде избран нито един от трите посочени града, трябва да използваш if-else if-else или по-чистия вариант според мен - switch-case.

Нещо такова:

switch(town) {
    case "Sofia":
        // code
        break;
    case "Varna":
        // code
        break;
    case "Plovdiv":
        // code
        break;
    default:
        Console.WriteLine("error");
        break;
}

// if (sales >= 0) goes here

Не съм чел условието на задачата, но входът винаги ли е с главна първа буква? Ако има възможност да се въведе друго, сложи едно toLower() да ти е мирна главата :)