Comments by merano99 (Top 128 by date)

The code snippet is .Net/CLI. Is help needed based on this, or should C++ be used?
A more detailed description of the desired functionality is probably required.
From the existing text I would assume that several form pages should be programmed, each with a forward and back button. For example, MFC AppWizard would generate something like this if you select property sheet with wizard mode.

Since the solution works, you get my 5, but I would prefer a pure C++ solution.

Code is incomplete. Missing struct student and substruct mark.

Show us your source code and ask specific questions about it. There will be no solution service here.
Note on the solution here: The numbers should only be output, storing all numbers in an array does not seem to be required.

In this particular case, it's an easy task. Almost no feature of C++ is used. What the many C++ includes are there for is not clear. There was someone at work who didn't exactly know what he was doing.

I am of course aware of this fact. However, a solution can also be used for learning. Anyone who tries to get along with copy & paste is neither willing to learn anything nor has they understood anything.
A turnkey solution that can be handed over directly without any personal contribution would really not make sense here. I agree.

Almost fine, but would have a two comments:

1. The compiler complains about the const declaration: this pointer cannot be converted from "const person" to "person &".

2. Missing <string> include and code should use std::string.

Remove your code here and improve your question above with it. Please use C code tags.

Remove your code here and improve your question above with it. Please use C code tags.

I would also accept your assumption that a copy'n'paste job went very wrong here. Someone who doesn't see this at first glance probably doesn't know exactly what he's actually doing. Apparently some lines of code have been duplicated and copied into each other. Nobody can read it like that. I removed the obvious duplicates and made small changes. The code looks like a typical Arduino program. Apparently other important information is missing as well.
Since we don't know exactly what the problem is, we can't go any further here without the author writing where exactly the problem is. After the rearrangement, the program can now be compiled without any problems...

The author seems to be very unclear here: The task was actually "You are given a 3x3 matrix", but it was
declared differently here. How the syntax of C works is also extremely unclear. Under such conditions it's
going to be a challenge. Nothing is said about the mathematical background.

Thanks! Allocate the memory is simple: *nbuf = (char*)malloc(length + 1);

The GetComputerNameExA() function works fine in a DLL. Provided that the text fits in the buffer and the terminating 0 is present. The Function strlen can kill your program at this point. I would suggest only write: buffer[length] = 0;

Its your Code, so its up to you to learn that this does not what you want.
case1:cout<<"balance is"<

+5, see comment in my solution

You mean VMR, right?

How the value 10 is calculated in the example is unclear. How exactly is the example calculated?

kbhit (conio.h) it is not only available at Borland.
https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/posix-kbhit

very nice solution. (my 5)
To calculate a tip with range i would suggest
int GenTip()
{
std::random_device rd; // define random device
std::mt19937 gen(rd()); // seed the generator
std::uniform_int_distribution<> distr(0, 10); // define the range
return distr(gen); // generate number
}

Note: the solution functionally covers the task well, but the reference to the circular linked list doesn't seem to be fulfilled?
"(i)Code the above scheme in C++, insert 0 when initializing circular linked list."
On the other hand, a circularly linked list would hardly be sortable.
It is unclear what the task giver was thinking.

You accept negative tips?

Richard had expressed the assumption that you are opening a modal dialog that cannot easily be minimized.
But we don't know what the dialog is used for, why it needs to be minimized and ultimately whether it should be a dialog at all. One might also ask why a modal dialog is used and not a non-modal one, or the other way around, why would you want to minimize it. There are many possibilities to change something here and the statement that it is not possible is not true with some certainty.

OK. Didn't think the solution was that simple. I wrote this as a solution. I would be happy if the solution is accepted.

Did you try SetBkMode(hdc,TRANSPARENT); ?

If a Python solution is desired, why is the question also marked as C++?
Completely different solutions could possibly be considered here.

Only a few letters wrong, see my solution:-)

You tagged the Question with many Tags: C++, C#, QT
The techniques are not compatible and, moreover, one could propose a solution with Win32 or MFC or .NET for C++ alone. It is best to show the existing code and only use tags for techniques that should also be used.
There is not enough information to be able to help here.

But that would be part of the task anyway: "Has appropriate getters and setters"

