0
public class car
{
      public static int carmode;
      public static int fueltype;
      public static int rashod;
      public int bak;
      public static string ftype,cmode;
}

public class car95:car
{
     public car95(int CRM, int b)
     {
          bak = b;
          carmode = CRM;
          fueltype = 95;
          rashod = 10 + carmode;
     }

}

Когда создаю переменную auto не могу ее использовать. Подскажите, что нужно исправить.

private void button1_Click(object sender, EventArgs e)
{
     if (fuel == 95) { car95 auto = new car95(mode, 50); }
     if (fuel == 100) { cardt auto = new cardt(mode, 50); }
     if (fuel == 200) { cargas auto = new cargas(mode, 50); }

}
3
  • 1
    Вам наверняка не нужна "глобальная переменная". Покажите где вы хотите использовать переменную.
    – 4per
    6 дек 2017 в 2:44
  • Но вряд ли Вам нужна глобальная переменная
    – Zufir
    6 дек 2017 в 2:45

2 ответа 2

7

Основная ошибка -- если начали использовать объекты, то откажитесь от идеи использовать статичность и глобальность. Когда разберётесь с объектами, то поймёте где иногда нужна статичность. Так, что убирайте static из ваших классов (как оно вообще скомпилировалось?)

public class car
{
    public int carmode;
    public int fueltype;
    public int rashod;
    public int bak;
    public string ftype,cmode;
}

public class car95:car
{
    public car95(int CRM, int b)
    {
        bak = b;
        carmode = CRM;
        fueltype = 95;
        rashod = 10 + carmode;
    }

}

Вторая ошибка -- области видимости, у переменных они есть свои у каждой.

if (fuel == 95) { car95 auto = new car95(mode, 50); }

У переменной auto область видимости { car95 auto = new car95(mode, 50); } и всё! Не ранее, не позднее к ней не обратиться. Можно1, т.к. форма на которой у вас кнопка button1 это тоже объект, завести поле в форме.

public class Form1 : Form
{
    private car auto;     

    ...
    ... 

    private void button1_Click(object sender, EventArgs e)
    {
        if (fuel == 95) { auto = new car95(mode, 50); }
        if (fuel == 100) { auto = new cardt(mode, 50); }
        if (fuel == 200) { auto = new cargas(mode, 50); }
    }
}

Обратите внимание, что для auto я поставил тип car, а не car95 или иной. В этом третья ваша ошибка -- нельзя для одной переменной или поля назначить три разных типа. Но если все предполагаемые типы наследуются от одного общего, то можно назначить тип предка.

Ну и нельзя не упомянуть:

  1. Реализация на форме логики, несвязанной с UI (интерфейс пользователя), не рекомендуется. Лучше отделять логику приложения от работы с UI. Хабрахабр. Паттерны для новичков: MVC vs MVP vs MVVM

  2. Хоть это и спорный момент, но всё-таки есть Соглашения о написании кода на C# , которые Ваш код и мои поправки нарушают.

  3. Странно, что у Вас столько int используется. Попробуйте enum Краткий обзор языка C#. Перечисления

  4. Публичные поля тоже не рекомендуется использовать, когда разберётесь используйте публичные свойства вместо них. Для чего нужны свойства?

  5. А здесь (Как объявить глобальную переменную уровня проекта) есть ответ про глобальные переменные.

3

Вообще, использовать глобальные переменные не желательно. Всё таки, я бы порекомендовал пересмотреть дизайн вашего приложения. Но если уж очень хочется - то можно оформить переменные в виде статических полей статического класса. Технически - это глобальные переменные. Только обращаться к ним нужно через имя класса. И мне кажется, судя по приведённому коду, лучше расположить переменные в том же классе, где и объявлен обработчик нажатия кнопки.

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.