Use a for loop to iterate over the months: def prepareadres(city,y,m,d): months = [31,28,31,30,31,30,31,31,30,31,30,31] d = d + 1 d0 = 0 m0 = 0 for month in months: if d > month: d = d – month m += 1 if d == sum(months): print(“year complete”) #print(df.iloc[0, :-2]) if (m < 10) and (d … Read more
Refactor the code that is repeated into a method that takes an Action<int[]> parameter. Something like this (untested): void Main() { Random rnd = new Random(Guid.NewGuid().GetHashCode()); int[] ArrayRandom = new int[200000]; for (int j = 0; j < ArrayRandom.Length; j++) ArrayRandom[j] = rnd.Next(int.MaxValue); performSort(“Heap Sort”, ArrayRandom, HeapSort); performSort(“Cocktail Sort”, ArrayRandom, HeapSort); performSort(“Selection Sort”, ArrayRandom, HeapSort); … Read more
Yes, you should use switch statement here. Chaining so many else if’s is considered bad practice. You can read about switch statement here. https://www.w3schools.com/js/js_switch.asp Code would look something like: function likes(names) { if(names.length<0) { return `${names} likes this`; //return early if possible } let answer=””; switch (names.length) { case 0: answer = `no one likes … Read more
There’s a few things I can see you’re misunderstanding based on your attempt to do this. One is the attempt to access variables in locations where they are not in scope. sender and msg are two examples. Another is that you’re not returning msg from the Send method so it can be used in the … Read more
Optimization was the wrong term, but the code can be rewritten to find a better (more reduced, or succinct to the business) logic. X < min(Y, Z) <=> X < Y and X < Z X < max(Y, Z) <=> X < Y or X < Z (less usefull) Then possible rewrites are return ((e … Read more
I’m going to suppose that you are trying to make a deep copy of the argv array, with that being a NULL-terminated array of strings such as the second parameter of a C program’s main() function. The function you present seems to assume that you have already allocated space for the destination array itself; its … Read more
You can make use of CRTP. Add a templated second base class to hold the ID handling code. template <class T> class ComponentID { public: ComponentID(std::string &id_) { // Same as updateID above } }; class Wire: public Component, ComponentID<Wire> { public: Wire(const std::string& name = “Wire”) : Component(name), ComponentID(id_) { } // … }; … Read more
Introduction Code duplication is a common problem in software development, and it can lead to a number of issues such as increased complexity, decreased readability, and decreased maintainability. Fortunately, there are a number of techniques that can be used to remove or minimize code duplication and provide the same functionality and behavior. In this article, … Read more
I work under the assumption that you have several constants like Group.GROUP_TYPE_NAME_HYDRO, and all your methods are just like the one you show us, but with different constants (I believe you should add those other methods to your question, since without them there is no visible repeated code). You have two possible scenarios (again, with … Read more