C / C++ / MFC

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

27-Dec-17 0:17

no, it doen't take input for first loop, it just skips to the second one, does'nt matter from where i is initialized.
Please run the code you will understand.

Richard MacCutchan26-Dec-17 23:29

Richard MacCutchan

26-Dec-17 23:29

Why are you asking for both team names if they are equal? You already have team_1 name so you only need a different name for team_2. You do not need to clear the input array as it will get overwritten on the next gets call. And it skips the first loop because scanf leaves a newline character in the input buffer which then is consumed by the first call to gets. If you did as I have suggested elsewhere you could figure all these things out for yourself.

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

Tarun Jha26-Dec-17 23:43

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

26-Dec-17 23:43

are you suggesting that i initialize input to zero before applying gets() as you told me to do in the previous question answered by you.

Richard MacCutchan27-Dec-17 0:28

Richard MacCutchan

27-Dec-17 0:28

No, I am telling you to go and get a proper study guide and learn the C language in complete detail.

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

Tarun Jha27-Dec-17 1:23

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

27-Dec-17 1:23

i will but befoer that can you show me how to correct the buffering problem of scanf() in order to remove newline charecter from gets().

Richard MacCutchan27-Dec-17 1:25

Richard MacCutchan

27-Dec-17 1:25

fflush(stdin) - Google Search[^]

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

Tarun Jha27-Dec-17 2:09

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

27-Dec-17 2:09

And here is the working code Smile | :)

#include <stdio.h>
#include <string.h>

int main()
{
	int i, log, j, c1=0, c2=0;
	char team_1[100], team_2[100], input[100], ch;

	printf("Enter name of Team-1: \n");
	gets(team_1);

	printf("Enter the name of Team-2\n");
	gets(team_2);

	if(strcmp(team_1, team_2)==0)
    {
        printf("Oops! both names are same. Please enter again: \n");
        while(strcmp(team_1, team_2)==0)
        {
            printf("Enter name of Team-1: \n");
            gets(team_1);

            printf("Enter the name of Team-2\n");
            gets(team_2);
        }
    }

    printf("Enter the number of logs: \n");
	scanf("%d%*c", &log);

	for(i=0; i<log; i++)
	{
		printf("\nEnter log-%d\n", i+1);
		gets(input);

		if(strcmp(input, team_1) == 0)
			c1++;

		else if(strcmp(input, team_2) == 0)
			c2++;
		else
			printf("Invalid entry !\n");

        for(j=0; j<=strlen(input); j++)
        {
            input[j]=0;
        }
	}

	if(c1>c2)
		printf("\nTeam-1 wins");
	else if(c1<c2)
		printf("\nTeam-2 wins");
	else
		printf("\nIt's a tie");

	return 0;
}

Tarun Jha27-Dec-17 2:11

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

27-Dec-17 2:11

And here is the working code..

#include <stdio.h>
#include <string.h>

int main()
{
	int i, log, j, c1=0, c2=0;
	char team_1[100], team_2[100], input[100], ch;

	printf("Enter name of Team-1: \n");
	gets(team_1);

	printf("Enter the name of Team-2\n");
	gets(team_2);

	if(strcmp(team_1, team_2)==0)
    {
        printf("Oops! both names are same. Please enter again: \n");
        while(strcmp(team_1, team_2)==0)
        {
            printf("Enter name of Team-1: \n");
            gets(team_1);

            printf("Enter the name of Team-2\n");
            gets(team_2);
        }
    }

    printf("Enter the number of logs: \n");
	scanf("%d%*c", &log);

	for(i=0; i<log; i++)
	{
		printf("\nEnter log-%d\n", i+1);
		gets(input);

		if(strcmp(input, team_1) == 0)
			c1++;

		else if(strcmp(input, team_2) == 0)
			c2++;
		else
			printf("Invalid entry !\n");

        for(j=0; j<=strlen(input); j++)
        {
            input[j]=0;
        }
	}

	if(c1>c2)
		printf("\nTeam-1 wins");
	else if(c1<c2)
		printf("\nTeam-2 wins");
	else
		printf("\nIt's a tie");

	return 0;
}

Vaclav_28-Dec-17 14:24

Vaclav_

28-Dec-17 14:24

You have identified "problem area " , so work on problem area first.

No matter how many books you read , they will NEVER tell you that
MAJORITY of errors are typos, and than you will have hidden SYNTAX errors.
The last errors are programming - LOGICAL errors.
Before you compare real INPUTS, you have to have correct program sequence working so emulating the actual inputs would be next step.

You are starting Backward with analysis of logical / real input errors.

PS You have a "numerical" application , so prefacing printed numbers with "minus" sign" seems little odd.

for(i=0; i<log; i++)
{

#ifdef DEBUG
printf( "Variable i = %d", i);
#endif

printf("Enter log-%d\n", i+1);
#ifdef DEBUG
printf( "Variable i = %d", i);
#endif

gets(input);

if(strcmp(input, team_1) == 0)
c1++;

else if(strcmp(input, team_2) == 0)
c2++;

for(k=0; k<=strlen(input); k++)
{
input[k]=0;
}
}

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

Victor Nijegorodov28-Dec-17 22:17

Victor Nijegorodov

28-Dec-17 22:17

Dear Vaclav_,
could you edit your post to properly format the code snippet?
Otherwise it is very hard to read/understand.

Re: To write a c program to maintain a log of football match b/w 2 teams and then compare the results of the log entered by user and announce the winner.

Tarun Jha31-Dec-17 13:30

Code consuming lot of RAM (Memory)

31-Dec-17 13:30

it works too..

C++

<pre>#include <stdio.h>
#include <string.h>
#include <math.h>

int main()
{
	int log, i, j, k, count1=1, count2=0, c;

	printf("Enter the number of logs: \n");
	scanf("%d%*c", &log);

	char names[log][100];
	
	//Entering names.
	printf("Enter names: \n");
	gets(names[0]);

	//completing in a single loop.
	for(i=1; i<log; i++)
	{
		gets(names[i]);
		if(strcmp(names[i], names[0])!= 0)
		{
			count2++;
			k = i;
			for(j=k+1; j<log; j++)
            {
                gets(names[j]);

                while((strcmp(names[j], names[0])!=0 && strcmp(names[j], names[k])!=0) || ((c=names[j][0])==EOF && c=='\n')) // prevents from entering more than 2 different types of names or giving a blank input.
                {
                    printf("Invalid entry!\n");
                    memset(names[j], 0, sizeof(names[j]));
                    gets(names[j]);
                }
                
                if((strcmp(names[j], names[0])==0))
                {
                	count1++;
				}
				else if((strcmp(names[j], names[k])==0))
				{
					count2++;
				}
            }
            
            if(count1>count2)
            {
            	puts(names[0]);
            	printf("Wins !");
			}
			else if(count2>count1)
			{
				puts(names[k]);
				printf("Wins !");
			}
			else if(count1==count2)
			{
				printf("It's a tie!!");
			}
            break;
		}
		count1++;
	}
	return 0;
}

Classes hierarchy?

Vaclav_24-Dec-17 8:53

Vaclav_

24-Dec-17 8:53

I am trying to come up with “calling scheme” to be able to go "from top to bottom".
For example “clear LCD device” via I2C interface and connected to GPIO.
The “clear sequence from “top” is to
send device specific command - “clear”
output binary values of “clear command ” to SDA/SCL interface
output binary values to GPIO
I have a 3 levels basic / working inheritance scheme - with class representing the device , class representing I2C communication format and class representing the GPIO.
I am not sure plain direct inheritance of chain of classes is going to do the job.
I think I need to come up with “nesting” scheme instead of just plain direct inheritance chain using only instance of the “top” class.
Any suggestions will be appreciated.
Something like this, but it looks too scary
http://www.linuxtopia.org/online_books/programming_books/c++_practical_programming/c++_practical_programming_254.html
Cheers
Vaclav

PS Please do not ask for code or "what is it for ? " , this project is under construction and not ready for public scrutiny.

Re: Classes hierarchy?

Rick York24-Dec-17 11:36

Rick York

24-Dec-17 11:36

I would advise you to try some experiments. It can be very simple. Write a simple class hierarchy with derived classes and have them call a virtual method each of them implements. Within the method, have it output a trace statement (TRACE, printf, cout<<, what ever...) You will see the calling sequence by the order of the output. One thing to be careful of is the order of the calls :

virtual void TestMethod( int n )
{
    __super::TestMethod( n+1 );   // call base class
    trace( _T( "in TestMethod( %d )\n" ), n );
}

// versus : (base class first or base class second)

virtual void TestMethod( int n )
{
    trace( _T( "in TestMethod( %d )\n" ), n );
    __super::TestMethod( n+1 );   // call base class
}

The output will show you which comes first. Implement as many derivation levels as you want, at least three I think. The base class won't have a __super so you will have to comment that off for it. Anyway, I recommend that you try this experiment. It doesn't take long and it is very informative.

User 1350945019-Dec-17 0:21

19-Dec-17 0:21

Below is the simplified version of the code I am using in my project. On running it, I have noticed that it consumes lots of RAM (Memory). I am not getting why is that happening, can anybody guide me in the right direction.

"main.cpp"

C++

#include "ESSolver.h"
using namespace std;

int maxIt;
int minVal;
int maxVal;

int main(){
	int function_eval = 1, nVar = 500, mu = 15, lbd = 100, rhoO = 2, rhoS = mu;
	maxIt = 1000;
	
	if( function_eval == 1){
		minVal = -32; maxVal = 32;
	}
	
	ESSolver es(nVar, mu, lbd, rhoO, rhoS);
	es.function_eval = function_eval;
	es.Init();
	es.Optimize();

	return 0;
}

"ESSolver.cpp"

C++

#include <iostream>
#include <vector>
#include "ESSolver.h"
#include <random>
using namespace std;

ESSolver::ESSolver(int _nVar, int _mu, int _lbd, int _rhoO, int _rhoS):nVar(_nVar), mu(_mu), lbd(_lbd), rhoO(_rhoO), rhoS(_rhoS)
{	
    muPop.resize(mu);
    muSigma.resize(mu);
    muSigma2.resize(mu);
    
    for (int i = 0; i < mu; i++) {
        muPop[i].resize(nVar);
        muSigma2[i].resize(nVar);
    }
    
    lambdaPop.resize(lbd);
    lambdaSigma.resize(lbd);
    lambdaSigma2.resize(lbd);
    
    for (int i = 0; i < lbd; i++) {
        lambdaPop[i].resize(nVar);
        lambdaSigma2[i].resize(nVar);
    }
    
    mulambdaPop.resize(mu+lbd);
    mulambdaSigma.resize(mu+lbd);
    mulambdaSigma2.resize(mu+lbd);
    
    for (int i = 0; i < mu+lbd; i++) {
        mulambdaPop[i].resize(nVar);
        mulambdaSigma2[i].resize(nVar);
    }
    
    rhoPop.resize(rhoO);
    rhoSigma.resize(rhoS);
    rhoSigma2.resize(rhoS);
    
    for (int i = 0; i < rhoO; i++) {
        rhoPop[i].resize(nVar);
    }
    
    for (int i = 0; i < rhoS; i++) {
        rhoSigma2[i].resize(nVar);
    }
    
	return;
}

ESSolver::~ESSolver(void) {
    return;
}

void ESSolver::Init() {
	for (int i = 0; i < mu; i++) {
		for (int j = 0; j < nVar; j++) {
			muPop[i][j] = rand_r(minVal, maxVal);
		}
	}
	
	if (mutOp == 0) {
	    for (int i = 0; i < mu; i++) {
            muSigma[i] = rand01_r();
	    }
	}
	else {
		for (int i = 0; i < mu; i++) {
		    for (int j = 0; j < nVar; j++) {
			    muSigma2[i][j] = rand01_r();
		    }
	    }
    } 
}

void ESSolver::Optimize() {
	int it     = 0; 	
	while (it < maxIt){
		// Manipulating values of 
        // muPop, muSigma, muSigma2, 
        // lambdaPop, lambdaSigma, lambdaSigma2, 
        // mulambdaPop, mulambdaSigma, mulambdaSigma2, 
        // rhoPop, rhoSigma, rhoSigma2
        it = it + 1;
	}
}

int ESSolver::rand_r(int minVal, int maxVal) {
    default_random_engine rng( random_device{}() );
    uniform_int_distribution<int> dist( minVal, maxVal);
    return dist(rng);
}

double ESSolver::rand01_r() {
    default_random_engine rng( random_device{}() );
	uniform_real_distribution<double> dist(0.0, 1.0);
    return dist(rng);
}

"ESSolver.h"

C++

#include <vector>
using namespace std;

extern int maxIt;
extern int minVal;
extern int maxVal;

class ESSolver
{
public:
	ESSolver(int, int, int, int, int);
	~ESSolver(void);
	
    void Init();
    void Optimize();
   
    int rand_r(int, int);								
    double rand01_r();
							
private:
    int nVar;
    int mu;
    int lbd;
    int rhoO;
    int rhoS;
	
	vector<vector<double>> muPop;
	vector<double> muSigma;
	vector<vector<double>> muSigma2;

	vector<vector<double>> lambdaPop;
	vector<double> lambdaSigma;
	vector<vector<double>> lambdaSigma2;

	vector<vector<double>> mulambdaPop;
	vector<double> mulambdaSigma;
	vector<vector<double>> mulambdaSigma2;

	vector<vector<double>> rhoPop;
	vector<double> rhoSigma;
	vector<vector<double>> rhoSigma2;
};

modified 29-Jan-21 21:01pm.

Manish K. Agarwal19-Dec-17 1:24

Manish K. Agarwal

19-Dec-17 1:24

seems mostly with vector resize which you are using extensively.

User 1350945019-Dec-17 1:30

19-Dec-17 1:30

But they will be executed only once during constructor call

modified 29-Jan-21 21:01pm.

Manish K. Agarwal19-Dec-17 1:48

Manish K. Agarwal

19-Dec-17 1:48

single run takes more than 1 MB

modified 19-Dec-17 7:01am.

User 1350945019-Dec-17 3:02

19-Dec-17 3:02

Actual code has threading. When I ran 10 threads independently, my entire ram of 4GB got full.

modified 29-Jan-21 21:01pm.

Jochen Arndt19-Dec-17 1:57

Jochen Arndt

19-Dec-17 1:57

What do you mean by lot of memory?

I think that about 2M are used by the ESSolver instance of your example code.

User 1350945019-Dec-17 3:01

19-Dec-17 3:01

I don't know my entire ram got full

modified 29-Jan-21 21:01pm.

Jochen Arndt19-Dec-17 3:35

Jochen Arndt

19-Dec-17 3:35

It must be somewhere else in your code because I have compiled the example code and it uses 3355 KiB (waiting for input at end of main and checked the process on another shell in Linux).

Jochen Arndt19-Dec-17 3:59

Jochen Arndt

19-Dec-17 3:59

As already noted, I have compiled and run your code. The result is as expected: It consumes less than 4 MiB of memory.

A common case for running out of memory is - besides allocating huge amounts of memory (e.g. many large objects) - assigning newly created instances to pointer member variables without deleting the old instance.

David Crow19-Dec-17 5:12

David Crow

19-Dec-17 5:12

AtineshS wrote:
On running it, I have noticed that it consumes lots of RAM (Memory).

How are you verifying this?

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

User 1350945019-Dec-17 6:14