C# Discussion Boards - CodeProject

Re: code smell ? C# public class with everything in it declared 'static

8-Feb-23 9:04

I don't see the point of creating a singleton when there is a class that will never be a component of anything else. My apps usually have some sort of "data repository" in memory that needs to be accessed from all over the app; a static class is supremely convenient in this case. "Static" doesn't define the data; it only defines the "references". They could all be observable collections with different subscribers. Dependency injection in this case is obsessing (IMO).

"Before entering on an understanding, I have meditated for a long time, and have foreseen what might happen. It is not genius which reveals to me suddenly, secretly, what I have to say or to do in a circumstance unexpected by other people; it is reflection, it is meditation." - Napoleon I

englebart6-Mar-23 17:22

englebart

6-Mar-23 17:22

If it makes sense as a “bundle” of statics, then make the constructor private.

If it is more class-like, then a singleton would make it easier to substitute a different implementation later.

I agree with you that it should be last resort and usually a code smell.

Running MSBuild Commands from My C# console application.

Madhurima Dutta7-Feb-23 4:56

Madhurima Dutta

7-Feb-23 4:56

i have been using cliwrap for running git commands and passing the arguments. But it seems to have failed while running my msbuild command from my c# console application.

What i am trying-
var stdOutSetup = new StringBuilder();
var stdErrSetup = new StringBuilder();
var Setup = await Cli.Wrap("msbuild")
.WithArguments("/t:scmclean && /t:setup")
.WithStandardOutputPipe(PipeTarget.ToStringBuilder(stdOutSetup))
.WithStandardErrorPipe(PipeTarget.ToStringBuilder(stdErrSetup))
.ExecuteBufferedAsync();

var stdOut2 = stdOutSetup.ToString();
var stdErr2 = stdErrSetup.ToString();

Console.WriteLine("Build Commands Output :");
Console.WriteLine(stdOut2);
Console.WriteLine(stdErr2);

is there any possible way where i can run msbuild commands from my c# console application?

Re: Running MSBuild Commands from My C# console application.

Dave Kreskowiak7-Feb-23 5:18

Dave Kreskowiak

7-Feb-23 5:18

Yes, you just have to supply the correct command line to run MsBuild. You don't say what the error is that you're getting back, and that's really important when troubleshooting problems.

My guess is going to be you're not supplying the path to MsBuild and/or you're not supplying any required command line parameters for it to know what you want to do.

AND STOPP SPAMMING THE SITE WITH THE SAME QUESTION OVER AND OVER AGAIN!

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

Re: Running MSBuild Commands from My C# console application.

jschell7-Feb-23 9:10

Re: Running MSBuild Commands from My C# console application.

7-Feb-23 9:10

Open a console window that represents the OS shell (console) that your command is supposed to be running in. Open in the root directory.

Then type the following. It will fail I am sure. It will not work in your program until you can get it to run from the shell.

msbuild /t:scmclean && /t:setup

I did not investigate it too much but I am rather certain that however you are doing it (which shell) that that '&&' is wrong.

Madhurima Dutta8-Feb-23 4:46

Madhurima Dutta

8-Feb-23 4:46

I have tried running msbuild /t:scmclean from my command prompt, it works fine , but when i try to call the same command from my c# console application using the code below

namespace BuildCode
{
class BuildCM
{
public static void Main()
{
Process msetup = new Process();
msetup.StartInfo.FileName = "cmd.exe";
msetup.StartInfo.CreateNoWindow = true;
msetup.StartInfo.RedirectStandardInput = true;
msetup.StartInfo.RedirectStandardOutput = true;
msetup.StartInfo.UseShellExecute = false;
msetup.Start();
msetup.StandardInput.WriteLine("msbuild.exe D:\\git\\btf_pi_testrack\\Build\\Tools /t:scmclean");
msetup.StandardInput.Flush();
msetup.StandardInput.Close();
msetup.WaitForExit();

Console.WriteLine(msetup.StandardOutput.ReadToEnd());
Console.ReadKey();

}
}
}

i get an error:
C:\Users\JediAdmin\source\repos\msbuild\bin\Debug\net7.0>msbuild.exe D:\git\btf_pi_testrack\Build\Tools /t:scmclean
MSBuild version 17.4.0+18d5aef85 for .NET
MSBUILD : error MSB1003: Specify a project or solution file. The current working directory does not contain a project or solution file., it fails to run msbuild command for the particular directory
but if i execute msbuild.exe D:\git\btf_pi_testrack\Build\Tools /t:scmclean in command prompt, it works fine.

Also i did a little bit of digging for this issue and came back with Microsoft.Build.Runtime and Microsoft.Build.utilities.core for running msbuild commands from console application, however i couldn't get any code example to understabd how it does that? any idea on this as well?

Re: Running MSBuild Commands from My C# console application.

jschell8-Feb-23 5:07

