Не удается неявно преобразовать тип string в system windows forms textbox

I have to calculate 3 variables which the method is being called from another class. But before I print the result, I have to convert the int result into predetermined letter of value. But my code isn’t work. I have this error on the code:

CS0029 C# Cannot implicitly convert type ‘string’ to ‘System.Windows.Forms.TextBox’
CS0161 C# »: not all code paths return a value

I’ve tried to convert function into string, but the code still won’t work.

This is the class:

 class Nilai
    {
        public int Calculate(int tugas, int uts, int uas)
        {
            int final = (tugas + uts + uas) / 3;
            return final;
        }
    }

This is the Form1:

namespace IP_Calculator
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {

            nilaiHuruf1 = nilaiHuruf();
        }

       public string nilaiHuruf()
        {
            int a = Convert.ToInt32(tugas1.Text);
            int b = Convert.ToInt32(uts1.Text);
            int c = Convert.ToInt32(uas1.Text);
            Nilai vnilai = new Nilai();
            int hasil = vnilai.Calculate(a, b, c);
            if (hasil >= 85)
            {
                nilaiHuruf1.Text = "A";
                return nilaiHuruf1.ToString();
            }
            else if (hasil >= 80)
            {
                nilaiHuruf1.Text = "A-";
                return nilaiHuruf1.ToString();
            }
            else if (hasil >= 75)
            {
                nilaiHuruf1.Text = "B+";
                return nilaiHuruf1.ToString();
            }
            else if (hasil >= 70)
            {
                nilaiHuruf1.Text = "B";
                return nilaiHuruf1.ToString();
            }

            else if (hasil >= 65)
            {
                nilaiHuruf1.Text = "B-";
                return nilaiHuruf1.ToString();
            }

            else if (hasil >= 60)
            {
                nilaiHuruf1.Text = "C+";
                return nilaiHuruf1.ToString();
            }

            else if (hasil >= 55)
            {
                nilaiHuruf1.Text = "C";
                return nilaiHuruf1.ToString();
            }
            else if (hasil >= 45)
            {
                nilaiHuruf1.Text = "D";
                return nilaiHuruf1.ToString();
            }
            else if (hasil <= 44.99)
            {
                nilaiHuruf1.Text = "E";
                return nilaiHuruf1.ToString();
            }
        }
            }

        }

I expect if I call the method from the Class Nilai to the function nilaiHuruf, then I will get the conversion into letter value. After that, I can call the nilaiHuruf function into the button.

First, I suggest to extract method (business logic and UI separation). In your case, the business logics is in Nilai class:

 //TODO: think over making entire class static: static class Nilai
 class Nilai {
   // static: you don't want "this" here
   public static int Calculate(int tugas, int uts, int uas) {
     return (tugas + uts + uas) / 3;
   }

   public static string Mark(int mark) {
     if (mark >= 85)
       return "A";
     else if (mark >= 80)
       return "A-";
     else if (mark >= 75)
       return "B+";
     else if (mark >= 70)
       return "B";
     else if (mark >= 65)
       return "B-";    
     else if (mark >= 60)
       return "C+";
     else if (mark >= 55)
       return "C"; 
     else if (mark >= 45)
       return "D"; 
     else
       return "E"; 
   }
 }

then use it (UI in Form1)

 public string nilaiHuruf() {
   int a = Convert.ToInt32(tugas1.Text);
   int b = Convert.ToInt32(uts1.Text);
   int c = Convert.ToInt32(uas1.Text); 

   // Since Calculate is static, we don't have to create Nilai instance
   string mark = Nilai.Mark(Nilai.Calculate(a, b, c));  

   nilaiHuruf1.Text = mark; // It's Text property which should be assigned

   // We should return mark, say "B+"; not nilaiHuruf1.ToString();
   return mark;
 } 

answered Sep 10, 2019 at 9:43

try to replace :

return nilaiHuruf1.ToString();

into your if statement to :

return nilaiHuruf1.Text;

Exception is your code is not all code paths return value. If you look at your last if, it doesn’t have else. You must return something from all code blocks.

And you just need to return nilaiHuruf1.Text

Alternative solution for you. Works with C# 7 and above.

private void button1_Click(object sender, EventArgs e)
{
    nilaiHuruf1.Text = nilaiHuruf();
}

public string nilaiHuruf()
{
    int.TryParse(tugas1.Text, out int a);
    int.TryParse(uts1.Text, out int b);
    int.TryParse(uas1.Text, out int c);

    Nilai vnilai = new Nilai();

    int hasil = vnilai.Calculate(a, b, c);

    string mark = "";

    switch (hasil)
    {
        case int n when (n >= 85):
            mark = "A";
            break;

        case int n when (n >= 80):
            mark = "A-";
            break;

        case int n when (n >= 75):
            mark = "B+";
            break;

        case int n when (n >= 70):
            mark = "B";
            break;

        //// fill other cases

        default:
            mark = "E";
            break;
    }

    return mark;
}

answered Sep 10, 2019 at 9:50

I’m building very basic BMI Calculator in C# and Win Forms using VS 2012, also I’m quite new to C#. I’m followed some examples and this code should work, but when run code, I’m getting those errors.

Error   3   Argument 1: cannot convert from 'System.Windows.Forms.TextBox' to 'string'  c:\users\dell\documents\visual studio 2012\Projects\bmi_calc\bmi_calc\Form1.cs  44  31  bmi_calc
Error   5   Argument 1: cannot convert from 'System.Windows.Forms.TextBox' to 'string'  c:\users\dell\documents\visual studio 2012\Projects\bmi_calc\bmi_calc\Form1.cs  45  31  bmi_calc
Error   1   Cannot implicitly convert type 'string' to 'System.Windows.Forms.TextBox'   c:\users\dell\documents\visual studio 2012\Projects\bmi_calc\bmi_calc\Form1.cs  39  25  bmi_calc
Error   2   The best overloaded method match for 'double.Parse(string)' has some invalid arguments  c:\users\dell\documents\visual studio 2012\Projects\bmi_calc\bmi_calc\Form1.cs  44  17  bmi_calc
Error   4   The best overloaded method match for 'double.Parse(string)' has some invalid arguments  c:\users\dell\documents\visual studio 2012\Projects\bmi_calc\bmi_calc\Form1.cs  45  17  bmi_calc

Here is my code:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace bmi_calc
{
    public partial class Form1 : Form
    {
        double v;
        double t;
        double r;


        public Form1()
        {
            InitializeComponent();
        }

        private void textBox2_TextChanged(object sender, EventArgs e)
        {

        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void button2_Click(object sender, EventArgs e)
        {
            txtTezina.Clear(); //Btn that resets height and weight field values.
            txtVisina.Clear();
            txtBmiRez = "";
        }

        private void button1_Click(object sender, EventArgs e)
        {
            v = Double.Parse (txtVisina);
            t = Double.Parse (txtTezina);

            r = t / (v * v);

            txtBmiRez.Text = String.Format("{0:f}", r);

        }

        private void button3_Click(object sender, EventArgs e)
        {
            Application.Exit(); // Close app
        }

    }
}

If someone could explain me this, I would be eternally grateful.

The variables txtVisina and txtTezina are TextBoxes but you are trying to use them as strings. All you need to use is their Text property, i.e.

v = Double.Parse (txtVisina.Text);
t = Double.Parse (txtTezina.Text);

answered Nov 18, 2016 at 13:22

You need to access the Text property of a TextBox to get its text. You’re trying to cast the text box itself.

For example, instead of

v = Double.Parse (txtVisina);

the following is correct:

v = Double.Parse (txtVisina.Text);

answered Nov 18, 2016 at 13:22

You need to use the Textbox.Text property to get the value entered in to the text box, for example

string enteredValue = textbox1.Text;

Easy mistake to make when you’re starting out with WinForms. I did it a lot!

answered Nov 18, 2016 at 13:22

for the Errors 3 and 5 you must use .Text property for the TextBoxes.

To assign any string to a textbox you must use following syntax:

textboxid.Text=»»;
and if you want to assign any non string value to it then use:.ToString() function to convert a variable to string before assigning it to textbox.
eg
int i=10;
Textbox1.Text=i.ToString();

answered Nov 18, 2016 at 13:31

Во-первых, я предлагаю извлечь метод (бизнес-логику и разделение пользовательского интерфейса). В вашем случае бизнес-логика находится вNilai учебный класс:

 //TODO: think over making entire class static: static class Nilai
 class Nilai {
   // static: you don't want "this" here
   public static int Calculate(int tugas, int uts, int uas) {
     return (tugas + uts + uas) / 3;
   }

   public static string Mark(int mark) {
     if (mark >= 85)
       return "A";
     else if (mark >= 80)
       return "A-";
     else if (mark >= 75)
       return "B+";
     else if (mark >= 70)
       return "B";
     else if (mark >= 65)
       return "B-";    
     else if (mark >= 60)
       return "C+";
     else if (mark >= 55)
       return "C"; 
     else if (mark >= 45)
       return "D"; 
     else
       return "E"; 
   }
 }

затем используйте его (пользовательский интерфейс в Form1)

 public string nilaiHuruf() {
   int a = Convert.ToInt32(tugas1.Text);
   int b = Convert.ToInt32(uts1.Text);
   int c = Convert.ToInt32(uas1.Text); 

   // Since Calculate is static, we don't have to create Nilai instance
   string mark = Nilai.Mark(Nilai.Calculate(a, b, c));  

   nilaiHuruf1.Text = mark; // It's Text property which should be assigned

   // We should return mark, say "B+"; not nilaiHuruf1.ToString();
   return mark;
 } 

Попробуйте заменить:

return nilaiHuruf1.ToString();

в ваш if заявление на:

return nilaiHuruf1.Text;

Я думаю, это просто. когдаnilaiHuruf1 является текстовым полем, вы не можете напрямую присвоить ему строку — вы должны установить Text свойство текстового поля:

private void button1_Click(object sender, EventArgs e)
{
    nilaiHuruf1.Text = nilaiHuruf();
}

В рамках nilaiHuruf() метод return nilaiHuruf1.ToString();вернет полное имя типа текстового поля, а не то, что вы ожидаете от текстового поля. Использоватьreturn nilaiHuruf1.Text;вместо. В дальнейшемToString()это метод, который есть у каждого объекта.

Обновить

Когда значение nilaiHuruf1.Text изменяется внутри метода nilaiHuruf() зачем вам вернуть это значение и снова установить его на nilaiHuruf1текстовое окно? Удалите назначение следующим образом, и все должно работать:

private void button1_Click(object sender, EventArgs e)
{
    nilaiHuruf();
}

Исключением является то, что ваш код не возвращает значение всех путей кода. Если вы посмотрите на свое последнее «если», то другого у него нет. Вы должны вернуть что-то из всех блоков кода.

И тебе просто нужно вернуться nilaiHuruf1.Text

Альтернативное решение для вас. Работает с C# 7 и выше.

private void button1_Click(object sender, EventArgs e)
{
    nilaiHuruf1.Text = nilaiHuruf();
}

public string nilaiHuruf()
{
    int.TryParse(tugas1.Text, out int a);
    int.TryParse(uts1.Text, out int b);
    int.TryParse(uas1.Text, out int c);

    Nilai vnilai = new Nilai();

    int hasil = vnilai.Calculate(a, b, c);

    string mark = "";

    switch (hasil)
    {
        case int n when (n >= 85):
            mark = "A";
            break;

        case int n when (n >= 80):
            mark = "A-";
            break;

        case int n when (n >= 75):
            mark = "B+";
            break;

        case int n when (n >= 70):
            mark = "B";
            break;

        //// fill other cases

        default:
            mark = "E";
            break;
    }

    return mark;
}

Ошибка Cannot implicitly convert type 'string' to 'System.Windows.Forms.TextBox'возникает, потому что вы назначаете возвращаемую строку из функции непосредственно TextBox. Вам нужно изменить это, чтобы использоватьText свойство — nilaiHuruf1.Text = nilaiHuruf();

Ошибка not all code paths return a valueвозникает, потому что вы не обрабатываете все случаи для значения int. В частности, вы последнийelse if. Измените его на простоelse поскольку вы уже прошли все остальные оценки.

В идеале код должен соответствовать СОП и может быть упрощен до следующего вида:

private void button1_Click(object sender, EventArgs e)
{
    nilaiHuruf1.Text = nilaiHuruf(Convert.ToInt32(tugas1.Text), Convert.ToInt32(uts1.Text), Convert.ToInt32(uas1.Text));
}

public string nilaiHuruf(int a, int b, int c)
{
    string rtn = "";
    Nilai vnilai = new Nilai();
    int hasil = vnilai.Calculate(a, b, c);
    if (hasil >= 85)
    {
        rtn = "A";
    }
    else if (hasil >= 80)
    {
        rtn = "A-";
    }
    else if (hasil >= 75)
    {
        rtn = "B+";
    }
    else if (hasil >= 70)
    {
        rtn = "B";
    }
    else if (hasil >= 65)
    {
        rtn = "B-";
    }
    else if (hasil >= 60)
    {
        rtn = "C+";
    }
    else if (hasil >= 55)
    {
        rtn = "C";
    }
    else if (hasil >= 45)
    {
        rtn = "D";
    }
    else
    {
        rtn = "E";
    }

    return rtn;
}

Поскольку функция nilaiHuruf больше не зависит от каких-либо значений в форме, это может быть возможность переместить его в Nilai класс в зависимости от цели этого класса.