What does the compiler say?
Everything is required except for the one line with PrepareFaceBank(std::string Path). The prototype should actually be somewhere. However, it is not in the linked document.
In addition, you should write a function yourself that frees up the memory. For testing, it should run like this.

I've just adjusted my suggestion a bit so that it compiles without errors.

See my solution for a wrapper that wraps the C++ data for delivery via a C DLL. Someone in the .NET forum probably knows how to unpack it in C#.

What is your Question?
The task would be to read the text row by row, since the row number should be output.

When the Program crashes bevor the loop the static array
struct star s[201][201];
and other local stuff is possibly to much for your Stack.

I suggest providing the option of specifying the number of rooms in the constructor.
Floor::Floor(int nrooms=0): available_rooms(nrooms){ }

"I tried to solve this code ..."
Please show the code so we can talk about it. Without an approach, you usually can't go any further here.

In atlstr.h you will find:

Line 1296 - typedef CAtlString CString;

so why not use CString?

MFC and ATL are both included in the free Visual-Studio Community Edition.
I can't say whether the atlstr.h could be used with a compiler other than Visual-Studio. You will surely be able to look at the headers.
(By the way: Fortunately, the times of the cropped express versions are over.)

To save and load a string, I would use a string instead of
char value[value_size+1];
You would then no longer have to worry about the length.

Since the field starts at 0, the last index should not be larger than (max_size-1).

size = diap("Input size-->", 0, max_size-1)

The code could fit this project:
A Simple Pie Chart Control - Improved 3-D Pie Chart by PrasadPerera
https://www.codeproject.com/Articles/35759/A-Simple-Pie-Chart-Control-Improved-3-D-Pie-Chart

BOOL CPieChartWnd::Create(LPCTSTR lpCaption, const RECT& rect, CWnd* pParentWnd, UINT nID);

As I have already written in my solution, this is not C ++ code and that it is rather unsolvable with a stack has already been written.
Please comment on the existing solutions and, if necessary, evaluate.

Displaying a message box with [Afx] MessageBox is a blocking operation, so it is normal.
Are you sure that the dialog is not behind other Windows, e.g. behind the IDE window?

The input appears to be read from a file. The attempted solution shown here has little to do with the task and is far too little. In addition, there are no specific questions with which one could help.

Windows that are closed do not generate any more messages. You could just leave the vector element unused.
Before you finally close the window you can of course delete the element from the vector, but you would then have to renumber all remaining windows.
To do this, you would have to write the window handle in the vector. But if this is needed anyway you can save the numbers completely. Simply search for the window handle and use the data under this index element.

Numbering the windows with SetWindowLongPtr () seems to be a good solution if you write a consecutive number after creating the window. It is unclear which information should be stored in data.

Are you shure that this code compiles?
I think several places will not:
e.g.
reinterpret_cast<char *="">
no.="" "<<acno;

I fully agree with you. Before you overwrite the old pointer, you should definitely check whether you don't have to free up the memory beforehand.

//code the 10 functions that are suppose to be coded ....

If you have tried, where are they?

If you solved it in Java what exactly is the problem. The algorithm will probably not differ much, just the syntax.
We usually expect some visible personal contribution here.

We would expect there to be a first program framework where some things have already been prepared.
Should it get caught somewhere, we would expect that the problem is described here and explained what does not work as desired and what you actually want.
It would then also be good to ask a specific question here.

What does Page 2 of 6 mean?

If you accept a variable length list, you cannot hardcode 3.
See my answer on a variable number of elements.

It seems that the hard coded 3 is the real problem.
Another problem arises when someone enters two words with spaces. It would make sense to read in the entire line with getline.

It definitely works.
Did you assign the return value in main as I suggested?

n = getBankinfo();
displayBankinfo(&n);

If you had said from the beginning what exactly you are allowed to and what not you could have given better answers immediately.

When you have to use * as reference you have to write:

void displayBankinfo(Bankinfo *n) {
cout << n->name << endl;
...

Yes, and in addition, it is better to move all of this to a subroutine.

As you can see from our discussion below, there is no known C ++ solution to this frequently asked question. Under Windows there are solutions in C that also work under C ++.

:-)
Out of desperation, because at this point it is neither platform-independent nor with "real" C ++.
If you don't like conio like me, I only see the possibility to use system calls directly. Unfortunately, it then becomes quite cumbersome. I also offered the solution here, see there.
I am not aware of any solution with cin that works without <enter>.