System.InvalidCastException: 'Unable to cast object of type 'System.Collections.Generic.List`1[FirstMVCApplication.Models.Products]' to type 'FirstMVCApplication.Models.Products'.'

8-Feb-23 5:07

Madhurima Dutta wrote:
msbuild /t:scmclean

First I doubt it ran find from the root directory.

Second that is not the command that you are attempting to run in the code you originally posted.

Third what you posted here does not match what msbuild documentation expects.

MSBuild Command-Line Reference - MSBuild | Microsoft Learn[^]

Member 136517316-Feb-23 21:41

Member 13651731

6-Feb-23 21:41

This is Model Class

public class CartInfo
    {
        [Key]
        public int S_NO { get; set; }
        public virtual int Bill_Id { get; set; }
        public virtual int Ship_Id { get; set; }
        public virtual int Product_Id { get; set; }
        [ForeignKey("Product_Id")]
        public List<Products> Products { get; set; }
        public int Sub_Total { get; set; }
        public int Product_Quantity { get; set; }  
    }
    [Table("Product_Detail")]
    public class Products
    {
      [Key, Column(Order = 0)]
        public int Product_Id { get; set; }
         public string Product_Code { get; set; }
        public string Product_Name { get; set; }
        public int Unit_Price { get; set; }
      
    }

Insert command

string code = (model.Cart[i]).Products[0].Product_Code;
                    var _products = (from p in db.Products where p.Product_Code ==code select p).FirstOrDefault();
                    int Product_Id = _products.Product_Id;
                    CartInfo cart = new CartInfo();
                    cart.Bill_Id = Bill_Id;
                    cart.Ship_Id = Ship_Id;
                    cart.Product_Id = Product_Id;
                    cart.Products = new List<Products>();
                    var pf = new Products();
                    //pf =(from p in db.Products where p.Product_Code == code select p).FirstOrDefault();
                    pf.Product_Code = (model.Cart[i]).Products[0].Product_Code;
                    pf.Product_Name = (model.Cart[i]).Products[0].Product_Name;
                    pf.Unit_Price = (model.Cart[i]).Products[0].Unit_Price;
                    cart.Products.Add(pf);
                    cart.Product_Quantity = (model.Cart[i]).Product_Quantity;
                    cart.Sub_Total = (model.Cart[i]).Sub_Total;
                    db.Cart_Details.Add(cart);   <big>// Getting error on this line.'Unable to cast object of type 'System.Collections.Generic.List`1</big>  
                    db.SaveChanges();

Re: System.InvalidCastException: 'Unable to cast object of type 'System.Collections.Generic.List`1[FirstMVCApplication.Models.Products]' to type 'FirstMVCApplication.Models.Products'.'

Eddy Vluggen7-Feb-23 1:13

Eddy Vluggen

7-Feb-23 1:13

The exception says you can't put a product where a list of products is expected. CartInfo specifies it as a list of products.

Bastard Programmer from Hell Suspicious | :suss:

"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

Re: System.InvalidCastException: 'Unable to cast object of type 'System.Collections.Generic.List`1[FirstMVCApplication.Models.Products]' to type 'FirstMVCApplication.Models.Products'.'

Gerry Schmitz7-Feb-23 13:26

7-Feb-23 13:26

A "Product" is single; products is plural.

xtendd

PV20235-Feb-23 20:43

PV2023

5-Feb-23 20:43

I have extended a class from ListCollectionView for Data Virtualizing. I am able to load after taking reference from this article - Data Virtualization using DataGrid[^] but I am not able to add GroupDescriptions in DataGrid.

Please suggest, what I need to do?

OriginalGriff5-Feb-23 20:44

OriginalGriff

5-Feb-23 20:44

Don't post this under generic forums - if you got the code from an article, then there is a "Add a Comment or Question" button at the bottom of that article, which causes an email to be sent to the author. They are then alerted that you wish to speak to them.
Posting this here relies on them "dropping by" and realising it is for them.

PV20235-Feb-23 20:48

PV2023

5-Feb-23 20:48

I did that but not yet received answer so posting here.

Eddy Vluggen7-Feb-23 1:16

Eddy Vluggen

7-Feb-23 1:16

You won't get much help regarding a specific article here. We'd have to read it before answering.

That sounds lazy.. We're volunteers answering questions, and questions that require to read an entire article are skipped. If you reformulate your question to the problem you're trying to solve may help, as well as a piece of code and/or the message of any exceptions.

Bastard Programmer from Hell Suspicious | :suss:

"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

How do I take datatable values into postgresql

RedDk7-Feb-23 7:05

RedDk

7-Feb-23 7:05

As Freddie has intimated by handing you the business, but not actually saying anything useful, you have to insult the article poster in such a way as to get his goad. That way he'll have to defend his code by further describing some cloudy aspect of it. With any luck, something he adds then will be of use to you.

My experience with article writers is that stroking their sense of entitlement for a couple of years can't do any harm. The side effect of this active participation, your part (especially if you haven't posted a tip/trick/article ... yet) is they get to know you. Sure, you have to be able to dodge interpretive criticism but, hey!

It's only a couple of years. Think of it this way. There's plenty of other stuff her on CP to browse and fill the up the time.

Member 141039872-Feb-23 8:22

Member 14103987

2-Feb-23 8:22

I´m trying to take the entire datatable with its values to postgresql but I seem to do something wrong. Code is as follows:

