C / C++ / MFC

9-Feb-02 0:20

I have a char vector allocated on the heap and want to test if one of the elements is equal with "\\" (simple \, but doen't I need to write \\ to tell the compiler that I mean a \, if you understand).

this is the if-statement...

if( localPath[i] == "\\" )
{
...
}

that gives me this error:

--------------------Configuration: Drag Hit - Win32 Release--------------------
Compiling...
source.cpp
C:\Program\Microsoft Visual Studio\MyProjects\Drag Hit\source.cpp(41) : error C2446: '==' : no conversion from 'char *' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast
C:\Program\Microsoft Visual Studio\MyProjects\Drag Hit\source.cpp(41) : error C2040: '==' : 'int' differs in levels of indirection from 'char [2]'
Error executing cl.exe.

Drag Hit.exe - 2 error(s), 0 warning(s)

------------------------------------
Rickard Andersson, Suza Computing
ICQ#: 50302279
I'm from the winter country SWEDEN!

------------------------------------

Christian Graus9-Feb-02 0:26

9-Feb-02 0:26

When you use the double quotes, I believe a NULL terminator is added to your string. Use single quotes when checking an individual character.

if( localPath[i] == '\\' )

Have I been evicted from your sig already ? Wink | ;-)

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Picture the daffodil. And while you do that, I'll be over here going through your stuff.

Rickard Andersson²⁰9-Feb-02 1:17

9-Feb-02 1:17

Then my app crashes... Cry | :((

Christian Graus wrote:
Have I been evicted from your sig already ?

Well, it didn't work with my HTML tags when I posted some messages... :/
weird... so I change it back.. but I realy want it to work! Smile | :)

------------------------------------
Rickard Andersson, Suza Computing
ICQ#: 50302279
I'm from the winter country SWEDEN!

------------------------------------

Christian Graus9-Feb-02 1:28

9-Feb-02 1:28

Rickard Andersson wrote:
Then my app crashes...

Well, that's the right way to do it. Why does it crash ?

Rickard Andersson wrote:
but I realy want it to work!

Me too - I enjoy my fleeting moments of fame when they come...

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Picture the daffodil. And while you do that, I'll be over here going through your stuff.

Rickard Andersson²⁰9-Feb-02 2:08

9-Feb-02 2:08

here is the code (it's not complete):

int GetFileName(char cPath, char* pFileName)
{
if(cPath == NULL)
{
::MessageBox(NULL,"cPath = NULL","Error",MB_OK);
return -1; // return error }

int len = strlen((char*)cPath);

char* localPath = new char[len];
strcpy(localPath,(char*)cPath);

int chars = 0;
for(int i=len; i>len; i--);
{
chars++;
if(localPath[i] == '\\')
{
char* filename = new char[chars+1];
i++;
for(int j=0; j<chars+1; j++)
{
strncpy(filename+j,(char*)localPath[j],1);
}

::MessageBox(NULL, filename,"filename",MB_OK);

delete [] filename;
}
}
delete [] localPath;

return 0;
}

------------------------------------
Rickard Andersson, Suza Computing
ICQ#: 50302279
I'm from the winter country SWEDEN!

------------------------------------

Rickard Andersson²⁰9-Feb-02 2:15

9-Feb-02 2:15

I mean that the function is not complete! Smile | :)

------------------------------------
Rickard Andersson, Suza Computing
ICQ#: 50302279
I'm from the winter country SWEDEN!

------------------------------------

Christian Graus9-Feb-02 2:33

9-Feb-02 2:33

It's a global function ? I don't know what you're trying to do, I presume you're trying to get the different parts of the filepath, i.e. seperate c:\my documents\my code\project1 into
c:\
my documents
my code
project1 ?

casting a char to a char * is not going to work, you need instead to do this

int len = strlen(&cPath);

This returns the address of the variable, (char*)cPath takes the value of cPath and assumes it is the address of a char array.

Overall, unless this is for an assignment, this is an ugly way of doing things. If you're not using MFC, your code would be a lot prettier if you use std::string. If you are using MFC, CString is another alternative.

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Picture the daffodil. And while you do that, I'll be over here going through your stuff.

Christian Graus9-Feb-02 2:45

9-Feb-02 2:45

I just realised something else that should have been screamingly obvious to me - cPath is a deep copy of one char, it no longer points to the char array that it did in the calling function. You need to pass it in as a char* if you want access to the entire string instead of a copy of the one character.

It's late - I'm going to bed Smile | :)

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Picture the daffodil. And while you do that, I'll be over here going through your stuff.

Ernest Laurentin9-Feb-02 3:59

Ernest Laurentin

9-Feb-02 3:59

cPath needs to be a pointer
declare your function like this:

int GetFileName(const char* cPath, char* pFileName);
OR
int GetFileName(LPCSTR cPath, LPSTR pFileName)

By the way, pFileName was never used in your function

- Dirty hands lead to important discovery...

moliate9-Feb-02 1:54

moliate

9-Feb-02 1:54

I tried it and it seems to work..

char* tst = "hkjsdh\\ks\\dj\\sdsdsd\\sfssf\\sfsfs";

for (int i=0; tst[i]; i++)
 {
  if (tst[i] == '\\')
     printf("[backslash]");
  else
     printf("%c", tst[i]);
 }

I suppose the bug is somewhere else in your program?

/moliate Confused | :confused:

Rickard Andersson²⁰9-Feb-02 5:59

9-Feb-02 5:59

Okay man, now I'm realy stucked...

Let's say that I have a path to a file: "C:\Program\MyApp\tehexe.exe"
My function's (GetFileName(...)) main thing is to return the file name (theexe.exe) from that path.
But can't get it to work!

You have the code for that function in some of my messages here... help me with it!!!!!

------------------------------------
Rickard Andersson, Suza Computing
ICQ#: 50302279
I'm from the winter country SWEDEN!

------------------------------------

Nish Nishant9-Feb-02 7:09

9-Feb-02 7:09

Try this code out :-

char *p=abc; //abc has full path
while(strchr(p,'\\'))
{	
	p=strchr(p,'\\');
	p++;
}
MessageBox(p); //p has filename

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

Michael Dunn9-Feb-02 7:34

Michael Dunn

9-Feb-02 7:34

Are you required to write this yourself? If not, then use PathFindFileName() in shlwapi.dll. Otherwise, use strrchr() to find the last backslash.

char* pLastBkslash = strrchr ( szPathName, '\\' );
char* pFilename;
 
  if ( NULL != pLastBkslash )
    pFilename = pLastBkslash + 1;

--Mike--
"Everyone has figured out what 'service pack' really means, so they had to go and change the language. Perhaps this is what Bill was talking about in the 'security is top priority' letter."
-- Daniel Ferguson, 1/31/2002
My really out-of-date homepage
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Nish Nishant9-Feb-02 7:41

9-Feb-02 7:41

Michael Dunn wrote:
char* pLastBkslash = strrchr ( szPathName, '\\' );

That assumes that there is only one slash???
Or did it implicitly imply that the guy has to repeat that till he gets the last backslash?

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

Michael Dunn9-Feb-02 7:53

Michael Dunn

9-Feb-02 7:53

I used strrchr (notice the extra 'r' meaning "from the right") which finds the last occurrence of a character.

--Mike--
"Everyone has figured out what 'service pack' really means, so they had to go and change the language. Perhaps this is what Bill was talking about in the 'security is top priority' letter."
-- Daniel Ferguson, 1/31/2002
My really out-of-date homepage
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Nish Nishant9-Feb-02 7:57

Re: QuertyPerformanceCounter

9-Feb-02 7:57

Michael Dunn wrote:
I used strrchr (notice the extra 'r' meaning "from the right") which finds the last occurrence of a character.

Oops! Sorry Mike. Blush | :O

I never use that one. Donno why!

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

QuertyPerformanceCounter

alex.barylski8-Feb-02 23:21

alex.barylski

8-Feb-02 23:21

If i'm clocking code speed using QueryPerfomanceCOunter()
would it be more accurate if if set the current process/thread to a highest priority...?

Windows time slicing gives so much exec time to each process higer the priority = more time...?

This would mean less likely an interuption correct...?

Thanx!

p.s.I just d/l 34 Mb file in less than 3 minutes Shaw cable...Incredible... OMG | :OMG:

"An expert is someone who has made all the mistakes in his or her field" - Niels Bohr

8-Feb-02 23:42

Take a look at the CPUTest article, it's related to your question. Most of the people tried it said it's better at real time priority. Just make sure you set the priority back to normal after the loop you want to clock

Fixed size

Mazdak8-Feb-02 23:11

Mazdak

8-Feb-02 23:11

I want my mainframe has fixed size and user
can't resize it,what code shouid I add to
CMainFrame::PreCreateWindow(CREATESTRUCT& cs)
Smile | :)

Mazy

Don't Marry a Person You Can Live With...
Marry Someone You Can Not Live Without

alex.barylski8-Feb-02 23:26

alex.barylski

8-Feb-02 23:26

I would figure

cs.x = FIXED_SIZE;
cs.y = FIXED_SIZE;

Although i dunno if this is enough...I'm thinking you'd probably control fixed sizes in OnSize.

Maybe returning false in PreCreateWindow would work, i dunno i'm too lazy to try it out first... Poke tongue | ;-P

Cheers

"An expert is someone who has made all the mistakes in his or her field" - Niels Bohr

Nish Nishant9-Feb-02 7:30

9-Feb-02 7:30

HockeyDude wrote:
i dunno if this is enough

Nope. Not enough.
He has to remove the WS_THICKFRAME style.

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

alex.barylski8-Feb-02 23:41

alex.barylski

8-Feb-02 23:41

Sorry man I must correct myself.

I thought you were after only resize to a fixed size then hault.

But if all you want is a fixed size...

cs.cx = FIXED;
cs.cy = FIXED;

cs.style = WS_DLGFRAME;

However you could probably use Onresize to control this agin, but it's unessecary..

Ciao!

"An expert is someone who has made all the mistakes in his or her field" - Niels Bohr