The linked artikel is using <string.h>. It works, but <string> would be the first choice for C ++.

1. Before delete you have to copy
2. Use memcopy instead of the for loop; that goes much faster

The Russian texts are comments generated by the Russian IDE. For me, German is usually also used in these places.

The functions drawPoint() and cycle() are not used, variable totalPoints not defined.
Call of drawPoint(100, i, totalPoints) does not match declaration.

Better to use cmath instead of math.h.

#define _USE_MATH_DEFINES
#include <cmath>

A timer is probably needed that calls OnDraw regularly by callin e.g. InvalidateRect().

Better not to use system("pause").

See:
https://stackoverflow.com/questions/1107705/systempause-why-is-it-wrong

Instead write a function pause() with cin.ignore() and cin.get().

Why not use

cout << "Enter element " << i << "\n";

Definition of arr is a problem here. In Your Solution this is not solved and
you missed to tell.

If you only use Visual Studio the solution is ok.
If the lib should also be usable for compilers other than Visual Studio, it would be advisable get rid of name mangling (also name decoration).

We discussed that yesterday. Havent you seen that?

https://www.codeproject.com/Questions/5315365/This-is-a-Cplusplus-ticket-booking-program-can-any

Instead of doing an idiot check, it might be better to use data types that avoid this.

typedef enum TYP_MONTH {M_JANUARY, M_FEBRUARY, M_MARCH, M_APRIL, M_MAY,
M_JUNE, M_JULY, M_AUGUST, M_SEPTEMBER, M_OCTOBER, M_NOVEMBER, M_DECEMBER};

int GetDaysInMonth( int year, TYP_MONTH month);

If C++ ist allowed, combine it with the solution from CPallini and you have a nice Solution.

You marked this as C ++, but you are writing pure C code. If C ++ is allowed, the solution from KarstenK would be recommended, otherwise that from CPalini.

In C ++ it is common to use <iostream> instead of <stdio.h>.

I don't see any reason to cross out the entire solution.
The correct spelling of COleDateTime and the link to the documentation are important parts of the solution.

As you have noticed, it works with a small example, but it is generally unusual and not recommended, as it causes problems with linking in larger projects.

See: What to put in a header file?
https://docs.microsoft.com/en-us/cpp/cpp/header-files-cpp

Ok. Then see my solution.

If I understand the linked documentation correctly, there are COleDateTime Relational Operators

bool operator==(const COleDateTime& date) const throw();
bool operator!=(const COleDateTime& date) const throw();
bool operator<(const COleDateTime& date) const throw();
bool operator>(const COleDateTime& date) const throw();
bool operator<=(const COleDateTime& date) const throw();
bool operator>=(const COleDateTime& date) const throw();

</stdio.h>

Are you serious?

5.

The function pthread_create looks like POSIX. Since you marked the code as Linux, you should link with -lpthread and maybe you need -lm as well.

Thank you.

Shouldn't the result be in arr1?

The approach is correct, but something goes wrong in the lower part.
It could possibly go like this.

// find base tens
if (number > 19) {
base = number / 10; // base = 3 tens
cout << tens[base] << " ";
number = number % 10; // number = 7
}
// find units up to 19
if (number > 0)
cout << units[number];

Yes, I saw that. But also: Updated 15 hours ago.
And since there was no answer, I decided to suggest something to solve that.

Show the program you tried and explain where the problem is.

Yes you are right. The text has been shortened and there is still an open if bracket in the original. I added. I´m not shure what it does. The suggestions in your solution would certainly be clearer.

The source code at hand is pure C. Why is it tagged as C ++?

hihi;) intiCount is missing a space: int iCount

The Titel ist: "How to use a string instead of a hardcoded value"
The file name is in a variable - in a string, and is therefore very easy to change. It was precisely about the remaining conversion problem, which I believe to have elegantly solved here. I don't think the downvotes are justified.

You ask for "simple marksheet program in c#".
Why do you post this with C and C++ Tags?
Please remove the Tags that you dont mean.

Essential parts of the source code are missing. It cannot be compiled without knowing the exact declaration for the self-defined data types.

