有几个问题。
错误是：

• 原生数组很难管理，VLA是被禁止的。更喜欢使用一些标准容器（这里是vector）;
• P未初始化;
• 循环控制是错误的，你必须测试一个“while”条件，同时索引小于大小（i < j而不是i >= j-1）。

建议如下：

• 不要用大写字母开始变量标识符，首选小写字母（强烈推荐）;
• 喜欢使用有意义的名字，（sum比P更好，array比S更好）;
• 如果可能的话，组织代码，使得计算由靠近在一起的指令进行（在计算之前初始化/声明sum）;
• 优先选择重要消息（"How many doubles you need?"优先于"j:"）
• 不喜欢使用using namespace std;
• i，j.

代码：

#include <iostream>
 #include <vector>

 int main() {
    using size_type = std::vector<double>::size_type;
    size_type n;
    std::cout << "How many doubles? ";
    std::cin >>n;

    // fill an "array" with values from the input
    std::vector<double> array;
    for (size_type i = 0; i < n; i++) {
        double v;
        std::cout << "array[" << i << "]? ";
        std::cin >> v;
        array.push_back(v);
    }   
 
    // sum elements up to the one that is 0
    int sum=0;    
    for (size_type i = 0; i < array.size(); i++) {
        if (array[i] == 0) break;
        else sum += array[i];
    }   
    std::cout << sum;
    return 0;
 }

字符串

像这样的可变长度数组

double S[j] = {};

字符串
其中j不是编译时常量表达式，它们不是标准C++特性。
如果你想处理一个数组，那么你需要动态地分配它，就像

double *S = new int[j];

型
或

double *S = new int[j](); // to zero-initialize elements of the array

型
或

double *S = new int[j]{}; // to zero-initialize elements of the array

型
在程序结束时，你需要释放数组，如

delete [] S;

型
或者使用标准的cintainer std::vector<double>。
两个for循环由于其条件而被跳过

for (int i = 0; i >= j-1; i++)

//..

for (int i = 0; i >= j; i++)

型
如果j至少大于1，因为0不能大于任何正数。
循环应该看起来像

for (int i = 0; i < j; i++)

型
和

for (int i = 0; i < j and S[i] != 0; i++)
{
    P += S[i];
}

型
变量P应具有double类型，并且必须初始化为零

double P = 0.0;

型