foreach (DataRow row in dt_IS.Rows)
{
  using (NpgsqlCommand cmdCopy = new NpgsqlCommand("INSERT INTO " + Table + " VALUES(@num1));", Conn))
  {
   object num1 = row.Table.Rows[0][0].ToString();
   cmdCopy.Parameters.AddWithValue("@num1", num1);
   Conn.Open();
   cmdCopy.ExecuteNonQuery();
   Conn.Close();
  }
}

Gerry Schmitz2-Feb-23 9:08

2-Feb-23 9:08

(One problem is that) You're creating a new connection for every row in the data table.

jschell3-Feb-23 5:23

3-Feb-23 5:23

However that idiom is certainly something that could run. It is just not optimal.

Gerry Schmitz3-Feb-23 7:10

3-Feb-23 7:10

Depends on your definition of "run" and "certain". Different standards obviously.

Dave Kreskowiak2-Feb-23 9:11

Dave Kreskowiak

2-Feb-23 9:11

Well, first, you're creating a new command and parameter for every record in the table. You don't need to do that and not doing it will improve performance.

Next, for every row in your Rows collection, you're treating each row as if it has a table itself, and only every looking at the first row and column of that table, no matter which record you're looking at. This makes no sense at all.

Lastly, you're only every inserting a single value per row into your new table. Is this correct? What is the datatype of the values you're sending to the new table? What datatype does the new table expect for that value? For example, you cannot send a string to a column that is expecting numbers, either integers or decimal values.

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

Richard MacCutchan2-Feb-23 21:40

Richard MacCutchan

2-Feb-23 21:40

In addition to the comments by others, you are ignoring the value returned by the call to ExecuteNonQuery, so you have no idea if it succeeded or not.

jschell3-Feb-23 5:30

3-Feb-23 5:30

Member 14103987 wrote:
but I seem to do something wrong.

It would help if you explained why it is 'wrong'. Is there an error? Or does the data just not show up.

Your code is attempting to insert one value into a table one row at a time. That will only work if the table only has one column. You can insert just one value, depending on how the table is created, but in that case your 'Insert' must define that column. The form would look like the following

Insert into MyTable (MyRow1) values(1)

And your code does not have the '(MyRow1)' part.

Second possibility is that because you say 'ToString()' that means the type of num1 is a String. Exactly as you requested.

But because you named it 'num1' it seems likely that you actually want a number. And very possible that AddWithValue() is then translating that to a string value for the database. So if your 'MyRow1' is a database type of 'int' but your code is creating the following, because 'num1' is a string, it will not work. (Notice the ticks around the number.)

Insert into MyTable values('1')

Member 141039873-Feb-23 21:33

Member 14103987

3-Feb-23 21:33

Thanks for the suggestions. Here is an update that would explain everything in more detail.
I have a datatable with values, the first column will always be a string and the rest (5 columns) will have numbers. I have used the code that I shared to make it work for a datagridview but now I want to make it work for a datatable. The way I though was best was to take each row into the postgresql server and store it until its needed later. It seems like a sqladapter seems to be a better way to handle this and therefore I had some help trying to figure this out with the following code:

using (NpgsqlDataAdapter adapter = new NpgsqlDataAdapter("SELECT num1, num2, num3, num4, num5, num6 FROM " + Table + ";", Conn))
  {
    NpgsqlCommand insert = new NpgsqlCommand("INSERT INTO " + Table + " VALUES (@num1, @num2, @num3, @num4, @num5, @num6);", Conn);
    insert.Parameters.Add("@num1", NpgsqlDbType.Text, 200, "1");
    insert.Parameters.Add("@num2", NpgsqlDbType.Text, 50, "2");
    insert.Parameters.Add("@num3", NpgsqlDbType.Text, 50, "3");
    insert.Parameters.Add("@num4", NpgsqlDbType.Text, 50, "4");
    insert.Parameters.Add("@num5", NpgsqlDbType.Text, 50, "5");
    insert.Parameters.Add("@num6", NpgsqlDbType.Text, 50, "6");

    adapter.InsertCommand = insert;
    adapter.MissingSchemaAction = MissingSchemaAction.AddWithKey;

    DataTable table = new DataTable();

    // Generate the DataTable schema.
    adapter.FillSchema(table, SchemaType.Source);

    // Add the new rows to the DataTable, e.g.
    DataRow row = table.NewRow();
    
    //Where I´m unsure what to do. This is obivously not the way to write the code but would appreciate it if I get some help to write this the right way
    row["1"] = row.Table.Rows[0][0].ToString(); //Does not continue after this line
    row["2"] = row.Table.Rows[0][1].ToString();
    row["3"] = row.Table.Rows[0][2].ToString();
    row["4"] = row.Table.Rows[0][3].ToString();
    row["5"] = row.Table.Rows[0][4].ToString();
    row["6"] = row.Table.Rows[0][5].ToString();
    
    table.Rows.Add(row);

    // Save the changes.
    adapter.Update(table);
  }
}

The code seems to work until the actual strings/values need to be put inside the SQL server table(see inside of code to know where the code breaks out). If anyone knows how to make the code work or is willing to show whats missing I would gladly appreciate it.