error C2065: "str": undeclared identifier
error C2065: "listNode": undeclared identifier
error C2440: "Initialization": "str" ??cannot be converted to "listNode *"

Edit controls are designed to either interpret Enter key presses as a new line (in a multiline edit control) or to ignore it. The way would be to subclass the edit control and call SetFocus () there. This then triggers the desired KillFocus message.

You should add ios :: in as you read.
myfile.open(filename, ios::in | ios::ate | ios::binary);

"I tried all of things but it doesn't work"

Show us what you have tried und tell us what does not work.

To start, you'll take a hello world program, declare some data types, and assign user input. Where is the problem?

I used VS2015 german version and testet your Problem in InitInstance().

The Call of
SetWindowText( m_pMainWnd->m_hWnd, _T("Hello Wörld!"));

changed the TitleBar of the main window has as expected.

I used the System call here, not the MFC call. Is there a diffence?

Can you post a link to your code?

As I wrote hours ago, the file is not read at all. See my solution below.

It is not clear where exact the problem is. We do not have the files "connec.txt" and "mesh.txt" and the compiler does not see errrors.

To access HKLM (HKEY_LOCAL_MACHINE) you need admin rights, e.g. elevate your application using UAC. You can also allow access to the Key by editing the permissions on the relevant registry key.

Code has several warning and space for better

1. // missing header
#include <string>
2. // convert string toupper
string type; cin >> type;
for (auto & c : type) c = toupper(c);
2. // conversion problems
par = lround((work[coun] - 40) * 1.5*fixed[coun]); // round
3. // variables not initialised
int coun = 0;

You are asking for a solution for Arm assembler but you also propose various Intel assembler solutions. The difference is obvious, but it doesn't seem clear to you. A little more preparatory work would be necessary here.
Solution one, three and four even seem to come from the DOS era, that was a long time ago and makes little sense here ...

Why this was marked with C and C ++ here is also not clear to me.

"set it to the appropriate value for the condition of not raining ..."
You screwed up.

Wondering why there is old C Code in this C++ Program.
Would expect:

typedef std::deque<char> ListNode;

class Solution {
public:
Solution(std::string text);
bool lsPalindrome();
...
private:
ListNode _head, _reversed;
};

What is "libraries.h" ?

You're right, but the original question has recently been changed. That made her stand up again. Thanks for the hint.

If you change to char *find_char(char c, const char *s)

C Compiler says:
warning C4090: "return": different "const" qualifiers

C++ Compiler says:
error C2440: "return": "const char *" can not converted to "char *".

error C2065: "greater": undeclared identifier
With other header would work for me.
// #include <algorithm>
#include <functional>

Note: A set cannot store duplicates. Ignoring duplicates was not required here.

Maybe some invisible characters in line 4. Copy the text into a simple editor (notepad or something similar) and save it again.

It was intended as an addition to your approach, not a correction.

When it ppsn is an char-array following is also not korrekt:
int checkPpsn(int ppsn)

Some datatypes should better be integer.
Effort and performance in the string comparison are not optimal.

Another point is the use of unsafe functions that are no longer common these days.
If you dont care use #define _CRT_SECURE_NO_WARNINGS

The Compiler reports several problems, but at least it runs with 32Bit. First is that glut.h is not found. Ich used freeglut.lib instead. Missing braces, and Codelines, that make no sense like: "sprintf(text, "WHERE THERE IS A WILL THERE'S A WAY", 5.00, 8.00);"
I compiled with Visual-Studio and there are many warnings saying "Shortening of "double" in "GLfloat""
At least it runs and shows a grafic screen. See no reason why it should not work in principle.

As Richard wrote, this are at least 4 different things.

Programming a PIC microcontroller (16/32 Bit) in C is common.
But it is very different from a C program that you can use for a System with operating system. One needs knowledge of the architecture and accesses hardware directly.

Developing the hardware yourself requires electronic knowledge. One creates circuit diagrams and layout, orders the circuit board or do that yourself.

Then you have to assemble the components. Good soldering skills are required for this
and measuring devices as well as knowledge of components and how to find defects.

If you want to connect the finished circuit to the PC, that stays nowadays only the USB interface.

