1

Tenho minha classe DadosConexao, esta classe é responsável por guardar a string de conexão do banco de dados, ela contem as seguintes propriedades:

Uma propriedade estatística StringConexao.

isServ indica se é servidor ou cliente.

servidor guarda o nome da maquina.

O problema é que eu não consigo mudar a propriedade StringConexao, que no caso e o "Data Source=" uma parte da string que seria o nome da maquina que contem o banco de dados.

Minha classe DadosConexao:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace DAL
{
    public class DadosConexao
    {

        private String servidor;
        private Boolean isServ;       

        public DadosConexao(bool isServ, string servidor)
        {
            this.isServ = isServ;
            this.servidor = servidor;
        }

        public static string StringConexao 
        {
            get 
            {
                if (!isServ)//Da erro aqui
                    servidor = Environment.MachineName.ToString();//Caso isServ for true pega o nome da maquina local.

                return "Data Source=" + servidor + 
                       "\\MINHAINSTANCIA;" + 
                       "Initial Catalog=MINHABASE;" + 
                       "Persist Security Info=True;" + 
                       "User ID=sa;" + 
                       "Password=123456";
            }
        }
    }
}

Erro apresentado pelo Visual Studio:

Error 1 An object reference is required 
for the non-static field, method, 
or property 'DAL.DadosConexao.isServ'

O sistema funciona como cliente ou servidor, se for cliente a propriedade servidor é definida nas configurações pelo usuário.

2
  • já tentou usar if (isServ == false) ? Commented 19/10/2015 às 2:32
  • 1
    Sim, ja foi resolvido o problema desta classe, e estou usando if (isServ).
    – gato
    Commented 19/10/2015 às 22:44

2 Respostas 2

2

Você pode resolver isto de duas formas. Uma é tirar static da propriedade. Veja mais sobre o static.

using System;

public static class Program {
    public static void Main() {
        var con = new DAL.DadosConexao(true, null);
    }
}

namespace DAL {
    public class DadosConexao {

        private string servidor;
        private bool isServ;       

        public DadosConexao(bool isServ, string servidor) {
            this.isServ = isServ;
            this.servidor = servidor;
        }

        public string StringConexao {
            get {
                if (!isServ)
                    servidor = Environment.MachineName.ToString();

                return "Data Source=" + servidor + 
                       "\\MINHAINSTANCIA;" + 
                       "Initial Catalog=MINHABASE;" + 
                       "Persist Security Info=True;" + 
                       "User ID=sa;" + 
                       "Password=123456";
            }
        }
    }
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Mas isto vai exigir criar uma instância da classe e ficar passando ela como parâmetro, ou ficar criando uma nova toda vez que precisa, que no caso do cliente que envolve uma lógica adicional para determinar o servidor. Isto não é ideal. Mais ainda. Se vai fazer isto, seria melhor aplicar o padrão Singleton nela.

Pra falar a verdade acho que a lógica que determina o que usar se for cliente deveria estar dentro desta classe também e não vir de fora.

A outra solução é transformar tudo em static onde precisa setar os dados de isServ e eventualmente de servidor uma vez e pegar o StringConexao quantas vezes quiser, como era esperado:

using System;

public static class Program {
    public static void Main() {
        DAL.DadosConexao.isServ = true;
        var con = DAL.DadosConexao.StringConexao;
    }
}

namespace DAL {
    public static class DadosConexao {
        public static string servidor { get; set; }
        public static bool isServ { get; set; }

        public static string StringConexao {
            get {
                if (!isServ) servidor = Environment.MachineName.ToString();
                return "Data Source=" + servidor + 
                       "\\MINHAINSTANCIA;" + 
                       "Initial Catalog=MINHABASE;" + 
                       "Persist Security Info=True;" + 
                       "User ID=sa;" + 
                       "Password=123456";
            }
        }
    }
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Provavelmente tem como melhorar isto, mas esta é a base.

7
  • Obrigado pela ajuda, a segunda sugestão e ótima.
    – gato
    Commented 18/10/2015 às 1:00
  • Como você determina qual é o servidor quando roda no cliente?
    – Maniero
    Commented 18/10/2015 às 1:02
  • O servidor vai ser determinado no momento da implementação do sistema, uma flag.
    – gato
    Commented 18/10/2015 às 1:03
  • Sim, mas como é implementado?
    – Maniero
    Commented 18/10/2015 às 1:05
  • A fase da implementação ainda falta algumas regras de negócio para serem definidas, para eu ver qual vai ser a forma mais viável e mais simples.
    – gato
    Commented 18/10/2015 às 1:08
1

Você está tentando acessar uma propriedade não-estática dentro de uma propriedade estática.

Pelo que consegui entender da sua regra de negócio, essa propriedade não precisa ser estático. Então, você só precisa trocar a assinatura dela de

public static string StringConexao { ... }

para

public string StringConexao { ... }

Você só vai precisar de um método/propriedade static quando a sua classe não precisa ser instanciada para chama-lo.

1
  • Vlw pela ajuda.
    – gato
    Commented 18/10/2015 às 1:00

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .