Skip to main content
adicionou 278 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
adicionou 1070 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Pra te dar um parâmetro do que seria um código bom para isto. Analise com cuidado pra ver tudo o que pode melhor no que está fazendo:

using static System.Console;

public class Circulo {
    public Circulo(double raio) => Raio = raio;
    public double Raio { get; }
    public double Area => 3.14159 * Raio * Raio;
    public string Imprima() => Raio + " " + Area + " ";
}

public class AppCirculo {
    static int Main() {
        Write("Digite o raio do círculo 1: ");
        if (!double.TryParse(ReadLine(), out var valor)) return 1;
        var circulo1 = new Circulo(valor);
        Write("Digite o raio do círculo 2: ");
        if (!double.TryParse(ReadLine(), out valor)) return 1;
        var circulo2 = new Circulo(valor);
        Write("Digite o raio do círculo 3: ");
        if (!double.TryParse(ReadLine(), out valor)) return 1;
        var circulo3 = new Circulo(valor);
        WriteLine(circulo1.Imprima());
        WriteLine(circulo2.Imprima());
        WriteLine(circulo3.Imprima());
        return 0;
    }
}

Pra te dar um parâmetro do que seria um código bom para isto. Analise com cuidado pra ver tudo o que pode melhor no que está fazendo:

using static System.Console;

public class Circulo {
    public Circulo(double raio) => Raio = raio;
    public double Raio { get; }
    public double Area => 3.14159 * Raio * Raio;
    public string Imprima() => Raio + " " + Area + " ";
}

public class AppCirculo {
    static int Main() {
        Write("Digite o raio do círculo 1: ");
        if (!double.TryParse(ReadLine(), out var valor)) return 1;
        var circulo1 = new Circulo(valor);
        Write("Digite o raio do círculo 2: ");
        if (!double.TryParse(ReadLine(), out valor)) return 1;
        var circulo2 = new Circulo(valor);
        Write("Digite o raio do círculo 3: ");
        if (!double.TryParse(ReadLine(), out valor)) return 1;
        var circulo3 = new Circulo(valor);
        WriteLine(circulo1.Imprima());
        WriteLine(circulo2.Imprima());
        WriteLine(circulo3.Imprima());
        return 0;
    }
}
adicionou 473 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Quando você usa um tipo como retorno na assinatura do método você tem que retornar um objeto daquele tipo em qualquer situação que aconteça na execução do método. Em geral este erro ocorre quando você retorna em um caminho mas não em outro, ou seja, tem uma parte condicional que faz o retorno e outra condição que não faz. Mas neste caso nem tem nada condicional, então deveria retornar alguma coisa de forma simples, mas no caso não tem retorno algum.

Ou então a solução é mudar o tipo do método e dizer que ele retorna void, neste caso não precisa retornar nada mais, porém não pode usar este método onde se espera algum retorno.

Pelo método eu tentaria tentar adivinhar o que ele deveria fazer, então pelo nome eu diria que tem que deixar o tipo double no método getArea() e ter um return. Mas aí eu teria que decidir o que retornar, e complicou porque esse método faz um monte de coisa mas não pega área do círculo, então nem sei o que retornar. Percebeu que mesmo que conserte esse erro seu código ainda estará confuso?

Eu vou chutar que o método seria assim, mas provavelmente isto é um erro (não faz sentido ter o método deste jeito). A classe toda é confusa.

public double GetArea() => 3.14159 * raio * raio;

Note que eu mudei o nome para ficar do jeito que se costuma fazer me C#. E tirei um monte de coisa que achei que não faz sentido. E usei uma sintaxe onde o retorno é implícito, assim não precisa usar o return, mas ele está retornando assim mesmo, ele retorna justamente o cálculo contido ali, seria o mesmo que fazer:

public double GetArea() {
    return 3.14159 * raio * raio;
}

E se for pensar bem em C# o mais certo é usar uma propriedade e não um método para fazer isto, assim:

public double Area => 3.14159 * raio * raio;

E o raio deveria ser assim:

public double Raio { get; set; }

E nem precisa do campo raio que é criado automaticamente. Assim não precisa dos métodos getter/setter que criou que não é idiomático em C#.

Não vejo porque o campo area deva existir, se acha que deve então deveria justificar.

leia também sobre construtores, porque isto também está errado na classe, de fato eu considero que o uso de setter até mesmo na propriedade é equivocado para este exemplo. Acredito que está aprendendo fazer errado e tentei te dar várias dicas do que pode melhorar. Pesquise aqui no site mais sobre tudo o que está tentando fazer, vai aprender muito, não prossiga sempre aprender o jeito certo de fazer as coisas porque isto irá consolidar e sempre continuará errando,

Normalmente usar o ToString() do jeito que está fazendo é considerado abuso, neste caso nada exagerado, mas pode ser considerado errado.

Tem outros problemas no código que não impedem o funcionamento ou que não são perceptíveis se você não digitar dados errados, mas não vou tentar resolver todos eles aqui.