If you don't want to write a driver yourself, it makes sense to make it HID-compatible. Just set the clock in the PIC for USB correctly is quite a task.

Accessing the USB interface on the PC side works with many Programming languages. If you wanted to stay close to the hardware there too ISO C ++ certainly not a bad choice.

I have already realized these things in exactly the same way many times.

Whereby I usually order the SMD board, because you can hardly do it yourself with PICs that have many connections and I also want to avoid working with acid.

To start with, I would recommend buying a ready-made PIC-based demo board with a USB interface.

Then you can deal with the programming and gain experience with the hardware.

If you try everything at the same time, with some certainty nothing will come of it in the end.
Alternatively, you could form a team and split the work.

You still did not get this right: "for(int n=1; .."
If you start at 1 or 0 you should count up, not down.

Your for loop "for (int i=1; i<=n; i--)" is wrong.
Init i with n and compare with 0.
Alternative increment i if yo plan to count other direction.

Instead of writing

cout << "Give 1st num: "; cin >> x;
cout << "Give 2nd num: "; cin >> y;

think about writing a method arith::getinput()

Look at my answer to your question before:
https://www.codeproject.com/Questions/5301659/I-want-to-to-return-in-main

I seems a good hint to solve the problem. The questioner asked what logic is behind the problem and the answer seems appropriate.

Does not compile with Visual-Studio

1. xxx.cpp(87): error C2065: "wtime": not declared
2. xxx.cpp(247): warning C4477: "printf": ... Typ "int **" (several warnings)

Are you really shure you want a loop with id?

"the task was to modify our code so the user can input as many numbers as they can"

Not shure, if scanf("%d", &digit) can realy do this. What happens if you type more than a long can handle?

There are some more errors in the code. e.g. count[digit > 0] does strange things.

I tried "2223334445566677888999222222222441345436257676776573" in VS and the result was -1. The code does not work for digits that are to much.
Better use a loop, read chars and count them.

On 32 Bit Operatin Systems you get a limit at 4GB (2^32). And you probably can allocate more Memory than you have. The OS will swap memory.

Dont know if it possible in C#. In C or C++ you could check the arguments and if they dont fit exit the process without opening a window at all.

Every x86 Prozessor starts in Real-Mode. The use of BIOS-Interrups works and you dont need to switch. Why is ist nessarary to switch?
After switching to Protected Mode you can emulate the Real Mode by switching to Virtual 8086 Mode to access BIOS-Interrups. From Protected Mode you have to Reset the CPU to come back to Real-Mode.

Perhaps this is usefull:
https://stackoverflow.com/questions/59894016/switch-from-protected-mode-to-real-mode

Perhaps this will help to begin
Accessing Pixels with CreateDIBSection
https://forums.codeguru.com/showthread.php?526563-Accessing-Pixels-with-CreateDIBSection

The linked tutorial is really bad.


"conio.h is a C header file used mostly by MS-DOS compilers to provide console input/output.[1] It is not part of the C standard library or ISO C, nor is it defined by POSIX."
https://en.wikipedia.org/wiki/Conio.h
Better never use it - neither for C nor for C ++

Why declare Id and Age as int?
Is it usefull to reach neagtiv?

Why declare name as "char Name[25]"?
Would expect string here.

This is what i found in Microsoft Hardware-Dev-Center:

Customize the Windows performance power slider

https://docs.microsoft.com/de-de/windows-hardware/customize/desktop/customize-power-slider

There is no way ... from the UI, although it is possible from the cmd line (via powercfg).

Slider availability
The Windows power slider is available for AMD and Intel platforms running Windows 10, build
1709 and newer builds of Windows. It is not available on devices with ARM64 processors.
The slider will appear on a device only when the Balanced power plan, or any plan that is
derived from Balanced, is selected. There is not an option for either users or OEMs to remove
the slider UX.

The playgame method can be improved too:

void playgame() {
  std::string input;

  while (true) {
	puts("Go player one");
	getline(std::cin, input);
	if (input != "") {
		int index = std::stoi(input) - 1;
		if ((index >= 0) && (index <= 8)) {
			int row = index / 3;
			int col = index % 3;
			char grid_position = map[row][col];
			...

Does it help to set HBRUSH to 0, not RGB ?