'new' not working to create array?

Question

I am going mental, boiled something down to simple, no matter how or what i try, an array will not work? whats up?

The code is simply a single cpp helloworld from cocos2dx. nothing more,

    double *Array = new double[333];
if (Array == nullptr)
    CCLOG("Error: memory could not be allocated");

//initialize it
for( int i = 0; i != 333; ++i){
    Array[i] = 333 - i;

}

for( int i = 0; i != 333; ++i){

    CCLOG("Hi %ld", Array[i] );     
}

The loop always prints 0....

Ive tried many loops, test, the array is never array. its ALWAYS just an int, or double, or whatever type of array i try?

Any thoguhts?

VS2012 cocos2dx helloworld stripped to nothing but an array now. Windows 10

I agreee with @juanchopanza. Instead of CCLOG, try cout << Array[i] << "\n"; (or printf ("%ld\n", Array[i]); if you prefer). — TonyK
– TonyK, Commented Jan 4, 2015 at 10:28
With a conforming C++ implementation the condition in if (Array == nullptr) will always be false, because new throws an exception if it fails. — Cheers and hth. - Alf
– Cheers and hth. - Alf, Commented Jan 4, 2015 at 10:36
Instead of a raw pointer and new, do consider using std::vector. It takes care of the memory management for you, and is resizable and copyable. — Cheers and hth. - Alf
– Cheers and hth. - Alf, Commented Jan 4, 2015 at 10:36

Serge Ballesta · Accepted Answer · 2015-01-04 10:40:22Z

6

Because you are trying to display a double with %ld format. %ld should be used only for long. IMHO you should :

either convert Array[i] to a long :
```
CCLOG("Hi %ld", (long) Array[i] );
```
or use %g format :
```
CCLOG("Hi %g", Array[i] );
```

Both methods should give you correct display.

answered Jan 4, 2015 at 10:40

Serge Ballesta

150k13 gold badges137 silver badges267 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Elgan Sayer Over a year ago

the formatting was my mistake, i had not noticed to change it after trying, int, double, long, float etc. The problem..went away after a reboot of my machine? std::srray people say? How would this be used dynamically ?

hon2a · Accepted Answer · 2015-01-04 11:11:26Z

0

%ld is for long int. Use %lf for double.

edited Jan 4, 2015 at 11:11

hon2a

7,2545 gold badges47 silver badges60 bronze badges

answered Jan 4, 2015 at 10:44

Pranit Bankar

4732 gold badges7 silver badges21 bronze badges

Comments

Elgan Sayer · Accepted Answer · 2015-01-04 21:04:19Z

0

the formatting was my mistake, i had not noticed to change it after trying, int, double, long, float etc.

The problem..went away after a reboot of my machine?

std::srray? How would this be used dynamically / resized?

answered Jan 4, 2015 at 21:04

Elgan Sayer

113 bronze badges

Collectives™ on Stack Overflow

'new' not working to create array?

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related