Quando você usa um tipo como retorno na assinatura do método você tem que retornar um objeto daquele tipo em qualquer situação que aconteça na execução do método. Em geral este erro ocorre quando você retorna em um caminho mas não em outro, ou seja, tem uma parte condicional que faz o retorno e outra condição que não faz. Mas neste caso nem tem nada condicional, então deveria retornar alguma coisa de forma simples, mas no caso não tem retorno algum.

Ou então a solução é mudar o tipo do método e dizer que ele retorna void, neste caso não precisa retornar nada mais, porém não pode usar este método onde se espera algum retorno.

Pelo método eu tentaria tentar adivinhar o que ele deveria fazer, então pelo nome eu diria que tem que deixar o tipo double no método getArea() e ter um return. Mas aí eu teria que decidir o que retornar, e complicou porque esse método faz um monte de coisa mas não pega área do círculo, então nem sei o que retornar. Percebeu que mesmo que conserte esse erro seu código ainda estará confuso?

Eu vou chutar que o método seria assim, mas provavelmente isto é um erro (não faz sentido ter o método deste jeito). A classe toda é confusa.

public double GetArea() => 3.14159 * raio * raio;

Note que eu mudei o nome para ficar do jeito que se costuma fazer me C#. E tirei um monte de coisa que achei que não faz sentido. E usei uma sintaxe onde o retorno é implícito, assim não precisa usar o return, mas ele está retornando assim mesmo, ele retorna justamente o cálculo contido ali, seria o mesmo que fazer:

public double GetArea() {
    return 3.14159 * raio * raio;
}

E se for pensar bem em C# o mais certo é usar uma propriedade e não um método para fazer isto, assim:

public double Area => 3.14159 * raio * raio;

E o raio deveria ser assim:

public double Raio { get; set; }

E nem precisa do campo raio que é criado automaticamente. Assim não precisa dos métodos getter/setter que criou que não é idiomático em C#.

Não vejo porque o campo area deva existir, se acha que deve então deveria justificar.

Normalmente usar o ToString() do jeito que está fazendo é considerado abuso, neste caso nada exagerado, mas pode ser considerado errado.

Tem outros problemas no código que não impedem o funcionamento ou que não são perceptíveis se você não digitar dados errados, mas não vou tentar resolver todos eles aqui.

Quando você usa um tipo como retorno na assinatura do método você tem que retornar um objeto daquele tipo em qualquer situação que aconteça na execução do método. Em geral este erro ocorre quando você retorna em um caminho mas não em outro, ou seja, tem uma parte condicional que faz o retorno e outra condição que não faz. Mas neste caso nem tem nada condicional, então deveria retornar alguma coisa de forma simples, mas no caso não tem retorno algum.

Ou então a solução é mudar o tipo do método e dizer que ele retorna void, neste caso não precisa retornar nada mais, porém não pode usar este método onde se espera algum retorno.

Pelo método eu tentaria tentar adivinhar o que ele deveria fazer, então pelo nome eu diria que tem que deixar o tipo double no método getArea() e ter um return. Mas aí eu teria que decidir o que retornar, e complicou porque esse método faz um monte de coisa mas não pega área do círculo, então nem sei o que retornar. Percebeu que mesmo que conserte esse erro seu código ainda estará confuso?

Eu vou chutar que o método seria assim, mas provavelmente isto é um erro (não faz sentido ter o método deste jeito). A classe toda é confusa.

public double GetArea() => 3.14159 * raio * raio;

Note que eu mudei o nome para ficar do jeito que se costuma fazer me C#. E tirei um monte de coisa que achei que não faz sentido. E usei uma sintaxe onde o retorno é implícito, assim não precisa usar o return, mas ele está retornando assim mesmo, ele retorna justamente o cálculo contido ali, seria o mesmo que fazer:

public double GetArea() {
    return 3.14159 * raio * raio;
}

E se for pensar bem em C# o mais certo é usar uma propriedade e não um método para fazer isto, assim:

public double Area => 3.14159 * raio * raio;

E o raio deveria ser assim:

public double Raio { get; set; }

E nem precisa do campo raio que é criado automaticamente. Assim não precisa dos métodos getter/setter que criou que não é idiomático em C#.

Não vejo porque o campo area deva existir, se acha que deve então deveria justificar.

leia também sobre construtores, porque isto também está errado na classe, de fato eu considero que o uso de setter até mesmo na propriedade é equivocado para este exemplo. Acredito que está aprendendo fazer errado e tentei te dar várias dicas do que pode melhorar. Pesquise aqui no site mais sobre tudo o que está tentando fazer, vai aprender muito, não prossiga sempre aprender o jeito certo de fazer as coisas porque isto irá consolidar e sempre continuará errando,

Normalmente usar o ToString() do jeito que está fazendo é considerado abuso, neste caso nada exagerado, mas pode ser considerado errado.

Tem outros problemas no código que não impedem o funcionamento ou que não são perceptíveis se você não digitar dados errados, mas não vou tentar resolver todos eles aqui.

adicionou 55 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 43 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
adicionou 142 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando